Skip to content

chore(test): use RequestCatcher webhook URLs in integration tests#80

Closed
FranDisalvo90 wants to merge 1 commit into
masterfrom
chore/fix-integration-webhook-urls
Closed

chore(test): use RequestCatcher webhook URLs in integration tests#80
FranDisalvo90 wants to merge 1 commit into
masterfrom
chore/fix-integration-webhook-urls

Conversation

@FranDisalvo90

Copy link
Copy Markdown
Collaborator

Problem

The daily scheduled Build & Test run has been failing since ~2026-06-01. ~10 integration test classes fail in setUp with:

java.lang.Error: Create item request responded with error, code=400,
message='Webhook url is a reserved placeholder and cannot be used'

Root cause

This is an API-side change, not a repo regression: the last green scheduled run (May 31) and the first red one (Jun 1) ran the same master with no code change in between (runtime dropped from ~6min to ~1.5min because the tests now abort early in setUp).

Around Jun 1 the API started rejecting reserved/placeholder webhook URLs. The tests hardcoded such URLs:

  • helper/ItemHelper.javahttps://webhookUrl.pluggy.ai (used by createPluggyBankItem, so it broke the setUp of almost every item-dependent test)
  • CreateItemTest / UpdateItemTesthttps://www.test.com / https://www.test2.com (no longer accepted either)

Tests that don't create items (connectors, categories, auth, connect token) keep passing.

Fix

Replace the hardcoded placeholders with public, reachable HTTPS RequestCatcher endpoints, as recommended by the Pluggy webhook docs for testing. Distinct paths are used where the assertions compare URLs (UpdateItemTest). The negative http://www.test.com case (which tests rejection of non-HTTPS) is left untouched.

Validation

  • mvn test-compile (JDK 11)
  • ⏳ Integration tests run against the live API with Doppler credentials in CI — this PR's Build & Test run will confirm the fix.

🤖 Generated with Claude Code

The API started rejecting reserved placeholder webhook URLs (e.g.
*.pluggy.ai) with HTTP 400 "Webhook url is a reserved placeholder and
cannot be used" around 2026-06-01, which broke item creation in the
setUp of most integration tests (and the www.test.com URLs were no
longer accepted either). Replace the hardcoded placeholders with
public, reachable HTTPS RequestCatcher endpoints, as recommended by the
Pluggy webhook docs for testing.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@FranDisalvo90

Copy link
Copy Markdown
Collaborator Author

Consolidado en #79: el commit del fix de los integration tests (RequestCatcher webhook URLs) se movió a la branch feat/account-reserved-balance. Cierro este PR en favor de #79.

@FranDisalvo90 FranDisalvo90 deleted the chore/fix-integration-webhook-urls branch June 9, 2026 18:55
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