Skip to content

fix: restore SecuScan CI baseline#635

Closed
saurabhhhcodes wants to merge 1 commit into
utksh1:mainfrom
saurabhhhcodes:fix/ci-baseline-tests
Closed

fix: restore SecuScan CI baseline#635
saurabhhhcodes wants to merge 1 commit into
utksh1:mainfrom
saurabhhhcodes:fix/ci-baseline-tests

Conversation

@saurabhhhcodes
Copy link
Copy Markdown
Contributor

Summary

  • fix the workflow scheduler lint failure by acquiring the database handle before resolving target policy
  • update stale execution-context expectations in ToolConfig and Workflows tests
  • align the timeout test with the current validated max value emitted by ToolConfig

Root Cause

The current CI baseline was failing on main-based PRs because:

  • backend/secuscan/workflows.py referenced db inside _run_workflow() without defining it
  • frontend tests still expected the pre-execution-context payload shape
  • the timeout test expected the older server-limit clamp value

Validation

  • python3 -m ruff check backend testing/backend
  • npm test -- --run testing/unit/pages/ToolConfigDynamic.test.tsx testing/unit/pages/ToolConfigTimeout.test.tsx testing/unit/pages/Workflows.test.tsx
  • npm test
  • git diff --check

Impact

Restores the CI baseline for unrelated frontend/backend PRs, including the hook coverage PR that exposed these existing failures.

@utksh1 utksh1 added level:beginner 20 pts difficulty label for small beginner-friendly PRs type:bug Bug fix work category bonus label type:testing Testing work category bonus label area:backend Backend API, database, or service work area:frontend Frontend React/UI work gssoc:invalid Admin validation: invalid for GSSoC scoring labels Jun 7, 2026
@utksh1
Copy link
Copy Markdown
Owner

utksh1 commented Jun 7, 2026

Closing this as a lower-scope duplicate of #629. #629 covers the same CI baseline repair with the missing API mocks and typed resource-list fixtures, so keeping both open would split the same fix across duplicate PRs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:backend Backend API, database, or service work area:frontend Frontend React/UI work gssoc:invalid Admin validation: invalid for GSSoC scoring level:beginner 20 pts difficulty label for small beginner-friendly PRs type:bug Bug fix work category bonus label type:testing Testing work category bonus label

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants