Skip to content
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.

Sign up for GitHub

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

use new ens-test-env #929

Merged
merged 24 commits into from
Feb 7, 2025
Merged

use new ens-test-env #929

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
d0bcd8a
use new ens-test-env
talentlessguy Jan 16, 2025
af28abe
fix ci
talentlessguy Jan 16, 2025
33a7dde
try mini-wait-on
talentlessguy Jan 22, 2025
d172e9a
fix
talentlessguy Jan 22, 2025
9067fb1
revert latest change
talentlessguy Jan 22, 2025
4f2d94a
undo
talentlessguy Jan 22, 2025
d01db5b
fix archive
talentlessguy Jan 22, 2025
0ae1083
ci: add stability window and timeout to wait-on commands
devin-ai-integration[bot] Jan 30, 2025
5f3079e
revert: remove stability window and timeout from wait-on commands
devin-ai-integration[bot] Jan 30, 2025
428c2d7
feat: use exitAfterDeploy flag in CI workflow
devin-ai-integration[bot] Jan 30, 2025
cf5a313
Merge branch 'main' into use-new-test-env
talentlessguy Jan 30, 2025
37baa05
try this
talentlessguy Jan 30, 2025
f21c144
update lockfile
talentlessguy Jan 30, 2025
066a41b
temporarily patch ens-test-env to track the error
talentlessguy Jan 30, 2025
8c11b49
put out console logs to see where exactly test-env gets stuck
talentlessguy Jan 30, 2025
0f9def2
unpatch deps and revert to older subdeps
talentlessguy Jan 30, 2025
c0a87f9
Merge branch 'main' of https://github.com/ensdomains/ens-app-v3 into …
talentlessguy Feb 3, 2025
33db41a
try with using node-fetch@2
talentlessguy Feb 3, 2025
66fe70f
temporarily fix corepack
talentlessguy Feb 3, 2025
c30312f
try a hotfix
talentlessguy Feb 3, 2025
eb50f50
update the archive
talentlessguy Feb 4, 2025
5602d5f
Merge branch 'use-new-test-env' of https://github.com/ensdomains/ens-…
talentlessguy Feb 4, 2025
5b87138
bump ens-test and fix a e2e test
talentlessguy Feb 4, 2025
d0fea43
remove patch
talentlessguy Feb 4, 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
2 changes: 1 addition & 1 deletion .github/workflows/knip.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ jobs:
- uses: actions/checkout@v3

- name: Install pnpm
run: corepack enable pnpm
run: npm i -g --force corepack && corepack enable pnpm

- name: Install Node.js
uses: actions/setup-node@v3
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/pages-deployment.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ jobs:
run: sudo apt-get install -y libcairo2-dev libjpeg8-dev libpango1.0-dev libgif-dev build-essential g++

- name: Install pnpm
run: corepack enable pnpm
run: npm i -g --force corepack && corepack enable pnpm

- name: Install Node.js
uses: actions/setup-node@v3
Expand Down
16 changes: 8 additions & 8 deletions .github/workflows/test.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ jobs:
- uses: actions/checkout@v3

- name: Install pnpm
run: corepack enable pnpm
run: npm i -g --force corepack && corepack enable pnpm

- name: Install Node.js
uses: actions/setup-node@v3
Expand All @@ -24,7 +24,7 @@ jobs:

- name: Get contract addresses
run: 'parallel --lb --halt now,success=1,fail=1 ::: \
"pnpm tenv start -ng -ns -nb" \
"pnpm tenv start --no-graph --no-scripts --no-build --exit-after-deploy" \
"pnpm wait-on ./.env.local"'

- run: pnpm test:coverage
Expand All @@ -35,7 +35,7 @@ jobs:
- uses: actions/checkout@v3

- name: Install pnpm
run: corepack enable pnpm
run: npm i -g --force corepack && corepack enable pnpm

- name: Install Node.js
uses: actions/setup-node@v3
Expand All @@ -47,7 +47,7 @@ jobs:

- name: Get contract addresses
run: 'parallel --lb --halt now,success=1,fail=1 ::: \
"pnpm tenv start -ng -ns -nb" \
"pnpm tenv start --no-graph --no-scripts --no-build --exit-after-deploy" \
"pnpm wait-on ./.env.local"'

- name: Build stateless and export
Expand All @@ -68,7 +68,7 @@ jobs:
- uses: actions/checkout@v3

- name: Install pnpm
run: corepack enable pnpm
run: npm i -g --force corepack && corepack enable pnpm

- name: Install Node.js
uses: actions/setup-node@v3
Expand Down Expand Up @@ -102,7 +102,7 @@ jobs:
- uses: actions/checkout@v3

- name: Install pnpm
run: corepack enable pnpm
run: npm i -g --force corepack && corepack enable pnpm

- name: Install Node.js
uses: actions/setup-node@v3
Expand All @@ -129,7 +129,7 @@ jobs:

- name: Run tests
run: |
PLAYWRIGHT_SHARD=${{matrix.shard}} PLAYWRIGHT_TOTAL=${{strategy.job-total}} pnpm e2e:ci -nb
PLAYWRIGHT_SHARD=${{matrix.shard}} PLAYWRIGHT_TOTAL=${{strategy.job-total}} pnpm e2e:ci --no-build

- uses: actions/upload-artifact@v4
if: always()
Expand All @@ -153,7 +153,7 @@ jobs:
- run: ./scripts/check-chrome.sh

- name: Install pnpm
run: corepack enable pnpm
run: npm i -g --force corepack && corepack enable pnpm

- name: Install Node.js
uses: actions/setup-node@v3
Expand Down
Binary file added archive.tar.gz
View file
Open in desktop
Binary file not shown.
Binary file removed archive.tar.lz4
View file
Open in desktop
Binary file not shown.
4 changes: 2 additions & 2 deletions e2e/specs/stateless/extendNames.spec.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -157,9 +157,9 @@ test('should be able to extend a single unwrapped name from profile', async ({

await test.step('should show correct fiat values', async () => {
await extendNamesModal.getCurrencyToggle.click({ force: true })
await expect(extendNamesModal.getInvoiceExtensionFee).toContainText('$10.00')
await expect(extendNamesModal.getInvoiceExtensionFee).toContainText('$10.01')
await expect(extendNamesModal.getInvoiceTransactionFee).toContainText('$0.13')
await expect(extendNamesModal.getInvoiceTotal).toContainText('$10.13')
await expect(extendNamesModal.getInvoiceTotal).toContainText('$10.14')
await extendNamesModal.getCounterMinusButton.click()
await expect(extendNamesModal.getInvoiceExtensionFee).toContainText('$5.00')
await expect(extendNamesModal.getInvoiceTransactionFee).toContainText('$0.13')
Expand Down
6 changes: 3 additions & 3 deletions package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@
"buildandexport": "pnpm build && pnpm export",
"buildandstart:glocal": "pnpm build:glocal && pnpm start",
"tenv": "ens-test-env -a",
"denv": "pnpm tenv start -ns -nb --extra-time 12232000 --verbosity 1",
"denv": "pnpm tenv start --no-graph --no-build --extra-time 12232000 --verbosity 1",
"lint": "next lint && pnpm stylelint stylelint \"./src/**/*.tsx\"",
"lint:types": "tsc --noEmit",
"lint:fix": "next lint --fix",
Expand Down Expand Up @@ -99,7 +99,7 @@
"devDependencies": {
"@cloudflare/workers-types": "^3.14.1",
"@ensdomains/buffer": "^0.1.1",
"@ensdomains/ens-test-env": "0.5.0-beta.2",
"@ensdomains/ens-test-env": "0.6.0",
"@ensdomains/headless-web3-provider": "^1.0.8",
"@ianvs/prettier-plugin-sort-imports": "^4.1.0",
"@next/bundle-analyzer": "^13.4.19",
Expand Down Expand Up @@ -166,7 +166,7 @@
"vite-plugin-magical-svg": "^1.3.0",
"vitest": "^2.0.5",
"vitest-canvas-mock": "^0.3.3",
"wait-on": "^6.0.1",
"wait-on": "^8.0.2",
talentlessguy marked this conversation as resolved.
Show resolved Hide resolved
"wrangler": "^3.26.0",
"ws": "^8.16.0"
},
Expand Down
Loading