Revert navigateToHome changes: restore original implementation#16
Merged
Revert navigateToHome changes: restore original implementation#16
Conversation
- Update BrowserStack Appium version from 2.0.0 to 3.0.0 to match local - Simplify click method: use waitForDisplayed + element.click() - Restore original simple click implementation
- Replace UIAutomator selectors with accessibility IDs using ~ prefix - Update BasePage: menuButton, homeButton, languageModalCloseButton - Update HomePage: all semester cards and buttons (Semester1-6, Interview, Blog, Compiler, RateUs, Share, Contribute) - Accessibility IDs are more reliable on BrowserStack for React Native apps
- BrowserStack doesn't support Appium 3.0.0 for Android - Remove appiumVersion to let BrowserStack use its default compatible version - Appium 3 code should work with BrowserStack's default version due to WebDriver protocol compatibility
- Add new compiler.spec.ts with compiler page tests - Remove failing WebView test from compiler suite - Refactor HomePage.ts: use accessibility IDs, reduce repetition - Convert all selectors to use ~ prefix for accessibility IDs - Group selectors with comments for better organization
- Change isOnHomePage() to use menuButton (more reliable) - Remove waitForDisplayed timeout from navigateToHome() - This fixes BrowserStack test failures where Semester1 element wasn't found - Menu button is always present on home page and more reliable indicator
- Remove GitHub Actions workflows (ci-browserstack.yml, e2e-tests.yml) - Remove BrowserStack configuration (wdio.browserstack.conf.ts) - Remove BrowserStack scripts (upload-to-browserstack.js) - Remove BrowserStack documentation (BROWSERSTACK_SETUP.md) - Remove BrowserStack npm scripts from package.json - Revert test files to original state (UIAutomator selectors) - Remove compiler test file - Clean project without CI/CD setup
- Add compiler page tests (navigation only, no WebView interactions) - Simplify HomePage.ts with one-line getters - Update e2e/README.md with current test coverage and structure - Add ROADMAP.md with future work items
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.