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

Develope skip and only functions #11

Merged
merged 10 commits into from
Jun 1, 2022
Merged

Develope skip and only functions #11

merged 10 commits into from
Jun 1, 2022

Conversation

natlibfi-jonollil
Copy link
Contributor

  • Metadata.json can contain skip and only parametters
    • Skip parametter makes test to be skipped
    • Only parametter makes only those test to be run
    • Added tests for both
  • Smarter test automatic testing
    • If folder is positive integer digits it is included in test name
    • Added test to naming
  • License to MIT
  • Drone swap to github actions

@natlibfi-koskimim
Copy link

Looks great!

README.md Outdated
- **useMetadataFile**: If set to true, reads and parses a file named `metadata.json` in each unit test directory. The content must be a JSON object and it's properties as passed to the unit test callback function.
- If a property `description` is present, it is used as the test's description (Defaults to *false* and folder name is used).
- If a property `skip` is present, it decides if test is skipped (Defaults to *false*).
- If a property `only` is present, it decides if this/theis tests are only run (Defaults to *false*).
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TYPO: this/these (instead of this/_theis_)

@natlibfi-jonollil natlibfi-jonollil merged commit 3b65e2c into master Jun 1, 2022
@natlibfi-jonollil natlibfi-jonollil deleted the dev-skip-only branch June 1, 2022 05:06
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.

3 participants