Skip to content

Conversation

rayluo
Copy link
Contributor

@rayluo rayluo commented Aug 12, 2025

Disables the relevant e2e test cases

@rayluo rayluo requested a review from a team as a code owner August 12, 2025 04:56
config["password"] = self.get_lab_user_secret(config["lab_name"])
self._test_username_password(**config)

@unittest.skip(_ADFS_LABS_DECOMMISSIONED)
Copy link
Member

Choose a reason for hiding this comment

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

ADFSv2 was not decommissioned

Copy link
Contributor Author

Choose a reason for hiding this comment

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

When running this test case, it will visit the MEX endpoint of the ADFS https://fs.msidlab4.com/adfs/services/trust/mex

That endpoint gives an http 504 error right now. If it is not decommissioned, then we will need to investigate whether it is not configured properly.

self._test_username_password(**config)

@unittest.skip(_ADFS_LABS_DECOMMISSIONED)
def test_adfs2019_fed_user(self):
Copy link
Member

Choose a reason for hiding this comment

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

not decommisioned

Copy link
Contributor Author

Choose a reason for hiding this comment

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

When running this test case, it will visit the MEX endpoint of the ADFS https://fs.msidlab8.com/adfs/services/trust/mex

That endpoint gives an http 504 error right now. If it is not decommissioned, then we will need to investigate whether it is not configured properly.

Copy link
Member

@bgavrilMS bgavrilMS left a comment

Choose a reason for hiding this comment

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

advs 2019 and v4 were not decommissioned.

config["password"] = self.get_lab_user_secret(config["lab_name"])
self._test_username_password(**config)

@unittest.skip(_ADFS_LABS_DECOMMISSIONED)

Choose a reason for hiding this comment

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

Suggested change
@unittest.skip(_ADFS_LABS_DECOMMISSIONED)
@unittest.skip(_ADFS_TESTS)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@bgavrilMS and @gladjohn ,
Just to be clear. All the modified test cases are currently failing, so I have to disable them to make the build pipeline passing (or else I wouldn't be able to ship MSAL Python 1.34.0b1 last week).

@gladjohn , if your concern is only about the wording, I/we could change the message in line 844.

@rayluo rayluo force-pushed the adfs-labs-decommissioned branch from 854136a to bebbdd4 Compare August 20, 2025 16:29
@rayluo rayluo merged commit bebbdd4 into dev Aug 20, 2025
22 of 29 checks passed
@rayluo rayluo deleted the adfs-labs-decommissioned branch August 20, 2025 20:07
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