-
Notifications
You must be signed in to change notification settings - Fork 10
Add UI_tests for GitHub link verification on DH #52
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the ✨ Finishing Touches🧪 Generate unit tests
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Your test is very verbose. Please consider if we really need all those console.log
statements - I understand it's useful for debugging, but it seems to be overhead for such a simple test.
The usual way is to have a logger rather then outputting console.log
, but I'm not sure if we already have some. So feel free to use console.log
if needed. Thank you
- Remove networkidle dependency, use domcontentloaded instead - Simplify locator strategy to single comprehensive selector - Reduce console.log statements for cleaner output - Update comment to reflect component assignment vs creation - Maintain authentication handling through separate auth.setup.ts Addresses feedback from review by Katka92 in PR redhat-appstudio#52
- Remove networkidle dependency, use domcontentloaded instead - Simplify locator strategy to single comprehensive selector - Reduce console.log statements for cleaner output - Update comment to reflect component assignment vs creation - Maintain authentication handling through separate auth.setup.ts Addresses feedback from review by Katka92 in PR redhat-appstudio#52
Thank you for addressing all the comments. The code is much cleaner and easier to read. 👍 Now we need to address best practices and follow the test design. So there is still some work needed, like:
I imagine it to be something like:
|
- Remove networkidle dependency, use domcontentloaded instead - Simplify locator strategy to single comprehensive selector - Reduce console.log statements for cleaner output - Update comment to reflect component assignment vs creation - Maintain authentication handling through separate auth.setup.ts Addresses feedback from review by Katka92 in PR redhat-appstudio#52
- Remove networkidle dependency, use domcontentloaded instead - Simplify locator strategy to single comprehensive selector - Reduce console.log statements for cleaner output - Update comment to reflect component assignment vs creation - Maintain authentication handling through separate auth.setup.ts Addresses feedback from review by Katka92 in PR redhat-appstudio#52
- Remove networkidle dependency, use domcontentloaded instead - Simplify locator strategy to single comprehensive selector - Reduce console.log statements for cleaner output - Update comment to reflect component assignment vs creation - Maintain authentication handling through separate auth.setup.ts Addresses feedback from review by Katka92 in PR redhat-appstudio#52
- Add 'Test Git Plugin' test suite for GitHub repository URL verification - Test verifies GitHub 'View Source' link on component Overview page - Replace random component names with IMAGE_REGISTRY_ORG env var for more reliable testing Test validates the Git plugin integration by checking that GitHub repository URLs are properly displayed and accessible from the component Overview page.
- Remove networkidle dependency, use domcontentloaded instead - Simplify locator strategy to single comprehensive selector - Reduce console.log statements for cleaner output - Update comment to reflect component assignment vs creation - Maintain authentication handling through separate auth.setup.ts Addresses feedback from review by Katka92 in PR redhat-appstudio#52
- Remove excessive console.log statements - Simplify locator strategy to single GitHub link selector - Remove unnecessary try-catch complexity for HTTP status check - Clean up code structure and remove step-by-step comments
Extract git verification logic from UI tests into dedicated plugin classes - Add checkViewSourceLink method to git UI interface and implementations - Extend GitHub and GitLab plugins with link verification functionality - Simplify UI test by delegating to git plugin methods - Add shell scripts for running authentication and UI test sequences
* Add UI_tests for GitHub link verification on DH - Add 'Test Git Plugin' test suite for GitHub repository URL verification - Test verifies GitHub 'View Source' link on component Overview page - Replace random component names with IMAGE_REGISTRY_ORG env var for more reliable testing Test validates the Git plugin integration by checking that GitHub repository URLs are properly displayed and accessible from the component Overview page. * Address PR review feedback for GitHub link verification test - Remove networkidle dependency, use domcontentloaded instead - Simplify locator strategy to single comprehensive selector - Reduce console.log statements for cleaner output - Update comment to reflect component assignment vs creation - Maintain authentication handling through separate auth.setup.ts Addresses feedback from review by Katka92 in PR redhat-appstudio#52 * Simplify and clean up GitHub link verification test - Remove excessive console.log statements - Simplify locator strategy to single GitHub link selector - Remove unnecessary try-catch complexity for HTTP status check - Clean up code structure and remove step-by-step comments * Update ui.test.ts * Update ui test * Refactor git UI verification into plugin architecture Extract git verification logic from UI tests into dedicated plugin classes - Add checkViewSourceLink method to git UI interface and implementations - Extend GitHub and GitLab plugins with link verification functionality - Simplify UI test by delegating to git plugin methods - Add shell scripts for running authentication and UI test sequences * Small updates * Small updates * Update UI tests * Remove files with secrets * Remove gitlab case
* Add UI_tests for GitHub link verification on DH - Add 'Test Git Plugin' test suite for GitHub repository URL verification - Test verifies GitHub 'View Source' link on component Overview page - Replace random component names with IMAGE_REGISTRY_ORG env var for more reliable testing Test validates the Git plugin integration by checking that GitHub repository URLs are properly displayed and accessible from the component Overview page. * Address PR review feedback for GitHub link verification test - Remove networkidle dependency, use domcontentloaded instead - Simplify locator strategy to single comprehensive selector - Reduce console.log statements for cleaner output - Update comment to reflect component assignment vs creation - Maintain authentication handling through separate auth.setup.ts Addresses feedback from review by Katka92 in PR #52 * Simplify and clean up GitHub link verification test - Remove excessive console.log statements - Simplify locator strategy to single GitHub link selector - Remove unnecessary try-catch complexity for HTTP status check - Clean up code structure and remove step-by-step comments * Update ui.test.ts * Update ui test * Refactor git UI verification into plugin architecture Extract git verification logic from UI tests into dedicated plugin classes - Add checkViewSourceLink method to git UI interface and implementations - Extend GitHub and GitLab plugins with link verification functionality - Simplify UI test by delegating to git plugin methods - Add shell scripts for running authentication and UI test sequences * Small updates * Small updates * Update UI tests * Remove files with secrets * Remove gitlab case
* Add UI_tests for GitHub link verification on DH - Add 'Test Git Plugin' test suite for GitHub repository URL verification - Test verifies GitHub 'View Source' link on component Overview page - Replace random component names with IMAGE_REGISTRY_ORG env var for more reliable testing Test validates the Git plugin integration by checking that GitHub repository URLs are properly displayed and accessible from the component Overview page. * Address PR review feedback for GitHub link verification test - Remove networkidle dependency, use domcontentloaded instead - Simplify locator strategy to single comprehensive selector - Reduce console.log statements for cleaner output - Update comment to reflect component assignment vs creation - Maintain authentication handling through separate auth.setup.ts Addresses feedback from review by Katka92 in PR #52 * Simplify and clean up GitHub link verification test - Remove excessive console.log statements - Simplify locator strategy to single GitHub link selector - Remove unnecessary try-catch complexity for HTTP status check - Clean up code structure and remove step-by-step comments * Update ui.test.ts * Update ui test * Refactor git UI verification into plugin architecture Extract git verification logic from UI tests into dedicated plugin classes - Add checkViewSourceLink method to git UI interface and implementations - Extend GitHub and GitLab plugins with link verification functionality - Simplify UI test by delegating to git plugin methods - Add shell scripts for running authentication and UI test sequences * Small updates * Small updates * Update UI tests * Remove files with secrets * Remove gitlab case
* [RHTAPUI-85] Adds authentication setup * Add UI_tests for GitHub link verification on DH (#52) * Add UI_tests for GitHub link verification on DH - Add 'Test Git Plugin' test suite for GitHub repository URL verification - Test verifies GitHub 'View Source' link on component Overview page - Replace random component names with IMAGE_REGISTRY_ORG env var for more reliable testing Test validates the Git plugin integration by checking that GitHub repository URLs are properly displayed and accessible from the component Overview page. * Address PR review feedback for GitHub link verification test - Remove networkidle dependency, use domcontentloaded instead - Simplify locator strategy to single comprehensive selector - Reduce console.log statements for cleaner output - Update comment to reflect component assignment vs creation - Maintain authentication handling through separate auth.setup.ts Addresses feedback from review by Katka92 in PR #52 * Simplify and clean up GitHub link verification test - Remove excessive console.log statements - Simplify locator strategy to single GitHub link selector - Remove unnecessary try-catch complexity for HTTP status check - Clean up code structure and remove step-by-step comments * Update ui.test.ts * Update ui test * Refactor git UI verification into plugin architecture Extract git verification logic from UI tests into dedicated plugin classes - Add checkViewSourceLink method to git UI interface and implementations - Extend GitHub and GitLab plugins with link verification functionality - Simplify UI test by delegating to git plugin methods - Add shell scripts for running authentication and UI test sequences * Small updates * Small updates * Update UI tests * Remove files with secrets * Remove gitlab case * feat(RHTAPUI-84): don't print secrets when loaded from env vars (#80) * [RHTAPUI-85] Adds authentication setup * feat(RHTAPUI-84): blur input field for 2FA token --------- Co-authored-by: Jan Smid <jsmid@redhat.com> Co-authored-by: BohdanMar <68649482+BohdanMar@users.noreply.github.com>
* Add UI_tests for GitHub link verification on DH - Add 'Test Git Plugin' test suite for GitHub repository URL verification - Test verifies GitHub 'View Source' link on component Overview page - Replace random component names with IMAGE_REGISTRY_ORG env var for more reliable testing Test validates the Git plugin integration by checking that GitHub repository URLs are properly displayed and accessible from the component Overview page. * Address PR review feedback for GitHub link verification test - Remove networkidle dependency, use domcontentloaded instead - Simplify locator strategy to single comprehensive selector - Reduce console.log statements for cleaner output - Update comment to reflect component assignment vs creation - Maintain authentication handling through separate auth.setup.ts Addresses feedback from review by Katka92 in PR #52 * Simplify and clean up GitHub link verification test - Remove excessive console.log statements - Simplify locator strategy to single GitHub link selector - Remove unnecessary try-catch complexity for HTTP status check - Clean up code structure and remove step-by-step comments * Update ui.test.ts * Update ui test * Refactor git UI verification into plugin architecture Extract git verification logic from UI tests into dedicated plugin classes - Add checkViewSourceLink method to git UI interface and implementations - Extend GitHub and GitLab plugins with link verification functionality - Simplify UI test by delegating to git plugin methods - Add shell scripts for running authentication and UI test sequences * Small updates * Small updates * Update UI tests * Remove files with secrets * Remove gitlab case
* [RHTAPUI-85] Adds authentication setup * Add UI_tests for GitHub link verification on DH (#52) * Add UI_tests for GitHub link verification on DH - Add 'Test Git Plugin' test suite for GitHub repository URL verification - Test verifies GitHub 'View Source' link on component Overview page - Replace random component names with IMAGE_REGISTRY_ORG env var for more reliable testing Test validates the Git plugin integration by checking that GitHub repository URLs are properly displayed and accessible from the component Overview page. * Address PR review feedback for GitHub link verification test - Remove networkidle dependency, use domcontentloaded instead - Simplify locator strategy to single comprehensive selector - Reduce console.log statements for cleaner output - Update comment to reflect component assignment vs creation - Maintain authentication handling through separate auth.setup.ts Addresses feedback from review by Katka92 in PR #52 * Simplify and clean up GitHub link verification test - Remove excessive console.log statements - Simplify locator strategy to single GitHub link selector - Remove unnecessary try-catch complexity for HTTP status check - Clean up code structure and remove step-by-step comments * Update ui.test.ts * Update ui test * Refactor git UI verification into plugin architecture Extract git verification logic from UI tests into dedicated plugin classes - Add checkViewSourceLink method to git UI interface and implementations - Extend GitHub and GitLab plugins with link verification functionality - Simplify UI test by delegating to git plugin methods - Add shell scripts for running authentication and UI test sequences * Small updates * Small updates * Update UI tests * Remove files with secrets * Remove gitlab case * feat(RHTAPUI-84): don't print secrets when loaded from env vars (#80) * [RHTAPUI-85] Adds authentication setup * feat(RHTAPUI-84): blur input field for 2FA token --------- Co-authored-by: Jan Smid <jsmid@redhat.com> Co-authored-by: BohdanMar <68649482+BohdanMar@users.noreply.github.com>
* Add UI_tests for GitHub link verification on DH - Add 'Test Git Plugin' test suite for GitHub repository URL verification - Test verifies GitHub 'View Source' link on component Overview page - Replace random component names with IMAGE_REGISTRY_ORG env var for more reliable testing Test validates the Git plugin integration by checking that GitHub repository URLs are properly displayed and accessible from the component Overview page. * Address PR review feedback for GitHub link verification test - Remove networkidle dependency, use domcontentloaded instead - Simplify locator strategy to single comprehensive selector - Reduce console.log statements for cleaner output - Update comment to reflect component assignment vs creation - Maintain authentication handling through separate auth.setup.ts Addresses feedback from review by Katka92 in PR #52 * Simplify and clean up GitHub link verification test - Remove excessive console.log statements - Simplify locator strategy to single GitHub link selector - Remove unnecessary try-catch complexity for HTTP status check - Clean up code structure and remove step-by-step comments * Update ui.test.ts * Update ui test * Refactor git UI verification into plugin architecture Extract git verification logic from UI tests into dedicated plugin classes - Add checkViewSourceLink method to git UI interface and implementations - Extend GitHub and GitLab plugins with link verification functionality - Simplify UI test by delegating to git plugin methods - Add shell scripts for running authentication and UI test sequences * Small updates * Small updates * Update UI tests * Remove files with secrets * Remove gitlab case
* [RHTAPUI-85] Adds authentication setup * Add UI_tests for GitHub link verification on DH (#52) * Add UI_tests for GitHub link verification on DH - Add 'Test Git Plugin' test suite for GitHub repository URL verification - Test verifies GitHub 'View Source' link on component Overview page - Replace random component names with IMAGE_REGISTRY_ORG env var for more reliable testing Test validates the Git plugin integration by checking that GitHub repository URLs are properly displayed and accessible from the component Overview page. * Address PR review feedback for GitHub link verification test - Remove networkidle dependency, use domcontentloaded instead - Simplify locator strategy to single comprehensive selector - Reduce console.log statements for cleaner output - Update comment to reflect component assignment vs creation - Maintain authentication handling through separate auth.setup.ts Addresses feedback from review by Katka92 in PR #52 * Simplify and clean up GitHub link verification test - Remove excessive console.log statements - Simplify locator strategy to single GitHub link selector - Remove unnecessary try-catch complexity for HTTP status check - Clean up code structure and remove step-by-step comments * Update ui.test.ts * Update ui test * Refactor git UI verification into plugin architecture Extract git verification logic from UI tests into dedicated plugin classes - Add checkViewSourceLink method to git UI interface and implementations - Extend GitHub and GitLab plugins with link verification functionality - Simplify UI test by delegating to git plugin methods - Add shell scripts for running authentication and UI test sequences * Small updates * Small updates * Update UI tests * Remove files with secrets * Remove gitlab case * feat(RHTAPUI-84): don't print secrets when loaded from env vars (#80) * [RHTAPUI-85] Adds authentication setup * feat(RHTAPUI-84): blur input field for 2FA token --------- Co-authored-by: Jan Smid <jsmid@redhat.com> Co-authored-by: BohdanMar <68649482+BohdanMar@users.noreply.github.com>
* Add UI_tests for GitHub link verification on DH - Add 'Test Git Plugin' test suite for GitHub repository URL verification - Test verifies GitHub 'View Source' link on component Overview page - Replace random component names with IMAGE_REGISTRY_ORG env var for more reliable testing Test validates the Git plugin integration by checking that GitHub repository URLs are properly displayed and accessible from the component Overview page. * Address PR review feedback for GitHub link verification test - Remove networkidle dependency, use domcontentloaded instead - Simplify locator strategy to single comprehensive selector - Reduce console.log statements for cleaner output - Update comment to reflect component assignment vs creation - Maintain authentication handling through separate auth.setup.ts Addresses feedback from review by Katka92 in PR redhat-appstudio#52 * Simplify and clean up GitHub link verification test - Remove excessive console.log statements - Simplify locator strategy to single GitHub link selector - Remove unnecessary try-catch complexity for HTTP status check - Clean up code structure and remove step-by-step comments * Update ui.test.ts * Update ui test * Refactor git UI verification into plugin architecture Extract git verification logic from UI tests into dedicated plugin classes - Add checkViewSourceLink method to git UI interface and implementations - Extend GitHub and GitLab plugins with link verification functionality - Simplify UI test by delegating to git plugin methods - Add shell scripts for running authentication and UI test sequences * Small updates * Small updates * Update UI tests * Remove files with secrets * Remove gitlab case
* [RHTAPUI-85] Adds authentication setup * Add UI_tests for GitHub link verification on DH (redhat-appstudio#52) * Add UI_tests for GitHub link verification on DH - Add 'Test Git Plugin' test suite for GitHub repository URL verification - Test verifies GitHub 'View Source' link on component Overview page - Replace random component names with IMAGE_REGISTRY_ORG env var for more reliable testing Test validates the Git plugin integration by checking that GitHub repository URLs are properly displayed and accessible from the component Overview page. * Address PR review feedback for GitHub link verification test - Remove networkidle dependency, use domcontentloaded instead - Simplify locator strategy to single comprehensive selector - Reduce console.log statements for cleaner output - Update comment to reflect component assignment vs creation - Maintain authentication handling through separate auth.setup.ts Addresses feedback from review by Katka92 in PR redhat-appstudio#52 * Simplify and clean up GitHub link verification test - Remove excessive console.log statements - Simplify locator strategy to single GitHub link selector - Remove unnecessary try-catch complexity for HTTP status check - Clean up code structure and remove step-by-step comments * Update ui.test.ts * Update ui test * Refactor git UI verification into plugin architecture Extract git verification logic from UI tests into dedicated plugin classes - Add checkViewSourceLink method to git UI interface and implementations - Extend GitHub and GitLab plugins with link verification functionality - Simplify UI test by delegating to git plugin methods - Add shell scripts for running authentication and UI test sequences * Small updates * Small updates * Update UI tests * Remove files with secrets * Remove gitlab case * feat(RHTAPUI-84): don't print secrets when loaded from env vars (redhat-appstudio#80) * [RHTAPUI-85] Adds authentication setup * feat(RHTAPUI-84): blur input field for 2FA token --------- Co-authored-by: Jan Smid <jsmid@redhat.com> Co-authored-by: BohdanMar <68649482+BohdanMar@users.noreply.github.com>
* Add UI_tests for GitHub link verification on DH - Add 'Test Git Plugin' test suite for GitHub repository URL verification - Test verifies GitHub 'View Source' link on component Overview page - Replace random component names with IMAGE_REGISTRY_ORG env var for more reliable testing Test validates the Git plugin integration by checking that GitHub repository URLs are properly displayed and accessible from the component Overview page. * Address PR review feedback for GitHub link verification test - Remove networkidle dependency, use domcontentloaded instead - Simplify locator strategy to single comprehensive selector - Reduce console.log statements for cleaner output - Update comment to reflect component assignment vs creation - Maintain authentication handling through separate auth.setup.ts Addresses feedback from review by Katka92 in PR #52 * Simplify and clean up GitHub link verification test - Remove excessive console.log statements - Simplify locator strategy to single GitHub link selector - Remove unnecessary try-catch complexity for HTTP status check - Clean up code structure and remove step-by-step comments * Update ui.test.ts * Update ui test * Refactor git UI verification into plugin architecture Extract git verification logic from UI tests into dedicated plugin classes - Add checkViewSourceLink method to git UI interface and implementations - Extend GitHub and GitLab plugins with link verification functionality - Simplify UI test by delegating to git plugin methods - Add shell scripts for running authentication and UI test sequences * Small updates * Small updates * Update UI tests * Remove files with secrets * Remove gitlab case
* [RHTAPUI-85] Adds authentication setup * Add UI_tests for GitHub link verification on DH (#52) * Add UI_tests for GitHub link verification on DH - Add 'Test Git Plugin' test suite for GitHub repository URL verification - Test verifies GitHub 'View Source' link on component Overview page - Replace random component names with IMAGE_REGISTRY_ORG env var for more reliable testing Test validates the Git plugin integration by checking that GitHub repository URLs are properly displayed and accessible from the component Overview page. * Address PR review feedback for GitHub link verification test - Remove networkidle dependency, use domcontentloaded instead - Simplify locator strategy to single comprehensive selector - Reduce console.log statements for cleaner output - Update comment to reflect component assignment vs creation - Maintain authentication handling through separate auth.setup.ts Addresses feedback from review by Katka92 in PR #52 * Simplify and clean up GitHub link verification test - Remove excessive console.log statements - Simplify locator strategy to single GitHub link selector - Remove unnecessary try-catch complexity for HTTP status check - Clean up code structure and remove step-by-step comments * Update ui.test.ts * Update ui test * Refactor git UI verification into plugin architecture Extract git verification logic from UI tests into dedicated plugin classes - Add checkViewSourceLink method to git UI interface and implementations - Extend GitHub and GitLab plugins with link verification functionality - Simplify UI test by delegating to git plugin methods - Add shell scripts for running authentication and UI test sequences * Small updates * Small updates * Update UI tests * Remove files with secrets * Remove gitlab case * feat(RHTAPUI-84): don't print secrets when loaded from env vars (#80) * [RHTAPUI-85] Adds authentication setup * feat(RHTAPUI-84): blur input field for 2FA token --------- Co-authored-by: Jan Smid <jsmid@redhat.com> Co-authored-by: BohdanMar <68649482+BohdanMar@users.noreply.github.com>
* Add UI_tests for GitHub link verification on DH - Add 'Test Git Plugin' test suite for GitHub repository URL verification - Test verifies GitHub 'View Source' link on component Overview page - Replace random component names with IMAGE_REGISTRY_ORG env var for more reliable testing Test validates the Git plugin integration by checking that GitHub repository URLs are properly displayed and accessible from the component Overview page. * Address PR review feedback for GitHub link verification test - Remove networkidle dependency, use domcontentloaded instead - Simplify locator strategy to single comprehensive selector - Reduce console.log statements for cleaner output - Update comment to reflect component assignment vs creation - Maintain authentication handling through separate auth.setup.ts Addresses feedback from review by Katka92 in PR #52 * Simplify and clean up GitHub link verification test - Remove excessive console.log statements - Simplify locator strategy to single GitHub link selector - Remove unnecessary try-catch complexity for HTTP status check - Clean up code structure and remove step-by-step comments * Update ui.test.ts * Update ui test * Refactor git UI verification into plugin architecture Extract git verification logic from UI tests into dedicated plugin classes - Add checkViewSourceLink method to git UI interface and implementations - Extend GitHub and GitLab plugins with link verification functionality - Simplify UI test by delegating to git plugin methods - Add shell scripts for running authentication and UI test sequences * Small updates * Small updates * Update UI tests * Remove files with secrets * Remove gitlab case
* [RHTAPUI-85] Adds authentication setup * Add UI_tests for GitHub link verification on DH (#52) * Add UI_tests for GitHub link verification on DH - Add 'Test Git Plugin' test suite for GitHub repository URL verification - Test verifies GitHub 'View Source' link on component Overview page - Replace random component names with IMAGE_REGISTRY_ORG env var for more reliable testing Test validates the Git plugin integration by checking that GitHub repository URLs are properly displayed and accessible from the component Overview page. * Address PR review feedback for GitHub link verification test - Remove networkidle dependency, use domcontentloaded instead - Simplify locator strategy to single comprehensive selector - Reduce console.log statements for cleaner output - Update comment to reflect component assignment vs creation - Maintain authentication handling through separate auth.setup.ts Addresses feedback from review by Katka92 in PR #52 * Simplify and clean up GitHub link verification test - Remove excessive console.log statements - Simplify locator strategy to single GitHub link selector - Remove unnecessary try-catch complexity for HTTP status check - Clean up code structure and remove step-by-step comments * Update ui.test.ts * Update ui test * Refactor git UI verification into plugin architecture Extract git verification logic from UI tests into dedicated plugin classes - Add checkViewSourceLink method to git UI interface and implementations - Extend GitHub and GitLab plugins with link verification functionality - Simplify UI test by delegating to git plugin methods - Add shell scripts for running authentication and UI test sequences * Small updates * Small updates * Update UI tests * Remove files with secrets * Remove gitlab case * feat(RHTAPUI-84): don't print secrets when loaded from env vars (#80) * [RHTAPUI-85] Adds authentication setup * feat(RHTAPUI-84): blur input field for 2FA token --------- Co-authored-by: Jan Smid <jsmid@redhat.com> Co-authored-by: BohdanMar <68649482+BohdanMar@users.noreply.github.com>
* Add UI_tests for GitHub link verification on DH - Add 'Test Git Plugin' test suite for GitHub repository URL verification - Test verifies GitHub 'View Source' link on component Overview page - Replace random component names with IMAGE_REGISTRY_ORG env var for more reliable testing Test validates the Git plugin integration by checking that GitHub repository URLs are properly displayed and accessible from the component Overview page. * Address PR review feedback for GitHub link verification test - Remove networkidle dependency, use domcontentloaded instead - Simplify locator strategy to single comprehensive selector - Reduce console.log statements for cleaner output - Update comment to reflect component assignment vs creation - Maintain authentication handling through separate auth.setup.ts Addresses feedback from review by Katka92 in PR #52 * Simplify and clean up GitHub link verification test - Remove excessive console.log statements - Simplify locator strategy to single GitHub link selector - Remove unnecessary try-catch complexity for HTTP status check - Clean up code structure and remove step-by-step comments * Update ui.test.ts * Update ui test * Refactor git UI verification into plugin architecture Extract git verification logic from UI tests into dedicated plugin classes - Add checkViewSourceLink method to git UI interface and implementations - Extend GitHub and GitLab plugins with link verification functionality - Simplify UI test by delegating to git plugin methods - Add shell scripts for running authentication and UI test sequences * Small updates * Small updates * Update UI tests * Remove files with secrets * Remove gitlab case
* [RHTAPUI-85] Adds authentication setup * Add UI_tests for GitHub link verification on DH (#52) * Add UI_tests for GitHub link verification on DH - Add 'Test Git Plugin' test suite for GitHub repository URL verification - Test verifies GitHub 'View Source' link on component Overview page - Replace random component names with IMAGE_REGISTRY_ORG env var for more reliable testing Test validates the Git plugin integration by checking that GitHub repository URLs are properly displayed and accessible from the component Overview page. * Address PR review feedback for GitHub link verification test - Remove networkidle dependency, use domcontentloaded instead - Simplify locator strategy to single comprehensive selector - Reduce console.log statements for cleaner output - Update comment to reflect component assignment vs creation - Maintain authentication handling through separate auth.setup.ts Addresses feedback from review by Katka92 in PR #52 * Simplify and clean up GitHub link verification test - Remove excessive console.log statements - Simplify locator strategy to single GitHub link selector - Remove unnecessary try-catch complexity for HTTP status check - Clean up code structure and remove step-by-step comments * Update ui.test.ts * Update ui test * Refactor git UI verification into plugin architecture Extract git verification logic from UI tests into dedicated plugin classes - Add checkViewSourceLink method to git UI interface and implementations - Extend GitHub and GitLab plugins with link verification functionality - Simplify UI test by delegating to git plugin methods - Add shell scripts for running authentication and UI test sequences * Small updates * Small updates * Update UI tests * Remove files with secrets * Remove gitlab case * feat(RHTAPUI-84): don't print secrets when loaded from env vars (#80) * [RHTAPUI-85] Adds authentication setup * feat(RHTAPUI-84): blur input field for 2FA token --------- Co-authored-by: Jan Smid <jsmid@redhat.com> Co-authored-by: BohdanMar <68649482+BohdanMar@users.noreply.github.com>
* Add UI_tests for GitHub link verification on DH - Add 'Test Git Plugin' test suite for GitHub repository URL verification - Test verifies GitHub 'View Source' link on component Overview page - Replace random component names with IMAGE_REGISTRY_ORG env var for more reliable testing Test validates the Git plugin integration by checking that GitHub repository URLs are properly displayed and accessible from the component Overview page. * Address PR review feedback for GitHub link verification test - Remove networkidle dependency, use domcontentloaded instead - Simplify locator strategy to single comprehensive selector - Reduce console.log statements for cleaner output - Update comment to reflect component assignment vs creation - Maintain authentication handling through separate auth.setup.ts Addresses feedback from review by Katka92 in PR redhat-appstudio#52 * Simplify and clean up GitHub link verification test - Remove excessive console.log statements - Simplify locator strategy to single GitHub link selector - Remove unnecessary try-catch complexity for HTTP status check - Clean up code structure and remove step-by-step comments * Update ui.test.ts * Update ui test * Refactor git UI verification into plugin architecture Extract git verification logic from UI tests into dedicated plugin classes - Add checkViewSourceLink method to git UI interface and implementations - Extend GitHub and GitLab plugins with link verification functionality - Simplify UI test by delegating to git plugin methods - Add shell scripts for running authentication and UI test sequences * Small updates * Small updates * Update UI tests * Remove files with secrets * Remove gitlab case
* [RHTAPUI-85] Adds authentication setup * Add UI_tests for GitHub link verification on DH (redhat-appstudio#52) * Add UI_tests for GitHub link verification on DH - Add 'Test Git Plugin' test suite for GitHub repository URL verification - Test verifies GitHub 'View Source' link on component Overview page - Replace random component names with IMAGE_REGISTRY_ORG env var for more reliable testing Test validates the Git plugin integration by checking that GitHub repository URLs are properly displayed and accessible from the component Overview page. * Address PR review feedback for GitHub link verification test - Remove networkidle dependency, use domcontentloaded instead - Simplify locator strategy to single comprehensive selector - Reduce console.log statements for cleaner output - Update comment to reflect component assignment vs creation - Maintain authentication handling through separate auth.setup.ts Addresses feedback from review by Katka92 in PR redhat-appstudio#52 * Simplify and clean up GitHub link verification test - Remove excessive console.log statements - Simplify locator strategy to single GitHub link selector - Remove unnecessary try-catch complexity for HTTP status check - Clean up code structure and remove step-by-step comments * Update ui.test.ts * Update ui test * Refactor git UI verification into plugin architecture Extract git verification logic from UI tests into dedicated plugin classes - Add checkViewSourceLink method to git UI interface and implementations - Extend GitHub and GitLab plugins with link verification functionality - Simplify UI test by delegating to git plugin methods - Add shell scripts for running authentication and UI test sequences * Small updates * Small updates * Update UI tests * Remove files with secrets * Remove gitlab case * feat(RHTAPUI-84): don't print secrets when loaded from env vars (redhat-appstudio#80) * [RHTAPUI-85] Adds authentication setup * feat(RHTAPUI-84): blur input field for 2FA token --------- Co-authored-by: Jan Smid <jsmid@redhat.com> Co-authored-by: BohdanMar <68649482+BohdanMar@users.noreply.github.com>
* Add UI_tests for GitHub link verification on DH - Add 'Test Git Plugin' test suite for GitHub repository URL verification - Test verifies GitHub 'View Source' link on component Overview page - Replace random component names with IMAGE_REGISTRY_ORG env var for more reliable testing Test validates the Git plugin integration by checking that GitHub repository URLs are properly displayed and accessible from the component Overview page. * Address PR review feedback for GitHub link verification test - Remove networkidle dependency, use domcontentloaded instead - Simplify locator strategy to single comprehensive selector - Reduce console.log statements for cleaner output - Update comment to reflect component assignment vs creation - Maintain authentication handling through separate auth.setup.ts Addresses feedback from review by Katka92 in PR redhat-appstudio#52 * Simplify and clean up GitHub link verification test - Remove excessive console.log statements - Simplify locator strategy to single GitHub link selector - Remove unnecessary try-catch complexity for HTTP status check - Clean up code structure and remove step-by-step comments * Update ui.test.ts * Update ui test * Refactor git UI verification into plugin architecture Extract git verification logic from UI tests into dedicated plugin classes - Add checkViewSourceLink method to git UI interface and implementations - Extend GitHub and GitLab plugins with link verification functionality - Simplify UI test by delegating to git plugin methods - Add shell scripts for running authentication and UI test sequences * Small updates * Small updates * Update UI tests * Remove files with secrets * Remove gitlab case
* [RHTAPUI-85] Adds authentication setup * Add UI_tests for GitHub link verification on DH (redhat-appstudio#52) * Add UI_tests for GitHub link verification on DH - Add 'Test Git Plugin' test suite for GitHub repository URL verification - Test verifies GitHub 'View Source' link on component Overview page - Replace random component names with IMAGE_REGISTRY_ORG env var for more reliable testing Test validates the Git plugin integration by checking that GitHub repository URLs are properly displayed and accessible from the component Overview page. * Address PR review feedback for GitHub link verification test - Remove networkidle dependency, use domcontentloaded instead - Simplify locator strategy to single comprehensive selector - Reduce console.log statements for cleaner output - Update comment to reflect component assignment vs creation - Maintain authentication handling through separate auth.setup.ts Addresses feedback from review by Katka92 in PR redhat-appstudio#52 * Simplify and clean up GitHub link verification test - Remove excessive console.log statements - Simplify locator strategy to single GitHub link selector - Remove unnecessary try-catch complexity for HTTP status check - Clean up code structure and remove step-by-step comments * Update ui.test.ts * Update ui test * Refactor git UI verification into plugin architecture Extract git verification logic from UI tests into dedicated plugin classes - Add checkViewSourceLink method to git UI interface and implementations - Extend GitHub and GitLab plugins with link verification functionality - Simplify UI test by delegating to git plugin methods - Add shell scripts for running authentication and UI test sequences * Small updates * Small updates * Update UI tests * Remove files with secrets * Remove gitlab case * feat(RHTAPUI-84): don't print secrets when loaded from env vars (redhat-appstudio#80) * [RHTAPUI-85] Adds authentication setup * feat(RHTAPUI-84): blur input field for 2FA token --------- Co-authored-by: Jan Smid <jsmid@redhat.com> Co-authored-by: BohdanMar <68649482+BohdanMar@users.noreply.github.com>
* [RHTAPUI-85] Adds authentication setup * Add UI_tests for GitHub link verification on DH (#52) * Add UI_tests for GitHub link verification on DH - Add 'Test Git Plugin' test suite for GitHub repository URL verification - Test verifies GitHub 'View Source' link on component Overview page - Replace random component names with IMAGE_REGISTRY_ORG env var for more reliable testing Test validates the Git plugin integration by checking that GitHub repository URLs are properly displayed and accessible from the component Overview page. * Address PR review feedback for GitHub link verification test - Remove networkidle dependency, use domcontentloaded instead - Simplify locator strategy to single comprehensive selector - Reduce console.log statements for cleaner output - Update comment to reflect component assignment vs creation - Maintain authentication handling through separate auth.setup.ts Addresses feedback from review by Katka92 in PR #52 * Simplify and clean up GitHub link verification test - Remove excessive console.log statements - Simplify locator strategy to single GitHub link selector - Remove unnecessary try-catch complexity for HTTP status check - Clean up code structure and remove step-by-step comments * Update ui.test.ts * Update ui test * Refactor git UI verification into plugin architecture Extract git verification logic from UI tests into dedicated plugin classes - Add checkViewSourceLink method to git UI interface and implementations - Extend GitHub and GitLab plugins with link verification functionality - Simplify UI test by delegating to git plugin methods - Add shell scripts for running authentication and UI test sequences * Small updates * Small updates * Update UI tests * Remove files with secrets * Remove gitlab case * feat(RHTAPUI-84): don't print secrets when loaded from env vars (#80) * RHTAPUI-84: blur input field for 2FA token (#81) * [RHTAPUI-85] Adds authentication setup * Add UI_tests for GitHub link verification on DH (#52) * Add UI_tests for GitHub link verification on DH - Add 'Test Git Plugin' test suite for GitHub repository URL verification - Test verifies GitHub 'View Source' link on component Overview page - Replace random component names with IMAGE_REGISTRY_ORG env var for more reliable testing Test validates the Git plugin integration by checking that GitHub repository URLs are properly displayed and accessible from the component Overview page. * Address PR review feedback for GitHub link verification test - Remove networkidle dependency, use domcontentloaded instead - Simplify locator strategy to single comprehensive selector - Reduce console.log statements for cleaner output - Update comment to reflect component assignment vs creation - Maintain authentication handling through separate auth.setup.ts Addresses feedback from review by Katka92 in PR #52 * Simplify and clean up GitHub link verification test - Remove excessive console.log statements - Simplify locator strategy to single GitHub link selector - Remove unnecessary try-catch complexity for HTTP status check - Clean up code structure and remove step-by-step comments * Update ui.test.ts * Update ui test * Refactor git UI verification into plugin architecture Extract git verification logic from UI tests into dedicated plugin classes - Add checkViewSourceLink method to git UI interface and implementations - Extend GitHub and GitLab plugins with link verification functionality - Simplify UI test by delegating to git plugin methods - Add shell scripts for running authentication and UI test sequences * Small updates * Small updates * Update UI tests * Remove files with secrets * Remove gitlab case * feat(RHTAPUI-84): don't print secrets when loaded from env vars (#80) * [RHTAPUI-85] Adds authentication setup * feat(RHTAPUI-84): blur input field for 2FA token --------- Co-authored-by: Jan Smid <jsmid@redhat.com> Co-authored-by: BohdanMar <68649482+BohdanMar@users.noreply.github.com> * add authorization button click when needed (#87) * RHTAPUI-84: describe security concerns in a README (#86) * RHTAPUI-84: describe security concerns in a readme * fix duplicated method * [pass_for_undefined] Skips Git tests for not yet implemented providers (#103) * ui(test): add CI provider verification on CI tab; keep only ui.test.ts change (#90) * [RHTAPUI-90] Implements UI tests for Docs tab (#88) * [ui-timeout-increase] Increases timeout for UI tests (#115) * [ui_tests] Fixes based on review --------- Co-authored-by: BohdanMar <68649482+BohdanMar@users.noreply.github.com> Co-authored-by: Katerina Foniok <kkanova@redhat.com>
* Add UI_tests for GitHub link verification on DH - Add 'Test Git Plugin' test suite for GitHub repository URL verification - Test verifies GitHub 'View Source' link on component Overview page - Replace random component names with IMAGE_REGISTRY_ORG env var for more reliable testing Test validates the Git plugin integration by checking that GitHub repository URLs are properly displayed and accessible from the component Overview page. * Address PR review feedback for GitHub link verification test - Remove networkidle dependency, use domcontentloaded instead - Simplify locator strategy to single comprehensive selector - Reduce console.log statements for cleaner output - Update comment to reflect component assignment vs creation - Maintain authentication handling through separate auth.setup.ts Addresses feedback from review by Katka92 in PR redhat-appstudio#52 * Simplify and clean up GitHub link verification test - Remove excessive console.log statements - Simplify locator strategy to single GitHub link selector - Remove unnecessary try-catch complexity for HTTP status check - Clean up code structure and remove step-by-step comments * Update ui.test.ts * Update ui test * Refactor git UI verification into plugin architecture Extract git verification logic from UI tests into dedicated plugin classes - Add checkViewSourceLink method to git UI interface and implementations - Extend GitHub and GitLab plugins with link verification functionality - Simplify UI test by delegating to git plugin methods - Add shell scripts for running authentication and UI test sequences * Small updates * Small updates * Update UI tests * Remove files with secrets * Remove gitlab case
* [RHTAPUI-85] Adds authentication setup * Add UI_tests for GitHub link verification on DH (redhat-appstudio#52) * Add UI_tests for GitHub link verification on DH - Add 'Test Git Plugin' test suite for GitHub repository URL verification - Test verifies GitHub 'View Source' link on component Overview page - Replace random component names with IMAGE_REGISTRY_ORG env var for more reliable testing Test validates the Git plugin integration by checking that GitHub repository URLs are properly displayed and accessible from the component Overview page. * Address PR review feedback for GitHub link verification test - Remove networkidle dependency, use domcontentloaded instead - Simplify locator strategy to single comprehensive selector - Reduce console.log statements for cleaner output - Update comment to reflect component assignment vs creation - Maintain authentication handling through separate auth.setup.ts Addresses feedback from review by Katka92 in PR redhat-appstudio#52 * Simplify and clean up GitHub link verification test - Remove excessive console.log statements - Simplify locator strategy to single GitHub link selector - Remove unnecessary try-catch complexity for HTTP status check - Clean up code structure and remove step-by-step comments * Update ui.test.ts * Update ui test * Refactor git UI verification into plugin architecture Extract git verification logic from UI tests into dedicated plugin classes - Add checkViewSourceLink method to git UI interface and implementations - Extend GitHub and GitLab plugins with link verification functionality - Simplify UI test by delegating to git plugin methods - Add shell scripts for running authentication and UI test sequences * Small updates * Small updates * Update UI tests * Remove files with secrets * Remove gitlab case * feat(RHTAPUI-84): don't print secrets when loaded from env vars (redhat-appstudio#80) * [RHTAPUI-85] Adds authentication setup * feat(RHTAPUI-84): blur input field for 2FA token --------- Co-authored-by: Jan Smid <jsmid@redhat.com> Co-authored-by: BohdanMar <68649482+BohdanMar@users.noreply.github.com>
Test validates the Git plugin integration by checking that GitHub repository URLs are properly displayed and accessible from the component Overview page.