Skip to content

test: approve a patch in the live app test - #47

Merged
guillaumegay13 merged 1 commit into
feat/urllib-on-mainfrom
test/live-app-approves-patch
Sep 26, 2026
Merged

guillaumegay13 merged 1 commit into
feat/urllib-on-mainfrom
test/live-app-approves-patch

Conversation

@guillaumegay13

@guillaumegay13 guillaumegay13 commented Sep 26, 2026 •

Copy link
Copy Markdown
Contributor

✨ What changed

  • The live test opens the issue, then approves a clamp patch as an operator.
  • Operator credentials come from MNFST_TEST_OPERATOR_EMAIL / _PASSWORD, defaulting to the dev seed.
  • It now covers aiohttp and urllib too: 5 clients succeed, plus the failed and inconclusive cases.
  • Each run uses a letters-only path segment, so it gets its own issue.

💭 Why

The app returns no_patch until an operator approves a patch, so test_live_app.py failed at its first call against any current app, whatever the SDK did.

📝 Notes

The app serves no patch until an operator approves one, so the live test
failed at its first call against any current app. It now opens the issue,
signs in as an operator and approves the clamp before the SDK calls, and
covers aiohttp and urllib alongside httpx and requests.
@guillaumegay13
guillaumegay13 merged commit 1d669a3 into feat/urllib-on-main Sep 26, 2026
3 checks passed
@guillaumegay13
guillaumegay13 deleted the test/live-app-approves-patch branch September 26, 2026 21:31
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.

1 participant