curationexperts / cypripedium / 75228403-6da3-4b76-8f5f-fe8e2adc69a7

94%

main: 99%

Build Type

Pull #600

circleci

Committed by mark-dce

Commit Message

[MAINTENANCE] Refactor TarBag to remove global variables

**ISSUE**
A number of instance variables were used to pass values across
methods which made code linkages and dependencies difficult to
analyze.

**RESOLUTION**
This change removes all instance variables in the class and passes
required data as paramaters to the method calls. Where data from
the instance initialization is required, we've replaced references
to instance variables with corresponding attribute readers.

This change also adds more detailed testing of the TAR file that
gets created to confirm it has the esxpected content.

Pull Request Pull Request #600: [MAINTENANCE] Refactor TarBag to remove global variables

Run Details

17 of 17 new or added lines in 3 files covered. (100.0%)

44 existing lines in 6 files now uncovered.

1246 of 1316 relevant lines covered (94.68%)

13.0 hits per line