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.

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

Add testing setup #106

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Add testing setup #106

wants to merge 5 commits into from

Conversation

ronen
Copy link
Contributor

@ronen ronen commented Sep 13, 2016

This PR adds a pytest setup; it's what I've been using while doing the refactoring.

Quick notes:

  • Presumes you've installed pytest. Execute tests by running py.test
  • Not (yet?) a fleshed-out test suite. Just some simple acceptance/regression tests to verify that the transcribers seem to work and do what they did before on the given inputs. No reason to think that it's exercised all interesting cases. But of course more tests could be added (especially TDD-style for future work).
  • Test input data is in tests/data/inputs/... and result data for regression testing is in tests/data/results/...
  • The full transcription results are likely to be somewhat platform specific at least in floating-point number values, and possibly even in the exact phonemes found. So probably the regression tests in this PR will fail for anybody other than me! Which means the tests should be changed from requiring exact equality to verifying for some higher-level correctness.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant