Clarifies tests intent (closes #62)

thbar / kiba

Data processing & ETL framework for Ruby

https://www.kiba-etl.org

Other

1.75k stars 87 forks source link

Closed thbar closed 5 years ago

thbar commented 5 years ago

This PR follows #62 and attempts to clarify a part of the testing.

I've removed a confusingly named test, and split its logic in 2 instead, to:

ensure that a transform yielding records on close will see the yielded records emitted in the pipeline
ensure that a transform not implementing close won't raise an error.