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

Improve test coverage #48

Open
niconoe opened this issue Jan 23, 2020 · 2 comments
Open

Improve test coverage #48

niconoe opened this issue Jan 23, 2020 · 2 comments
Assignees
@niconoe
Labels
🛀 cleanup Code cleanup enhancement New feature or request

Comments

@niconoe
Copy link
Collaborator

niconoe commented Jan 23, 2020

No description provided.

@niconoe niconoe added enhancement New feature or request 🛀 cleanup Code cleanup labels Jan 23, 2020
@niconoe niconoe self-assigned this Jan 23, 2020
niconoe pushed a commit that referenced this issue Jan 31, 2020
Test data to compare profile integration code to BioRAD's output (see #…
eb48cce 
…48,  #45)
@niconoe
Copy link
Collaborator Author

niconoe commented Feb 5, 2020 •
edited
Loading

UPDATE: basic tests for the TimelineChart component. Probable next steps:

  • Reimplement VPChart(done) and VPIChart to make them testable (see Use Vue for all DOM manipulations #62)
  • Write/improve a basic test suite for all 3 components + the crow Form/app itself
  • Implement tests to compare our integration output's to BioRAD

@niconoe niconoe changed the title Implement unit tests Improve test coverage Feb 2, 2021
@niconoe
Copy link
Collaborator Author

niconoe commented Feb 2, 2021

Update: there should ideally bemore tests (especially if the code is reused for other projects such as GloBAM), but at least the current test suite pass again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@niconoe niconoe

Labels
🛀 cleanup Code cleanup enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@niconoe