Skip to content

fix(ci): fix CI workflow, backend tests, and security assertions - #7

Merged
ANUJ760 merged 4 commits into
ANUJ760:mainfrom
Adarsh2709:fix/ci-pipeline
Sep 28, 2026
Merged

ANUJ760 merged 4 commits into
ANUJ760:mainfrom
Adarsh2709:fix/ci-pipeline

Conversation

@Adarsh2709

Copy link
Copy Markdown
Collaborator

Summary of Changes

This PR includes fixes for CI pipeline execution, backend test discovery, module pollution prevention, and security route checks:

  1. CI/CD Workflow:

    • Fixed the pip cache dependency path in .github/workflows/ci.yml.
    • Ensured PYTHONPATH=. is set during test execution so backend module imports resolve cleanly.
  2. Backend Module Resolution:

    • Added package __init__.py files across backend directories to ensure standardized package discovery.
  3. Test Stability & Isolation:

    • Resolved global boto3 client mocking pollution in tests so test cases don't interfere with each other.
    • Updated end-to-end assertions to match current status outputs.
  4. Security & Access Control:

    • Enforced require_editor permission checks on /deploy and /revert endpoints.
    • Updated E2E test assertions to reflect role-based access enforcement.
  5. Repository Cleanliness:

    • Added build_lambda/ and *.zip to .gitignore to avoid checking in build and packaging artifacts.

@ANUJ760
ANUJ760 merged commit 22880ea into ANUJ760:main Sep 28, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants