Skip to content

[Bugfix:Autograding] revise system call category name#12076

Merged
bmcutler merged 1 commit intomainfrom
shorten_category_name
Sep 22, 2025
Merged

[Bugfix:Autograding] revise system call category name#12076
bmcutler merged 1 commit intomainfrom
shorten_category_name

Conversation

@bmcutler
Copy link
Copy Markdown
Member

Why is this Change Important & Necessary?

We recently changed the system call categories and did not test and update all provided examples

What is the New Behavior?

removed 'ALLOW_SYSTEM_CALL_CATEGORY_' from the category names in a few sample autograding configs.

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

These autograding examples should now work when tested manually :)

Automated Testing & Documentation

We are working on adding automated tests of most/every provided example, but these aren't tested yet.

Other information

n/a

@github-project-automation github-project-automation bot moved this to Seeking Reviewer in Submitty Development Sep 22, 2025
@bmcutler bmcutler changed the title [Bugfix:Autograding] revise system call category name in examples [Bugfix:Autograding] revise system call category name Sep 22, 2025
@codecov
Copy link
Copy Markdown

codecov bot commented Sep 22, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 21.69%. Comparing base (7fbcef5) to head (ac72784).
⚠️ Report is 55 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##               main   #12076      +/-   ##
============================================
+ Coverage     21.66%   21.69%   +0.03%     
- Complexity     9528     9587      +59     
============================================
  Files           268      268              
  Lines         36395    36593     +198     
  Branches        475      475              
============================================
+ Hits           7886     7940      +54     
- Misses        28038    28182     +144     
  Partials        471      471              
Flag Coverage Δ
autograder 21.39% <ø> (+0.08%) ⬆️
js 2.07% <ø> (ø)
migrator 100.00% <ø> (ø)
php 20.71% <ø> (+0.03%) ⬆️
python_submitty_utils 80.08% <ø> (ø)
submitty_daemon_jobs 90.72% <ø> (ø)

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.

bmcutler added a commit to Submitty/Tutorial that referenced this pull request Sep 22, 2025
SImilar to Submitty/Submitty#12076

---------

Co-authored-by: Barb Cutler <Barb Cutler>
@bmcutler bmcutler merged commit 88e39f8 into main Sep 22, 2025
25 of 28 checks passed
@bmcutler bmcutler deleted the shorten_category_name branch September 22, 2025 17:34
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.

1 participant