Skip to content

test: read the site endpoint contract from SiteEndpoints.cs after the Program.cs split - #116

Merged
lukyrys merged 1 commit into
mainfrom
fix/contract-specs-endpoint-files
Oct 27, 2026
Merged

lukyrys merged 1 commit into
mainfrom
fix/contract-specs-endpoint-files

Conversation

@blackenzee

@blackenzee blackenzee commented Oct 26, 2026 •

Copy link
Copy Markdown

Summary

Stacked on #98. warnings-wrapper-contract.spec.ts checks the { site, warnings, hints } wrapper of POST /api/sites and PUT /api/sites/{domain} at the source level. It still read Program.cs, but #94/#98 moved those handlers to Sites/SiteEndpoints.cs, so both cases failed with no behaviour change. The spec now reads the new file. The regexes are unchanged and still match.

Verification

cd src/frontend && npx playwright test tests/playwright/tests/warnings-wrapper-contract.spec.ts --reporter=line   # 8 passed

@coderabbitai

coderabbitai Bot commented Oct 26, 2026 •

Copy link
Copy Markdown

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 57 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: fc4ccc0d-17dd-41f6-9c78-5fa4cdbbfa52
Commits

Reviewing files that changed from the base of the PR and between 8a25209 and 18eee27.

Files selected for processing: 1
  • src/frontend/tests/playwright/tests/warnings-wrapper-contract.spec.ts
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@lukyrys
lukyrys changed the base branch from refactor/program-endpoints-2 to main October 27, 2026 15:01
@lukyrys
lukyrys merged commit 99e151b into main Oct 27, 2026
9 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