Skip to content

Add automated tests for prs#7

Merged
ShiosOS merged 7 commits intomainfrom
ci-updates
Mar 8, 2026
Merged

Add automated tests for prs#7
ShiosOS merged 7 commits intomainfrom
ci-updates

Conversation

@ShiosOS
Copy link
Owner

@ShiosOS ShiosOS commented Mar 8, 2026

Context

There is current no way to autorun tests on prs, this PR aims to add that.

This Change

  • Add yml for automated github tests on prs

@chatgpt-codex-connector
Copy link

💡 Codex Review

Assert.Equal(2, leftovers!.Count);

P1 Badge Reset database state between endpoint tests

These tests share a single TestApplicationFactory instance for the whole class, and that factory creates one SQLite file that is never cleared, so data written in one test leaks into the next. In this class, CreateLeftover_PersistsAndCanBeFetched can leave an extra row that makes GetAllLeftovers_ReturnsSortedByExpiry fail on Count == 2 depending on xUnit test order, creating flaky CI failures.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@ShiosOS ShiosOS changed the title Ci updates Add automated tests for prs Mar 8, 2026
@ShiosOS ShiosOS linked an issue Mar 8, 2026 that may be closed by this pull request
@ShiosOS ShiosOS merged commit a6a18bb into main Mar 8, 2026
7 checks passed
@ShiosOS ShiosOS deleted the ci-updates branch March 8, 2026 09:05
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.

Add CI/CD testing pipeline

1 participant