feature/refactor-ci #43
Merged
Conversation
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
Le-Caignec
changed the title
10 tasks
…ack-based-on-fork' into feature/refactor-ci
- Bump @graphprotocol/graph-cli to version 0.96.0 - Bump @graphprotocol/graph-ts to version 0.38.0 - Add ts-node as a dependency with version 10.9.2 - Remove unnecessary newline in scripts section
🔒 Trivy Security Scan ResultsClick to expand detailed results |
Le-Caignec
commented
Apr 17, 2025
Comment on lines
+38
to
+39
| "@graphprotocol/graph-cli": "^0.96.0", | ||
| "@graphprotocol/graph-ts": "^0.38.0", |
There was a problem hiding this comment.
Error: Unsupported platform: Linux x64 24
at TestCommand.getPlatform (/home/runner/work/PoCo-subgraph/PoCo-subgraph/node_modules/@graphprotocol/graph-cli/dist/commands/test.js:188:11)
at async TestCommand.runBinary (/home/runner/work/PoCo-subgraph/PoCo-subgraph/node_modules/@graphprotocol/graph-cli/dist/commands/test.js:127:22)
Node.js v20.19.0
In the CI we have this issue. We need to update graphprotocol dependencies ✅
zguesmi
reviewed
Apr 18, 2025
|
|
|
TODO: remove |
… feature/refactor-ci
🔒 Trivy Security Scan ResultsClick to expand detailed results |
🔒 Trivy Security Scan ResultsClick to expand detailed results |
🔒 Trivy Security Scan ResultsClick to expand detailed results |
1 similar comment
🔒 Trivy Security Scan ResultsClick to expand detailed results |
…ack-based-on-fork' into feature/refactor-ci
🔒 Trivy Security Scan ResultsClick to expand detailed results |
🔒 Trivy Security Scan ResultsClick to expand detailed results |
| "test": "graph test", | ||
| "test": "npm run test:unit && npm run test:e2e", | ||
| "test:unit": "graph codegen && graph test", | ||
| "test:e2e": "NETWORK_NAME=${NETWORK_NAME:-bellecour} mocha", |
There was a problem hiding this comment.
does it work ? can't make it work locally
There was a problem hiding this comment.
yeah you can make npm run start-test-stack && npm run test to test locally
There was a problem hiding this comment.
ok, but npm run test won't run start-test-stack ?
There was a problem hiding this comment.
I agree with Gabriel, npm run test:e2e should start and stop the local stack.
Also I have an error with npm run test:unit
There was a problem hiding this comment.
The issue is resolved locally by running sudo apt install libpq-dev.
I am able to successfully run both unit and e2e tests:
Co-authored-by: gfournieriExec <100280020+gfournieriExec@users.noreply.github.com>
🔒 Trivy Security Scan ResultsClick to expand detailed results |
🔒 Trivy Security Scan ResultsClick to expand detailed results |
zguesmi
reviewed
Apr 23, 2025
zguesmi
reviewed
Apr 23, 2025
🔒 Trivy Security Scan ResultsClick to expand detailed results |
1 similar comment
🔒 Trivy Security Scan ResultsClick to expand detailed results |
Co-authored-by: Zied Guesmi <26070035+zguesmi@users.noreply.github.com>
🔒 Trivy Security Scan ResultsClick to expand detailed results |
…syntax in integration tests
🔒 Trivy Security Scan ResultsClick to expand detailed results |
Le-Caignec
merged commit 8703e18
into
feature/setup-agnostique-test-stack-based-on-fork
2 of 3 checks passed
zguesmi
added a commit
that referenced
this pull request
Apr 29, 2025
* Add Docker support and environment setup for poco-subgraph - Introduced .dockerignore and Dockerfile for containerization. - Updated networks.json with new addresses and start blocks. - Enhanced package.json with new scripts for cleaning and managing the build process. - Created .gitignore for test-stack to exclude .env files. - Added docker-compose.yml to define services for the test environment. - Implemented prepare-test-env.js for dynamic environment variable generation. - Removed tsconfig.json as it is no longer needed. * Update CHANGELOG and README with Docker subgraph deployer details * Fix Docker run command to use correct image name for poco-subgraph-deployer * Add PR number in changelog * Remove unused 'fork-test' network configuration and update Docker Compose to use dynamic NETWORK_NAME for Ethereum service * feat: add typechain generation script and update dependencies - Added a new script to generate TypeChain types for ethers-v6 in package.json. - Updated the start-test-stack script to ensure proper environment setup. - Modified the itest script to include NETWORK_NAME in the environment. - Removed unused callHandlers from subgraph.yaml. - Enhanced docker-compose.yml to include a stack-ready service for better orchestration. - Updated prepare-test-env.js to conditionally update networks.json based on NETWORK_NAME. - Introduced tsconfig.json for TypeScript configuration with strict settings. * docs: update CHANGELOG to include README section for Docker-based subgraph deployer * chore: update .gitignore to include test-stack/.env and remove redundant .gitignore file * test: update integration test description to reflect app indexing functionality * refactor: restructure prepare-test-env.js to improve block number fetching and environment file creation * fix: simplify itest script by removing DEBUG flag * fix: update start-test-stack script to use export for NETWORK_NAME * Update test-stack/prepare-test-env.js Co-authored-by: gfournieriExec <100280020+gfournieriExec@users.noreply.github.com> * refactor: streamline package.json scripts by removing redundant clean and codegen commands * refactor: update build and test scripts for consistency and clarity * fix: simplify remove drone logic * fix: ensure typechain generation runs before starting test stack * refactor: simplify README instructions for local development and testing * fix: reorder unit test execution in CI workflow and update README for build instructions * Remove unused files (#42) * Remove obsolete Docker configurations, environment files, and subgraph definitions for bellecour, test, and viviani. Refactor unit tests for IexecCategoryManager and IexecPoco, consolidating mock utilities and event parameter builders into a unified structure. * refactor:github-action-ci (#43) * chore: update GitHub workflows for coverage, deployment, and testing * refactor: add formatting scripts and .prettierignore for code consistency * chore: update dependencies in package.json * fix: correct paths in .gitignore for test artifacts * feature/migrate-integration-tests (#44) * Feature/remove dead handler (#45) * fix: Fix Subgraph deployment (#47) * refactor: update CI workflow to include health checks and integration tests, remove obsolete Jenkinsfile * fix: update fork URL and block number retrieval in CI workflow * fix: update command options and environment variable retrieval in CI workflow * fix: correct health check options and update fork URL retrieval in CI workflow * fix: update IPFS image and health check commands in CI workflow * fix: update CI workflow to include genesis block retrieval and health checks for Graph Node * fix: update IPFS image to use kubo version in docker-compose * fix: retrieve fork URL and block number * fix: remove blockchain-fork service configuration from CI workflow * fix: update fetch block number to use static values and add IPFS readiness check * fix: remove code generation step from README * fix: remove unused services and streamline CI workflow * fix: update FORK_URL to use the correct endpoint for integration tests * fix: uncomment steps for installing dependencies, checking format, building, and running unit tests in CI workflow * fix: uncomment steps for preparing local stack environment and update FORK_BLOCK in test environment setup * fix: update fetch block step to source environment variables and use dynamic values for FORK_URL and FORK_BLOCK * fix: add NETWORK_NAME environment variable to prepare local stack environment step * fix: update fetch block step to source the correct environment file * fix: remove .idea directory from .gitignore * fix: reorder build step to occur before running unit tests * fix: reorder steps to run unit tests after build in CI workflow * fix: move NETWORK_NAME environment variable to the job level in CI workflow * Update test-stack/prepare-test-env.ts Co-authored-by: Zied Guesmi <26070035+zguesmi@users.noreply.github.com> * fix: update changelog for integration tests migration to GitHub Actions * fix: streamline test stack preparation in CI workflow (#46) * fix: remove NETWORK_NAME environment variable from job level in CI workflow * fix: update .env generation message in prepare-test-env script * fix: update integration test structure and add .env.template file * fix: add missing network configurations for avalancheFuji and arbitrumSepolia * fix: add Arbitrum & Avalanche network to changelog * fix: add dotenv and zod to package.json and package-lock.json; create env.ts for environment variable validation * fix: update test:unit script to include 'unit' argument for clarity * fix: update startBlock values and rename avalancheFuji to fuji in networks.json * feat: add Protocol entity to Core data source in subgraph.yaml * fix: update test:e2e script to specify test file pattern * fix: downgrade @graphprotocol/graph-ts to version 0.35.1 in package.json and package-lock.json * feat: add dotenv-cli dependency and update build scripts to use dotenv --------- Co-authored-by: Ugo Mignon <ugo.mignon@gameverse.app> Co-authored-by: Zied Guesmi <26070035+zguesmi@users.noreply.github.com> --------- Co-authored-by: gfournieriExec <100280020+gfournieriExec@users.noreply.github.com> Co-authored-by: Ugo Mignon <ugo.mignon@gameverse.app> Co-authored-by: Zied Guesmi <26070035+zguesmi@users.noreply.github.com>
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.
No description provided.