Skip to content

Add JSON APIs for submission details and use them in SolutionChecker.py - #1712

Merged
veluca93 merged 5 commits into
cms-dev:mainfrom
veluca93:api-submission-details
Sep 10, 2026
Merged

veluca93 merged 5 commits into
cms-dev:mainfrom
veluca93:api-submission-details

Conversation

@veluca93

Copy link
Copy Markdown
Contributor

No description provided.

@veluca93
veluca93 requested review from gollux and prandla August 22, 2026 16:04
@codecov

codecov Bot commented Aug 22, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 87.42058% with 99 lines in your changes missing coverage. Please review.
✅ Project coverage is 56.65%. Comparing base (6ca9c9f) to head (28936c8).
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
cmscontrib/SolutionChecker.py 74.16% 77 Missing ⚠️
cms/server/contest/handlers/api.py 80.64% 12 Missing ⚠️
cms/grading/scoretypes/abc.py 91.07% 5 Missing ⚠️
cms/grading/scoretypes/Sum.py 85.00% 3 Missing ⚠️
...suite/unit_tests/cmscontrib/SolutionCheckerTest.py 99.43% 1 Missing ⚠️
cmstestsuite/unit_tests/server/contest/api_test.py 99.07% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1712      +/-   ##
==========================================
+ Coverage   55.47%   56.65%   +1.17%     
==========================================
  Files         342      344       +2     
  Lines       27551    28234     +683     
==========================================
+ Hits        15285    15997     +712     
+ Misses      12266    12237      -29     
Flag Coverage Δ
functionaltests 0.00% <0.00%> (ø)
unittests 56.65% <87.42%> (+1.17%) ⬆️

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

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@veluca93
veluca93 requested a review from Virv12 August 22, 2026 20:48
Comment thread cmscontrib/SolutionChecker.py Outdated
Comment thread cmscontrib/SolutionChecker.py Outdated
Comment thread cmscontrib/SolutionChecker.py Outdated
Comment thread docs/API.rst Outdated
Comment thread cms/server/contest/handlers/api.py
@veluca93
veluca93 force-pushed the api-submission-details branch from bc08ea8 to e82cec0 Compare September 7, 2026 08:26
@veluca93
veluca93 force-pushed the api-submission-details branch from e82cec0 to 54cc975 Compare September 10, 2026 18:12
@veluca93
veluca93 force-pushed the api-submission-details branch from 54cc975 to 28936c8 Compare September 10, 2026 18:12
@veluca93
veluca93 merged commit b411a0f into cms-dev:main Sep 10, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants