OpenRefine / OpenRefine / 15868678801

51%

push

github

fix storeBlankColumns: delete only after project parsed (#7272)

* rewrite delete-empty-columns tests to start at parse method

* cover case: delete empty columns after all files evaluated

* delete empty columns only after checking all imported files

* Revert "delete empty columns only after checking all imported files"

This reverts commit 6f7752949.

* reintroduce assertions for exact values

* reintroduce column names

* cover case: delete empty columns but keep file column

* mv storeBlankColumns functionality to parse method

* fix tests (empty -> blank)

* mv creation of options up to ImporterTest

* replace getDefaultTabularImportingOptions with empty options

* optimize determination of blank columns

* deprecate deleteEmptyColumns instead of removing it

* add test deleting blank cols from different positions

* rename test to include column position

* restore original tests for deleting blank cols

* make more options explicit

5680 of 12814 branches covered (44.33%)

Branch coverage included in aggregate %.

14435 of 27074 relevant lines covered (53.32%)

2.47 hits per line