Skip to content

feat: add Azure DevOps dependency dashboard support#42758

Open
RahulGautamSingh wants to merge 9 commits intorenovatebot:mainfrom
RahulGautamSingh:feat/azure-dep-dashboards
Open

feat: add Azure DevOps dependency dashboard support#42758
RahulGautamSingh wants to merge 9 commits intorenovatebot:mainfrom
RahulGautamSingh:feat/azure-dep-dashboards

Conversation

@RahulGautamSingh
Copy link
Copy Markdown
Collaborator

@RahulGautamSingh RahulGautamSingh commented Apr 20, 2026

Summary

Reviving #38025 (originally by @VNovotna).
Adds dependency dashboard support for Azure DevOps via WorkItems, plus addresses the remaining review feedback that hadn't been applied:

  • Cache IssueService instance, recreate on initRepo (per @viceice's suggestion)
  • Use partial<IWorkItemTrackingApi> instead of as any in tests
  • Drop the per-test vi.resetModules() pattern in issue.spec.ts and the redundant integration block

All credit for the original feature goes to @VNovotna; their commits are preserved.

Context

  • This closes an existing Issue, Closes: Support issues for Azure platform #9592
  • This doesn't close an Issue, but I accept the risk that this PR may be closed if maintainers disagree with its opening or implementation

AI assistance disclosure

  • Yes — substantive assistance: Claude was used to identify the unaddressed review comments on the original PR and apply the three fixes above. The original feature implementation by @VNovotna was not modified beyond those targeted fixes.

Documentation

  • No documentation update is required

How I've tested my work

  • Newly added/modified unit tests

VNovotna and others added 8 commits September 15, 2025 14:30
Resolved conflicts in:
- lib/modules/platform/azure/azure-got-wrapper.ts
- lib/modules/platform/azure/index.ts
- lib/modules/platform/azure/types.ts
- lib/modules/platform/utils/read-only-issue-body.ts
Add test cases to ensure proper handling when:
- Trying to reopen an issue without a number
- Trying to update an issue without a number

This improves code coverage for the null checks added in previous commit.
there was mismatch in mock resulting in testing of different code branch
@github-actions github-actions bot requested a review from viceice April 20, 2026 06:48
@RahulGautamSingh RahulGautamSingh added the ci:allow-undesirable Skip the undesirable files CI check label Apr 20, 2026
Copy link
Copy Markdown
Member

@viceice viceice left a comment

Choose a reason for hiding this comment

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

otherwise LGTM

() =>
({
queryByWiql: vi.fn().mockResolvedValue({ workItems: [] }),
}) as any,
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

use partial helper instead of any

({
queryByWiql: vi.fn().mockResolvedValue({ workItems: [] }),
createWorkItem: createWorkItemMock,
}) as any,
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

same

]),
updateWorkItem: updateWorkItemMock,
createWorkItem: createWorkItemMock,
}) as any,
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

same

const azureApiWit = await azureApi.workItemTrackingApi();

let wiql = `
SELECT [System.Id], [System.Title], [System.State], [System.CreatedDate]
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

does it also support an updated date? so we can use local caching?

Comment on lines +64 to +66
wi.fields!['System.State'] === 'New' ||
wi.fields!['System.State'] === 'Active' ||
wi.fields!['System.State'] === 'To Do'
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

isn't it easier to check for closed?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci:allow-undesirable Skip the undesirable files CI check

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support issues for Azure platform

3 participants