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.

Sign up for GitHub

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

Jump to bottom

Adding tests to Tin #41

Open
1 of 11 tasks
JasonGrace2282 opened this issue Jun 3, 2024 · 1 comment
Open
1 of 11 tasks

Adding tests to Tin #41

JasonGrace2282 opened this issue Jun 3, 2024 · 1 comment
Assignees
@JasonGrace2282
Labels
enhancement New feature or request

Comments

@JasonGrace2282
Copy link
Member

JasonGrace2282 commented Jun 3, 2024 •
edited
Loading

After #28 got merged, it's now possible to add backend tests to Tin.

Backend Apps Remaining

These are the apps that need to be covered by backend logic tests

Frontend Apps Remaining

Additionally, it's important to make sure buttons and stuff still appear where they should.
These are the apps that need to be covered by frontend tests.

  • assignments
  • courses
  • docs
  • submissions
  • venvs
@JasonGrace2282 JasonGrace2282 mentioned this issue Jun 3, 2024
Merged
7 tasks
@krishnans2006 krishnans2006 added enhancement New feature or request area/backend labels Jun 3, 2024
@JasonGrace2282 JasonGrace2282 pinned this issue Jun 17, 2024
@JasonGrace2282 JasonGrace2282 mentioned this issue Jun 17, 2024
Closed
7 tasks
krishnans2006 added a commit that referenced this issue Jun 29, 2024
@JasonGrace2282 @krishnans2006
Add tests for the assignments app (#40)
3ded115 
Adds basic tests for every view in `tin/apps/assignments/views.py` (see
#41)

- [X] Basic assignment stuff
- [X] Graders
- [x] Files
- [x] Submissions
- [x] Quizzes
- [x] Scores
- [x] Folders

Co-authored-by: Krishnan Shankar <[email protected]>
@JasonGrace2282
Copy link
Member Author

A framework for testing UI has not yet been satisfactorily implemented. The last approach involved parsing html elements into a flattened list, which was just not efficient. An HTML testing framework should ideally allow us to just has has_button().with_text().with_url() instead of looking for a <a href="..."></a>.

@JasonGrace2282 JasonGrace2282 mentioned this issue Aug 28, 2024
Open
@JasonGrace2282 JasonGrace2282 changed the title Adding tests to tin Adding tests to Tin Sep 5, 2024
@JasonGrace2282 JasonGrace2282 self-assigned this Sep 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@JasonGrace2282 JasonGrace2282

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@krishnans2006 @JasonGrace2282