Add Beta regression coverage for transaction writes - #6535
Conversation
|
Here's a visual recap of what changed: Open the full interactive recap |
There was a problem hiding this comment.
Builder reviewed your changes — looks good ✅
Review Details
Incremental Review Summary
The latest commit updates the two Beta regression specs without changing application code. In the Design test, it now supplies a UUID at creation and asserts the returned top-level ID, and teardown aggregates cleanup failures with the primary test failure. In the Analytics test, deletion is checked via a 404 readback and a listing that includes archived and hidden dashboards. These changes address all three concerns from the prior review. Both independent review passes found no new actionable issues. Risk remains low because this is test-only coverage; the action/readback assertions and cleanup checks are appropriate for the reported transaction paths.
🧪 Browser testing: Skipped — PR only modifies Beta E2E test files, with no user-facing UI impact.

Summary
Verification
pnpm guardspassed all 82 checks.Test fixture limit
The Beta E2E account cannot create a pinned Google Slides native item: it has no Google Drive grant or Creative Context Lab setup. The Slides clone call path is covered by Core scope and Slides action tests, but needs a dedicated Drive-backed beta fixture for a live clone check.