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 monster schema #134

Merged
merged 2 commits into from
Jul 18, 2019
Merged

Add monster schema #134

merged 2 commits into from
Jul 18, 2019

Conversation

osrsbox
Copy link
Owner

@osrsbox osrsbox commented Jul 16, 2019

As discussed in #116 - this is the JSON schema for the monster database. This PR moves to add the authored and revised JSON schema into the master branch to allow others to more easily work towards adding in monster data into the project.

This PR also adds a pytest module named test/test_monster_database.py. The test can be run using: python -m pytest test/test_monster_database.py when in the root directory of the project. The test is disabled by default so that the project Travis build does not fail. To enable the test, temporarily remove or comment the following line: @pytest.mark.skip(reason="Skipped: The monster data is not currently populated.").

@osrsbox osrsbox added enhancement monsters Anything to do with the monsters data, API or builder tests Concerning any code testing labels Jul 16, 2019
@osrsbox osrsbox self-assigned this Jul 16, 2019
@osrsbox osrsbox merged commit 7aabb3f into master Jul 18, 2019
@osrsbox osrsbox mentioned this pull request Jul 18, 2019
@osrsbox osrsbox deleted the monster-schema branch July 19, 2019 21:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement monsters Anything to do with the monsters data, API or builder tests Concerning any code testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant