Skip to content

Conversation

somalaya
Copy link
Contributor

@somalaya somalaya commented Sep 12, 2025

  • Delete LTW test TestCase2572249.java which tests the preference order of active broker and then uninstalls the inactive one to see if SSO is preserved. No longer needed because we do test the preference order in other tests and SSO as well in those tests. We do not have to test by uninstalling the inactive one as it is a given that SSO will be preserved.

Fixes AB#3369362

@somalaya somalaya requested a review from a team as a code owner September 12, 2025 01:48
@Copilot Copilot AI review requested due to automatic review settings September 12, 2025 01:48
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR removes a redundant LTW (Link to Work) test that was testing broker preference order and SSO preservation after uninstalling an inactive broker. The test is no longer necessary as the functionality is adequately covered by other existing tests.

  • Removes TestCase2572249.java which tested broker preference and SSO after LTW uninstall
  • Eliminates redundant test coverage since broker preference order and SSO preservation are tested elsewhere
  • Simplifies the test suite by removing an unnecessary test scenario

Copy link

❌ Work item link check failed. Description does not contain AB#{ID}.

Click here to Learn more.

@somalaya somalaya changed the title Deleted LTW test Deleted LTW test, AB#3369362 Sep 12, 2025
@somalaya somalaya changed the title Deleted LTW test, AB#3369362 Deleted LTW test, Fixes AB#3369362 Sep 12, 2025
Copy link

✅ Work item link check complete. Description contains link AB#3369362 to an Azure Boards work item.

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