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

Create unit tests for each backlog item #8

Closed
TheSirLancelot opened this issue Oct 23, 2023 · 5 comments · Fixed by #15
Closed

Create unit tests for each backlog item #8

TheSirLancelot opened this issue Oct 23, 2023 · 5 comments · Fixed by #15
Assignees
Labels
documentation Improvements or additions to documentation epic Multiple PRs needed to complete

Comments

@TheSirLancelot
Copy link
Owner

TheSirLancelot commented Oct 23, 2023

DUE ON 7NOV23
Changed to Final Submission milestone

For each feature we are going to add, we need to create unit tests. Those unit tests need to be put into the test plan for submission/grade.

Edit: After speaking to the professor, this isn't due at this time. We'll speak to it for our test plan, but the unit tests don't need to be done yet.

@TheSirLancelot
Copy link
Owner Author

I went ahead and converted this to an issue because we know we will need to get this done. We should start working on this before we start writing the code for any of the features (for true test-driven-development).

I recommend treating this like an epic. E.g. this will not be complete until it's done for each feature. It will be assigned to everyone because we will all be working it. For each PR, we should link that PR to this issue so we can track which features have been completed easily.

@TheSirLancelot TheSirLancelot added the epic Multiple PRs needed to complete label Oct 23, 2023
@TheSirLancelot TheSirLancelot pinned this issue Oct 23, 2023
@TheSirLancelot TheSirLancelot added the documentation Improvements or additions to documentation label Oct 23, 2023
@TheSirLancelot TheSirLancelot added this to the Test Plan milestone Oct 23, 2023
@TheSirLancelot
Copy link
Owner Author

TheSirLancelot commented Oct 24, 2023

I think everyone is using VS Code from what I've heard. I think I found a good way of testing everything using pytest-playwright, and there's a good VS code extension that plays well:

Edit: nvm, that was using javascript

@TheSirLancelot
Copy link
Owner Author

Reopening issue. Github hasn't figure out how to let us link an issue without it getting auto-closed with the PR is merged...

@TheSirLancelot
Copy link
Owner Author

Issues with tests written for them (for Phase 1):

#16 #19

Issues still requiring tests:
#21 #18 #22 #5

@TheSirLancelot
Copy link
Owner Author

All tests have been added!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation epic Multiple PRs needed to complete
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants