biocore / kl-metapool / 17308384169

93%

Build Type

push

github

Committed by web-flow

Commit Message

MetaT replicate removal (#325)

* Validate sample sheets on load (#319)

* WIP: validate sample sheets on load (and fix test files that are invalid :O )

* WIP: validate sample sheets on load (and fix test files that are invalid :O )

* Correct test sample sheets

* Update versions to match corrected sample sheets

* string literal replacements and small helper functions

* fix bug failing to lcase col names compared to sheet_to_dataframe output

* refactor demuxing not to change col names

* add KLSampleSheetWithReplicates, refactor katharoseq and replicates to use new optional_col_sets to determine required cols

* modularize, extend validation

* new test sample sheet file

* lint fixes, correct _normalize_df_sections_booleans location

* make defer_validate=False the default to preserve past behavior, make tests validate on load, add deprecation warning

* catch dup columns caused by remapping, update tests

* lint

* Address @antgonza code review comments, fix #233

* Rename test projects

* Make validate on load the default; retain DeprecationWarning code commented out for now

* Add tests for validate on load settings

* Replicate fix 2 (#320)

* rename tests for clarity, move mp_string tests into appropriate test file

* rename columns so orig_name exists and optional columns are clearly indicated

* clarify how to input source plate info when using plate replication

* clarify error if contains_replicates not in Bioinformatics section

* break out katharoseq no-state-change test as separate function

* Refactor PlateReplication to make internal stuff private, rename for clarity, add comments, add `unmake_replicates`. Refactor prep and sample_sheet to use `unmake_replicates` instead of duplicating code

* lint

* lint

* change default value for katharoseq_dir

* add shared interface between PlateReplication and PlateRemapper

* add PlateReplication get_384_well_location and unmake_replicates tests, do cleanup

* test ErrorMessage strings not ... (continued)

Run Details

946 of 979 new or added lines in 13 files covered. (96.63%)

29 existing lines in 3 files now uncovered.

6479 of 7024 relevant lines covered (92.24%)

0.92 hits per line