curationexperts / cypripedium / 3aaed53d-544e-4371-88af-219470313040

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%)

12 existing lines in 6 files now uncovered.

1246 of 1316 relevant lines covered (94.68%)

12.98 hits per line