-
Notifications
You must be signed in to change notification settings - Fork 0
Integration tests fixes #11
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
Merged
Merged
Changes from all commits
Commits
Show all changes
47 commits
Select commit
Hold shift + click to select a range
5a0ec64
chore: refactor http package to memory and add new test files
ziflex 15a2c81
refactor: clean up driver handling and improve XPath queries
ziflex d44b8c8
refactor: normalize loopback addresses to localhost in CDP driver and…
ziflex c058054
feat: implement DOM event subscription and dispatch functionality
ziflex 2792348
feat: extend DOM event handling with subscription options, event dele…
ziflex a399ba8
refactor: update tests and DOCUMENT function to use CDP driver config…
ziflex 1c5ba1e
fix: adjust timeout calculation in document context to use milliseconds
ziflex 615ed4b
refactor: simplify argument validation and parsing logic in `parse.go…
ziflex 139370d
refactor: enhance cookie management by refining argument parsing and …
ziflex 811234e
chore: update MontFerret/ferret dependency to v2.0.0-alpha.10 and cle…
ziflex 7cbd9c3
refactor: improve argument casting logic in `select.go` by using `run…
ziflex 868d6d7
docs: add `AGENTS.md` as the canonical guide for repository contributors
ziflex 4567a98
fix: correct variable reference in option selection logic in `select.go`
ziflex 0975095
chore: update MontFerret/ferret dependency to v2.0.0-alpha.11 across …
ziflex ba03916
refactor: streamline attribute setting logic by replacing type checks…
ziflex a49f95a
refactor: update `Frames` function for improved argument casting and …
ziflex 8f72416
feat: implement session management for CDP targets with navigation ev…
ziflex f1a628e
feat: enhance iframe handling by adding `WAIT_ELEMENT` support and im…
ziflex aded577
refactor: remove unsupported methods from memory backend and update r…
ziflex a913c5b
refactor: replace `drivers` capability resolvers with root-scoped fun…
ziflex 99b9ddb
refactor: add root-scoped functions for element capability resolution…
ziflex ae450f1
feat: add capability testing matrix and extend query contract tests
ziflex 7dfc2dd
refactor: replace root reload logic with main frame loading and enhan…
ziflex 75b6025
chore: update react-bootstrap version to 2.10.10 for improved compati…
ziflex 8b4d5b7
feat: implement error handling for stream closure and add tests for m…
ziflex 0eef5af
feat: add select interaction capability and enhance test coverage for…
ziflex 0931161
feat: add article fixture and integration tests for article extraction
ziflex 69b344e
feat: add integration workflow and update unit test naming in build c…
ziflex a882371
Potential fix for pull request finding 'CodeQL / Workflow does not co…
ziflex b31c571
feat: introduce utility packages for logging and query parsing, and r…
ziflex 9e1ecb1
Merge remote-tracking branch 'origin/chore/integration-tests' into ch…
ziflex 13a76b8
feat: implement frame management and document interaction methods
ziflex 31ad97c
Update tests/data/pages/dynamic/utils/qs.js
ziflex 3ada6a0
Update tests/data/pages/dynamic/utils/qs.js
ziflex 55df74e
Update modules/web/html/drivers/cdp/input/manager_scroll.go
ziflex 0883f02
refactor: rename access package to data and update references
ziflex a2475cb
refactor: move cssx package to internal and update references
ziflex d4c8d34
refactor: update AttributeSet to use root attribute target and improv…
ziflex 6159eae
chore: update integration.yml to install Ferret Lab using Go instead …
ziflex 7721a3b
refactor: replace fmt.Errorf with runtime.Error for improved error ha…
ziflex 2015576
chore: update integration.yml and local files to use vendored assets …
ziflex 5cc1309
chore: update Ferret Lab version in integration.yml to v2.0.0-alpha.8
ziflex 61ce94c
chore: update Chromium image in integration.yml to use latest tag
ziflex 80fbcc2
chore: update Chromium image in integration.yml to use latest tag
ziflex a0d6d55
refactor: enhance error handling in event stream and client message r…
ziflex 3889222
chore: update integration.yml to configure host resolution for integr…
ziflex 7d561c3
refactor: improve keyboard event handling and add tests for key events
ziflex File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
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
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,87 @@ | ||
| name: Integration | ||
|
|
||
| on: | ||
| push: | ||
| branches: | ||
| - main | ||
| - master | ||
| pull_request: | ||
|
|
||
| permissions: | ||
| contents: read | ||
|
|
||
| jobs: | ||
| discover-integration-modules: | ||
| name: Discover integration modules | ||
| runs-on: ubuntu-latest | ||
| outputs: | ||
| modules: ${{ steps.discover.outputs.modules }} | ||
|
|
||
| steps: | ||
| - name: Checkout | ||
| uses: actions/checkout@v4 | ||
|
|
||
| - name: Discover integration modules | ||
| id: discover | ||
| shell: bash | ||
| run: | | ||
| modules_json="$( | ||
| make modules \ | ||
| | while IFS= read -r module; do | ||
| if [ -d "tests/modules/$module" ]; then | ||
| printf '%s\n' "$module" | ||
| fi | ||
| done \ | ||
| | jq -R . \ | ||
| | jq -s -c . | ||
| )" | ||
|
|
||
| echo "modules=$modules_json" >> "$GITHUB_OUTPUT" | ||
| echo "Discovered integration modules: $modules_json" | ||
|
|
||
| integration: | ||
| name: Integration (${{ matrix.module }}) | ||
| runs-on: ubuntu-latest | ||
| needs: discover-integration-modules | ||
|
|
||
| strategy: | ||
| fail-fast: false | ||
| matrix: | ||
| module: ${{ fromJson(needs.discover-integration-modules.outputs.modules) }} | ||
|
|
||
| services: | ||
| chromium: | ||
| image: ghcr.io/montferret/chromium:latest | ||
| options: --add-host=host.docker.internal:host-gateway | ||
| ports: | ||
| - 9222:9222 | ||
|
|
||
| steps: | ||
| - name: Checkout | ||
| uses: actions/checkout@v4 | ||
|
|
||
| - name: Setup Go | ||
| uses: actions/setup-go@v5 | ||
| with: | ||
| go-version-file: go.work | ||
| cache: true | ||
|
|
||
| - name: Install Ferret Lab | ||
| run: | | ||
| GOBIN="$GITHUB_WORKSPACE" go install github.com/MontFerret/lab/v2@v2.0.0-alpha.8 | ||
| "$GITHUB_WORKSPACE/lab" version | ||
|
|
||
| - name: Add Ferret Lab to PATH | ||
| run: echo "$GITHUB_WORKSPACE" >> "$GITHUB_PATH" | ||
|
|
||
| - name: Resolve Lab static host on runner | ||
| run: echo "127.0.0.1 host.docker.internal" | sudo tee -a /etc/hosts | ||
|
|
||
| - name: Build runtime | ||
| run: make build ${{ matrix.module }} | ||
|
|
||
| - name: Run integration tests | ||
| env: | ||
| LAB_SERVE_HOST: host.docker.internal | ||
| LAB_SERVE_BIND: 0.0.0.0 | ||
| run: make test-integration ${{ matrix.module }} | ||
|
github-advanced-security[bot] marked this conversation as resolved.
Fixed
|
||
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
Oops, something went wrong.
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.