Skip to content

Update Artifact Lambda Unit Tests & Refactor for ruff#6

Merged
Alrobbertz merged 2 commits into
swxsoc:mainfrom
Alrobbertz:refactor-artifacts
Jun 16, 2026
Merged

Update Artifact Lambda Unit Tests & Refactor for ruff#6
Alrobbertz merged 2 commits into
swxsoc:mainfrom
Alrobbertz:refactor-artifacts

Conversation

@Alrobbertz

Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings June 12, 2026 20:00

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the repository’s developer tooling and CI workflows to use Ruff, refreshes unit test mocking to align with newer Moto patterns, and migrates repository documentation from README.md to README.rst.

Changes:

  • Update dev dependencies (pytest/moto) and replace Black/Flake8 with Ruff.
  • Refactor unit tests to use Moto’s unified AWS mocking and introduce a pytest fixture.
  • Modernize GitHub Actions workflows for testing, coverage upload, and Ruff-based lint/format.

Reviewed changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated 9 comments.

Show a summary per file
File Description
requirements.dev.txt Updates test/dev dependencies and switches linting/formatting tooling to Ruff.
README.rst Adds a new reStructuredText README with updated test/run instructions and badges.
README.md Removes the previous Markdown README in favor of README.rst.
lambda_function/tests/test_process_artifacts.py Updates Moto usage and refactors test setup to use a pytest fixture.
lambda_function/src/process_artifacts/process_artifacts.py Ruff-oriented refactor: import ordering, typing, and updated docstrings.
lambda_function/src/lambda.py Adds typing and refreshes handler documentation.
lambda_function/src/config.yaml Removes the previously committed config file.
.github/workflows/testing.yml Updates CI environment/actions and adds Codecov upload step.
.github/workflows/codestyle.yml Replaces Black/Flake8 workflow with Ruff-based lint/format.
.github/workflows/build_and_push.yml Removes the prior self-hosted Docker build/push workflow.
Comments suppressed due to low confidence (1)

requirements.dev.txt:8

  • All other dev tools here are version-pinned, but ruff is left unpinned. This can make CI results non-reproducible when Ruff releases new rules/formatting. Consider pinning Ruff to a known-good version.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/testing.yml
Comment thread .github/workflows/testing.yml
Comment thread .github/workflows/codestyle.yml
Comment thread lambda_function/tests/test_process_artifacts.py Outdated
Comment thread lambda_function/src/process_artifacts/process_artifacts.py
Comment thread README.rst
Comment thread README.rst
Comment thread README.rst
Comment thread lambda_function/tests/test_process_artifacts.py Outdated
@codecov

codecov Bot commented Jun 16, 2026

Copy link
Copy Markdown

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

@Alrobbertz Alrobbertz merged commit b05408c into swxsoc:main Jun 16, 2026
4 of 5 checks passed
@Alrobbertz Alrobbertz deleted the refactor-artifacts branch June 16, 2026 13:00
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