search

yoavnir / specs2016

New version of the specs pipeline stage based on what's in current CMS pipelines
MIT License
1 stars 1 forks source link

Rename TestReader #211

Closed yoavnir closed 1 year ago

yoavnir commented 1 year ago

It's not a test. It's used whenever we don't read input.

Call it something else. Perhaps DummyReader?

Maybe we can forego it altogether?

yoavnir commented 1 year ago

Can ig.processDo() accept a null third argument?

yoavnir commented 1 year ago

In the end I didn't rename TestReader, but I made it so that it's only used in tests, so now the name sort of fits.