curationexperts / cypripedium / 31b67b34-c2ab-4e04-ac75-e6c8fe0d25af

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

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

27 existing lines in 11 files now uncovered.

1161 of 1229 relevant lines covered (94.47%)

15.11 hits per line