Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
28 commits
Select commit Hold shift + click to select a range
9e4ac65
refactor: update CI workflow to include health checks and integration…
Le-Caignec Apr 24, 2025
85a202f
fix: update fork URL and block number retrieval in CI workflow
Le-Caignec Apr 24, 2025
3076c4f
fix: update command options and environment variable retrieval in CI …
Le-Caignec Apr 24, 2025
315e600
fix: correct health check options and update fork URL retrieval in CI…
Le-Caignec Apr 24, 2025
c11ac45
fix: update IPFS image and health check commands in CI workflow
Le-Caignec Apr 24, 2025
8fed9da
fix: update CI workflow to include genesis block retrieval and health…
TartanLeGrand Apr 24, 2025
c4451d0
fix: update IPFS image to use kubo version in docker-compose
Le-Caignec Apr 24, 2025
8656831
fix: retrieve fork URL and block number
TartanLeGrand Apr 24, 2025
2041a68
fix: remove blockchain-fork service configuration from CI workflow
TartanLeGrand Apr 24, 2025
bc85f68
fix: update fetch block number to use static values and add IPFS read…
Le-Caignec Apr 24, 2025
7f171a4
Merge branch 'feature/migrate-integration-tests' of https://github.co…
Le-Caignec Apr 24, 2025
52ca2ae
fix: remove code generation step from README
Le-Caignec Apr 24, 2025
c41a006
fix: remove unused services and streamline CI workflow
Le-Caignec Apr 24, 2025
4ec3980
fix: update FORK_URL to use the correct endpoint for integration tests
Le-Caignec Apr 24, 2025
52e0101
fix: uncomment steps for installing dependencies, checking format, bu…
Le-Caignec Apr 24, 2025
9bcffbc
fix: uncomment steps for preparing local stack environment and update…
Le-Caignec Apr 24, 2025
c3bed0b
fix: update fetch block step to source environment variables and use …
Le-Caignec Apr 24, 2025
e5999ae
fix: add NETWORK_NAME environment variable to prepare local stack env…
Le-Caignec Apr 24, 2025
51761db
fix: update fetch block step to source the correct environment file
Le-Caignec Apr 24, 2025
9e17544
fix: remove .idea directory from .gitignore
Le-Caignec Apr 24, 2025
a20ad2d
fix: reorder build step to occur before running unit tests
Le-Caignec Apr 24, 2025
32f7059
fix: reorder steps to run unit tests after build in CI workflow
Le-Caignec Apr 24, 2025
2ee27cd
fix: move NETWORK_NAME environment variable to the job level in CI wo…
Le-Caignec Apr 24, 2025
d9c8167
Update test-stack/prepare-test-env.ts
Le-Caignec Apr 25, 2025
b660ee1
fix: update changelog for integration tests migration to GitHub Actions
Le-Caignec Apr 25, 2025
124a15b
fix: streamline test stack preparation in CI workflow (#46)
Le-Caignec Apr 25, 2025
cebad13
fix: remove NETWORK_NAME environment variable from job level in CI wo…
Le-Caignec Apr 25, 2025
874e4c0
fix: update .env generation message in prepare-test-env script
Le-Caignec Apr 25, 2025
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 8 additions & 3 deletions .github/workflows/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,9 +15,9 @@ concurrency:
cancel-in-progress: true

jobs:
build-subgraph:
build-and-test:
runs-on: ubuntu-latest

steps:
- name: Checkout
uses: actions/checkout@v4
Expand All @@ -28,7 +28,7 @@ jobs:
node-version: 20
cache: 'npm'

- name: Init
- name: Install dependencies
run: npm ci

- name: Check Format
Expand All @@ -40,3 +40,8 @@ jobs:
- name: Run unit tests
run: npm run test:unit

- name: Prepare test stack
run: npm run start-test-stack

- name: Run integration tests
run: npm run test:e2e
3 changes: 2 additions & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,8 @@

## vNEXT

- Remove unused files (#42)
- Migrate integration tests from Jenkins to GitHub Actions. (#44)
- Remove unused files to streamline the repository. (#42)
- Rewrite Jenkins CI for future migration (#36, #37)
- Add a modern test stack, agnostic to the forked network. (#38)
- Add a section in the README to explain how to use the Docker-based subgraph deployer. (#38)
Expand Down
31 changes: 0 additions & 31 deletions Jenkinsfile-itest

This file was deleted.

6 changes: 0 additions & 6 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -30,12 +30,6 @@ install project deps
npm ci
```

generate code

```sh
npm run codegen
```

build the project and generate the necessary files:

```sh
Expand Down
2 changes: 1 addition & 1 deletion test-stack/docker-compose.yml
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ services:

ipfs:
restart: unless-stopped
image: ipfs/go-ipfs:v0.34.1
image: ipfs/kubo:v0.34.1
expose:
- 8080
- 5001
Expand Down
16 changes: 8 additions & 8 deletions test-stack/prepare-test-env.ts
Original file line number Diff line number Diff line change
Expand Up @@ -35,14 +35,14 @@ async function createEnvFiles(forkBlockNumber: number) {
console.log('Creating .env file for docker-compose test-stack');
writeFileSync(
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

revert this change to reduce PR diff ?

'.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}`,
'############ THIS FILE IS GENERATED ############\n' +
'# run "node prepare-test-env.js" to regenerate #\n' +
'################################################\n' +
'\n' +
'# blockchain node to use as the reference for the local fork\n' +
`FORK_URL=${forkUrl}\n` +
'# block number to fork from\n' +
`FORK_BLOCK=${forkBlockNumber}\n`,
);
}

Expand Down