feature/migrate-integration-tests #44
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
… tests, remove obsolete Jenkinsfile
Le-Caignec
changed the title
🔒 Trivy Security Scan ResultsClick to expand detailed results |
🔒 Trivy Security Scan ResultsClick to expand detailed results |
🔒 Trivy Security Scan ResultsClick to expand detailed results |
🔒 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 |
🔒 Trivy Security Scan ResultsClick to expand detailed results |
1 similar comment
🔒 Trivy Security Scan ResultsClick to expand detailed results |
…m/iExecBlockchainComputing/PoCo-subgraph into feature/migrate-integration-tests
🔒 Trivy Security Scan ResultsClick to expand detailed results |
10 tasks
🔒 Trivy Security Scan ResultsClick to expand detailed results |
🔒 Trivy Security Scan ResultsClick to expand detailed results |
🔒 Trivy Security Scan ResultsClick to expand detailed results |
🔒 Trivy Security Scan ResultsClick to expand detailed results |
2 similar comments
🔒 Trivy Security Scan ResultsClick to expand detailed results |
🔒 Trivy Security Scan ResultsClick to expand detailed results |
| */ | ||
| async function createEnvFiles(forkBlockNumber: number) { | ||
| console.log('Creating .env file for docker-compose test-stack'); | ||
| writeFileSync( |
There was a problem hiding this comment.
revert this change to reduce PR diff ?
zguesmi
reviewed
Apr 25, 2025
test-stack/prepare-test-env.ts
Outdated
Comment on lines
38
to
45
| `############ THIS FILE IS GENERATED ############ | ||
| # run "node prepare-test-env.js" to regenerate # | ||
| ################################################ | ||
| # run "node prepare-test-env.js" to regenerate # | ||
| ################################################ | ||
| # blockchain node to use as the reference for the local fork | ||
| FORK_URL=${forkUrl} | ||
| # block number to fork from | ||
| FORK_BLOCK=${forkBlockNumber}`, | ||
| # blockchain node to use as the reference for the local fork | ||
| FORK_URL=${forkUrl} | ||
| # block number to fork from | ||
| FORK_BLOCK=${forkBlockNumber}`, |
There was a problem hiding this comment.
You can do this:
writeFileSync(
'.env',
'############ THIS FILE IS GENERATED ############' +
'# run "node prepare-test-env.js" to regenerate #' +
'################################################' +
'' +
'# blockchain node to use as the reference for the local fork' +
`FORK_URL=${forkUrl}` +
'# block number to fork from' +
`FORK_BLOCK=${forkBlockNumber}`,
);
.github/workflows/main.yml
Outdated
Comment on lines
45
to
47
| - name: Prepare local stack env | ||
| id: prepare | ||
| run: npx tsx ./test-stack/prepare-test-env.ts |
There was a problem hiding this comment.
Why not use npm run start-test-stack which runs prepare-test-env.ts?
It's better to have the same workflow and use the same tooling. Or did you have an issue?
|
Please update Changelog file (until we activate release please) |
TartanLeGrand
approved these changes
Apr 25, 2025
Co-authored-by: Zied Guesmi <26070035+zguesmi@users.noreply.github.com>
🔒 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 |
🔒 Trivy Security Scan ResultsClick to expand detailed results |
zguesmi
approved these changes
Apr 25, 2025
Le-Caignec
merged commit 876ea82
into
feature/setup-agnostique-test-stack-based-on-fork
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.