Skip to content

Conversation

@MintsInc
Copy link
Member

Context

BDD test undo operations previously could only extract parameters from response bodies or request bodies. This prevented proper cleanup of resources where identifiers exist only in the request URL path (e.g., POST /integrations/{integration_name}/accounts).

This adds support for origin: "path" in undo.json definitions, allowing undo operations to reference path parameters from the original request.

Changes

  • Added @path_parameters hash to store path parameters
  • Modified undo lambda to handle origin: "path"
  • Store path parameters during request setup in step definitions

Tests

Tested with Fastly integration endpoint that requires path parameter for cleanup:

  • Created Fastly service with path parameter account_id
  • Verified undo operation correctly captured and used path parameter
  • Confirmed backward compatibility with existing undo operations

Support extracting path parameters from request URLs for use in undo
operations. This enables cleanup of resources identified by path
parameters (e.g., /api/v2/resource/{id}).

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@MintsInc MintsInc marked this pull request as ready for review January 21, 2026 12:24
@MintsInc MintsInc requested review from a team as code owners January 21, 2026 12:24
@MintsInc MintsInc merged commit 46bb176 into master Jan 23, 2026
19 of 20 checks passed
@MintsInc MintsInc deleted the ulysse.mavrocordatos/add-path-parameter-undo-support branch January 23, 2026 09:06
github-actions bot pushed a commit that referenced this pull request Jan 23, 2026
Support extracting path parameters from request URLs for use in undo
operations. This enables cleanup of resources identified by path
parameters (e.g., /api/v2/resource/{id}).

Co-authored-by: Claude Sonnet 4.5 <noreply@anthropic.com> 46bb176
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants