Skip to content

Skip redhat.fabric8-analytics extension test in DS 3.22.0 and higher; log testing extensions - #23524

Merged
dmytro-ndp merged 4 commits into
mainfrom
CRW-9266
Aug 16, 2025
Merged

Skip redhat.fabric8-analytics extension test in DS 3.22.0 and higher; log testing extensions#23524
dmytro-ndp merged 4 commits into
mainfrom
CRW-9266

Conversation

@dmytro-ndp

@dmytro-ndp dmytro-ndp commented Aug 15, 2025

Copy link
Copy Markdown
Contributor

What does this PR do?

This PR fixes the failing RecommendedExtensions.spec.ts test by:

  1. Corrects extension section detection: Updates the expected section title from "Other Recommendations" to "WORKSPACE RECOMMENDATIONS" to match the actual VS Code extension view
  2. Handles problematic extension: Skips the redhat.fabric8-analytics extension in Dev Spaces 3.22+ due to known installation issues
  3. Enhances debugging: Adds comprehensive logging to show found extensions, authors, and comparison results for better troubleshooting
  4. Skip steps in 'Node.js Express' RecommendedExtensions tests without failing the test suite.

Technical Changes:

  • Fixed section title matching in getSectionForCategory() function
  • Added version comparison logic >= '3.22' for Dev Spaces
  • Enhanced error handling with try-catch blocks
  • Added detailed debug logging for extension detection process

Screenshot/screencast of this PR

No screenshot/screencast provided - this is a test infrastructure fix

What issues does this PR fix or reference?

How to test this PR?

Successful test run: https://jenkins-csb-crwqe-main.dno.corp.redhat.com/job/Testing/job/e2e/job/basic/job/typescript-tests/41695/

PR Checklist

As the author of this Pull Request I made sure that:

Reviewers

Reviewers, please comment how you tested the PR when approving it.

--
Assisted-by Cursor

…nsions test

Signed-off-by: Dmytro Nochevnov <dnochevn@redhat.com>
Signed-off-by: Dmytro Nochevnov <dnochevn@redhat.com>
@dmytro-ndp
dmytro-ndp merged commit aeecc2a into main Aug 16, 2025
6 checks passed
@dmytro-ndp
dmytro-ndp deleted the CRW-9266 branch August 16, 2025 13:50
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.

2 participants