-
Notifications
You must be signed in to change notification settings - Fork 80
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
[MRG] provide "protocol" tests for
Index
, CollectionManifest
, and…
… `LCA_Database` classes (#1936) * add LCA database test for tricky ordering * add test for jaccard ordering to SBTs * add test_index_protocol * add tests of indices after save/load * match Index definition of __len__ in sbt * more index tests * add some generic manifest tests * define abstract base class for CollectionManifest * fix GTDB example, sigh * test hashval_to_idx * add actual test for min num in rankinfo * update 'get_lineage_assignments' in lca_db * update comment * make lid_to_idx and idx_to_ident private * moar comment * add, then hide, RevIndex test * update the LCA_Database protocol * finish testing the rest of the Index classes * cleanup * upd * backport 08ac110 * remove test for now-implemented func * switch away from a row tuple in CollectionManifest * more clearly separate internals of LCA_Database from public API * add saved/loaded manifest * add test coverage for exceptions in LazyLoadedIndex * add docstrings to manifest code * add docstrings / comments * use names in namedtuple; add containment test * add numerical values to jaccard order tests * add required_keys check * fix diagnostic output during sourmash index #1949 * fix test name
- Loading branch information
Showing
15 changed files
with
969 additions
and
284 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.