Repository navigation
fix: stabilize CI test harness for browser and Android cleanup - #302
Open
GitToTheHub wants to merge 23 commits into
Open
GitToTheHub wants to merge 23 commits into
GitToTheHub wants to merge 23 commits into
Conversation
The Cordova <asset> element in cordova-plugin-test-framework may not reliably copy assets to www/cdvtests/ during browser platform CI runs. Add ensureCdvTestsExists() that explicitly copies the installed plugin's www/assets/ to www/cdvtests/ if index.html is missing. Also revert start page back to cdvtests/index.html (the actual Jasmine test runner) and update the CSP patching to target that file instead of index.html (which lacks Jasmine and the test runner logic).
Use github:GitToTheHub/cordova-plugin-test-framework which includes the jasmine boot0.js fix required for Jasmine 5.x initialization
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
- PR apache/cordova-plugin-test-framework#69 was merged and the master can now be used
5 tasks
GitToTheHub
added a commit
that referenced
this pull request
Jun 19, 2026
- During testing PR #302, the iOS 26.x test did not succeed by the strange error message: `xcodebuild: error: Unable to find a destination matching the provided destination specifier: { generic:1, platform:iOS Simulator } and also later iOS 26.1 is not installed. Please download and install the platform from Xcode > Settings > Components.`.After changing xcode-version: 26.1 to xcode-version: 26.x XCode 26.5 was choosen and the error disappeared
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
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.
Summary
This PR ports the CI harness fixes validated in downstream plugin CI runs.
Changes
www/cdvtests/for browser runs when needed.cdvtests/index.htmlfor test execution.localhost/127.0.0.1HTTP+WS endpoints used by Paramedic.adb logcatis unavailable during teardown.Motivation
Recent CI failures were caused by harness/start-page mismatches and post-test cleanup failures, not test logic regressions.
Validation
adb logcatteardown errors after successful specs.Downstream Validation
A dedicated downstream test PR is open in
apache/cordova-plugin-network-information:That PR consumes this change chain and demonstrates successful CI runs.
Generated-By: GPT-5.3-Codex