[ ] Do these changes modify the system output in any way? [Yes or No]
Closes #7
No, additional functions.
Pull Request Testing
[ ] Describe testing already performed for these changes:
Run python/coverage.sh -- requires a local dynamodb running for most tests and a 'junk' profile in your AWS credentials file. This can be automated in the CodeBuild testing later.
[ ] Recommend testing for the reviewer(s) to perform, including the location of input datasets, and any additional instructions:
Setup for testing could be non-trivial. All of the tests have passed in my environment. We should prioritize setting up CodeBuild once we get our AWS account.
[ ] Do these changes include sufficient documentation updates, ensuring that no errors or warnings exist in the build of the documentation? [Yes or No]
[ ] Do these changes include sufficient testing updates? [Yes or No]
Yes
[ ] Will this PR result in changes to the test suite? [Yes or No]
If yes, describe the new output and/or changes to the existing output:
Yes. New unit tests have been added.
[ ] Please complete this pull request review by [Fill in date].
Pull Request Checklist
[ ] Review the source issue metadata (labels, project, and milestone).
[ ] Complete the PR definition above.
[ ] Ensure the PR title matches the feature or bugfix branch name.
[ ] Define the PR metadata, as permissions allow.
Select: Reviewer(s)
Select: Project
Select: Milestone as the version that will include these changes
Select: Development to link to the original development issue.
[ ] After the PR is approved, merge your changes. If permissions do not allow this, request that the reviewer do the merge.
[ ] Close the linked issue and delete your feature or bugfix branch from GitHub.
Closes #7
Expected Differences
No, additional functions.
Pull Request Testing
[ ] Describe testing already performed for these changes: Run python/coverage.sh -- requires a local dynamodb running for most tests and a 'junk' profile in your AWS credentials file. This can be automated in the CodeBuild testing later.
[ ] Recommend testing for the reviewer(s) to perform, including the location of input datasets, and any additional instructions: Setup for testing could be non-trivial. All of the tests have passed in my environment. We should prioritize setting up CodeBuild once we get our AWS account.
[ ] Do these changes include sufficient documentation updates, ensuring that no errors or warnings exist in the build of the documentation? [Yes or No]
[ ] Do these changes include sufficient testing updates? [Yes or No] Yes
[ ] Will this PR result in changes to the test suite? [Yes or No] If yes, describe the new output and/or changes to the existing output: Yes. New unit tests have been added.
[ ] Please complete this pull request review by [Fill in date].
Pull Request Checklist