-
Notifications
You must be signed in to change notification settings - Fork 26
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 integration testing support that uses the sample repositories #197
Comments
I'm working on a |
There is a command line tool called |
Hi @sjones-gh, I think that it is fine if you want to use |
@gkapfham the original plan was to call it using a shell script, but if I find that it has a tangled mess of dependencies, I’ll see if there’s some python lib for this. I would like to avoid leaning on yet another python library, though. |
I'm sure that there are multiple Python libraries that would solve this issue. You would need to add it as a dev dependency in pipenv. |
Issue #192 explains that we want to create new repositories that contain examples that are organized in a new fashion and that adhere to a new naming convention.
Once all of these repositories exist, then it is possible to implement a new approach to integration testing. The idea would be that we can use a tool to clone all of the example repositories when we are testing new features of GatorGrader that are currently in a PR.
We would know that a repository like this one:
https://github.com/GatorEducator/java-assignment-starter-100-01
is a starter repository and thus, by design, the checks for this repo should fail, producing a non-zero exit code.
Alternatively, if we were running
gradle grade
in this repository:https://github.com/GatorEducator/java-assignment-solution-100-01
then since it is a solution repository, all of the checks should pass, producing a zero exit code.
What I would like to have is a new approach to integration testing that makes it possible to run these checks automatically for all of the existing sample repositories that follow the new naming convention.
The text was updated successfully, but these errors were encountered: