Skip to content

Align deprecation regression test with current Bitbucket schema - #130

Open
FabianSchurig with Copilot wants to merge 1 commit into
mainfrom
copilot/fix-daily-run-issue
Open

Align deprecation regression test with current Bitbucket schema#130
FabianSchurig with Copilot wants to merge 1 commit into
mainfrom
copilot/fix-daily-run-issue

Conversation

Copilot AI commented Aug 23, 2026

Copy link
Copy Markdown
Contributor

The daily schema sync failed because Bitbucket removed /user/permissions/workspaces. Regenerated commands now expose the deprecated /workspaces endpoint as list-workspaces-for-user.

  • Test alignment
    • Update TestDeprecatedCommandsAreMarked to target the current deprecated command.
    • Preserve coverage that generated deprecated endpoints are marked in Cobra.
cmd.Find([]string{"workspaces", "list-workspaces-for-user"})

Co-authored-by: FabianSchurig <11216482+FabianSchurig@users.noreply.github.com>
Copilot AI changed the title Fix daily schema sync deprecation test Align deprecation regression test with current Bitbucket schema Aug 23, 2026
Copilot AI requested a review from FabianSchurig August 23, 2026 06:07
@FabianSchurig
FabianSchurig marked this pull request as ready for review August 23, 2026 06:19
Copilot AI lite review requested due to automatic review settings August 23, 2026 06:19

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

The change is a minimal, verified test-path update that matches the currently generated deprecated command and preserves the existing assertion.

Pull request overview

Updates a CLI regression test to match the currently generated deprecated Bitbucket endpoint after upstream schema drift removed /user/permissions/workspaces. This keeps the “deprecated commands are marked” coverage aligned with the auto-generated Cobra command tree.

Changes:

  • Update TestDeprecatedCommandsAreMarked to look up workspaces list-workspaces-for-user instead of the removed command path.
File summaries
File Description
cmd/bb-cli/main_test.go Adjusts deprecated-command lookup to the current generated command name so the deprecation marker assertion remains valid.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

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.

3 participants