IDE-5298: Improve federated auth error message to include AH_ORGANIZATION_UUID guidance

[deepakmishra2]

This pull request improves the guidance provided to users when they encounter authentication errors related to Federated Authentication. The main change is that the help message is now dynamic: it checks if the AH_ORGANIZATION_UUID environment variable is already set and provides tailored instructions accordingly. The test suite has also been updated to cover this new behavior.

Improvements to error handling and user guidance:

• Updated the help messages in ExceptionListener.php to check for AH_ORGANIZATION_UUID in the environment and provide step-by-step instructions based on its presence. If set, users are prompted to run acli auth:login; if not, users are guided to export the UUID first, then run the login command.

Test coverage enhancements:

• Updated the expected help messages in providerTestHelp in ExceptionListenerTest.php to match the new instructions for when the UUID is not set.
• Added a new test, testFederatedAuthHelpWithOrgUuidAlreadySet, to verify that the correct help message is shown when AH_ORGANIZATION_UUID is already present in the environment.
• Added the Group attribute import to support test grouping.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.54%. Comparing base (9270dd3) to head (9b11cb4).

Additional details and impacted files

@@ Coverage Diff @@## main #2031 +/- ##
=========================================
Coverage 92.54% 92.54% - Complexity 2022 2023 +1 
=========================================
Files 126 126 Lines 7307 7310 +3 =========================================
+ Hits 6762 6765 +3 
Misses 545 545 

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[Copilot]

Pull request overview

This pull request updates the Federated Authentication-related error guidance emitted by ExceptionListener so that instructions differ depending on whether AH_ORGANIZATION_UUID is already present in the environment, and extends PHPUnit coverage for the new branching behavior.

Changes:

• Make the federated-auth help text conditional on AH_ORGANIZATION_UUID being set.
• Update existing test expectations for the new “UUID not set” instructions.
• Add a new test covering the “UUID already set” branch (serial group due to env mutation).

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File	Description
src/EventListener/ExceptionListener.php	Adds dynamic federated-auth remediation messages based on AH_ORGANIZATION_UUID.
tests/phpunit/src/Misc/ExceptionListenerTest.php	Updates provider expectations and adds a new test for the “UUID already set” branch.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[github-actions]

Try the dev build for this PR: https://acquia-cli.s3.amazonaws.com/build/pr/2031/acli.phar

curl -OL https://acquia-cli.s3.amazonaws.com/build/pr/2031/acli.phar
chmod +x acli.phar

[deepakmishra2]

@copilot can you apply the suggestion given by you

[Copilot]

@copilot can you apply the suggestion given by you

Done. Replaced <uuid> with YOUR_ORG_UUID in both ExceptionListener.php (line 100) and the matching test expectation in ExceptionListenerTest.php (line 131), so the placeholder is safe from Symfony Console's output formatter. Committed in fa47f97.

[acquia-stalebot-platauto]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Please remove the stale label to avoid it being closed. Thank you for your contributions. More info: https://github.com/acquia/devops-github-administration/blob/main/docs/operations_related_to_repositories.md#acquia-stale-bot