Skip to content

[Bugfix:Testing] Revert leaderboard Cypress test#12704

Merged
IDzyre merged 1 commit into
Submitty:mainfrom
GarvitKhandelwal31:revert-leaderboard-test
Mar 31, 2026
Merged

[Bugfix:Testing] Revert leaderboard Cypress test#12704
IDzyre merged 1 commit into
Submitty:mainfrom
GarvitKhandelwal31:revert-leaderboard-test

Conversation

@GarvitKhandelwal31

@GarvitKhandelwal31 GarvitKhandelwal31 commented Mar 31, 2026

Copy link
Copy Markdown
Contributor

Why is this Change Important & Necessary?

Related to #12683
This reverts the leaderboard Cypress test to its previous version because recent changes introduced persistent failures in CI. The revert restores stability to the test suite while the underlying issues are investigated.

What is the New Behavior?

The leaderboard end-to-end Cypress test is reverted to its last known stable state. No new features or UI changes are introduced.

What steps should a reviewer take to reproduce or test the bug or new feature?

  1. Run the Cypress test suite locally or in CI.
  2. Confirm that the leaderboard test now passes as it did before the tied-ranking changes.
image

Automated Testing & Documentation

  • This PR is a test-only change.
  • No new documentation is required.
  • No new or updated automated tests are included, as this is a revert to a previously passing test.

Other information

  • Not a breaking change.
  • No migrations or sysadmin actions required.
  • No security concerns.

@codecov

codecov Bot commented Mar 31, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 21.64%. Comparing base (02638f6) to head (aeeec48).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff            @@
##               main   #12704   +/-   ##
=========================================
  Coverage     21.64%   21.64%           
  Complexity     9654     9654           
=========================================
  Files           268      268           
  Lines         36245    36245           
  Branches        487      487           
=========================================
  Hits           7846     7846           
  Misses        27916    27916           
  Partials        483      483           
Flag Coverage Δ
autograder 21.32% <ø> (ø)
js 2.04% <ø> (ø)
migrator 100.00% <ø> (ø)
php 20.66% <ø> (ø)
python_submitty_utils 80.08% <ø> (ø)
submitty_daemon_jobs 91.13% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-project-automation github-project-automation Bot moved this from Seeking Reviewer to Awaiting Maintainer Review in Submitty Development Mar 31, 2026
@IDzyre IDzyre changed the title [Testing:Submission] Revert leaderboard Cypress [Bugfix:Testing] Revert leaderboard Cypress test Mar 31, 2026
@IDzyre
IDzyre merged commit d8acd34 into Submitty:main Mar 31, 2026
29 checks passed
@github-project-automation github-project-automation Bot moved this from Awaiting Maintainer Review to Done in Submitty Development Mar 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants