Skip to content

[Quality Management] Enhance error messaging for test value type changes and improve validation logic#8291

Open
JakovljevicDusan wants to merge 3 commits into
mainfrom
bugs/QM-MoreInformativeErrorMessageWhenChangingTestValueType
Open

[Quality Management] Enhance error messaging for test value type changes and improve validation logic#8291
JakovljevicDusan wants to merge 3 commits into
mainfrom
bugs/QM-MoreInformativeErrorMessageWhenChangingTestValueType

Conversation

@JakovljevicDusan
Copy link
Copy Markdown
Contributor

@JakovljevicDusan JakovljevicDusan commented May 25, 2026

What & why

Existing code suggested that there is actionable error message, while it was regular error message. Enhance that experience with basic actionable error message, and clean-up code.

https://dynamicssmb2.visualstudio.com/1fcb79e7-ab07-432a-a3c6-6cf5a88ba4a5/_apis/wit/attachments/8a094565-e9b3-4c07-82c2-a2c7028a79a7?fileName=image.png

Linked work

Fixes AB#619344

@JakovljevicDusan JakovljevicDusan requested a review from a team as a code owner May 25, 2026 18:21
@github-actions github-actions Bot added the AL: Apps (W1) Add-on apps for W1 label May 25, 2026
@JakovljevicDusan JakovljevicDusan enabled auto-merge (squash) May 25, 2026 18:22
@github-actions github-actions Bot modified the milestone: Version 29.0 May 25, 2026
@github-actions
Copy link
Copy Markdown
Contributor

$\textbf{🟡\ Medium\ Severity\ —\ Style} \quad \color{gray}{\texttt{\small Iteration\ 1}}$

Flaky test re-enabled without evidence of fix

The comments "Test disabled due to inconsistent behavior across environments" and the reference to Bug 613059 were removed, re-enabling ApplicationAreaMgmt_IsQualityManagementApplicationAreaEnabled. If the root cause tracked by Bug 613059 was not resolved as part of this PR, the test will continue to be flaky and may cause intermittent CI failures.

Recommendation:

  • Only remove the disabled comments if Bug 613059 has been verified as fixed. Add a comment referencing the fix commit or work item that resolved the instability, so reviewers can confirm the test is now stable.
// Re-enabled after fix for Bug 613059 (resolved in <commit/workitem>)
[Test]
procedure ApplicationAreaMgmt_IsQualityManagementApplicationAreaEnabled()

Line mapping was unavailable, so this was posted as an issue comment.

👍 useful · ❤️ especially valuable · 👎 wrong - reply with why

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

Labels

AL: Apps (W1) Add-on apps for W1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant