Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Write additional unit/functional tests for MultiIndex #1408

Open
ctb opened this issue Mar 26, 2021 · 0 comments
Open

Write additional unit/functional tests for MultiIndex #1408

ctb opened this issue Mar 26, 2021 · 0 comments
Labels
good next issue An issue that should be ready to resolve. repeatable quest Issues with many, many subtasks; good for people new to codebase.

Comments

@ctb
Copy link
Contributor

ctb commented Mar 26, 2021 •
edited
Loading

#1374 introduced a MultiIndex class that wraps multiple Index classes. It's mostly being used for wrapping LinearIndex classes, but should work for more classes. Right now it's mostly tested via the command line tests in test_sourmash.py, but it is a nice consolidation of functionality and could usefully be tested in more detail via the Python API. Make it so!

(this is a potential good first issue or repeatable quest, just need to write down some suggested tests to implement!)
@ctb ctb mentioned this issue Mar 27, 2021
Merged
12 tasks
@bluegenes bluegenes added good next issue An issue that should be ready to resolve. repeatable quest Issues with many, many subtasks; good for people new to codebase. labels Jun 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
good next issue An issue that should be ready to resolve. repeatable quest Issues with many, many subtasks; good for people new to codebase.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ctb @bluegenes