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

Jump to bottom

Tests Refactoring #60

Draft
wants to merge 16 commits into
base: next
Choose a base branch
from
Draft

Tests Refactoring #60

wants to merge 16 commits into from

Conversation

PJColombo
Copy link
Member

@PJColombo PJColombo commented Jan 10, 2023 •
edited
Loading

It adds the following:

  • Migration to vitest.
  • Turborepo integration.
  • A testing server to manage forked chains in concurrent test execution environments.
  • Small fixes to some unit tests and fixtures.
  • Unit tests refactoring using snapshots.

@PJColombo
test: create test-helpers package
6e7d2ec
@PJColombo
chore: add the following:
57d3f6d 
- Set up Turborepo
- Add `tsup`
- Move shared test package to `/common`
- Remove `husky` hooks
- Remove `babel`
- Move env variables to root folder
@PJColombo
chore: set up common eslint config for all packages
e0c22be
@PJColombo
chore(lint): add turbo config
a94fa40
@PJColombo
chore: refactor tsconfig usage by creating a shared package
35a4715
@PJColombo
refactor: add the following:
5b137d8 
- separate `giveth` module into its own package
- separate `aragonos` module into its own package
@PJColombo
refactor(aragonOS): move tests to their package
ef78214
@PJColombo
refactor(Tenderly): move module to its own package
91af4d2
@PJColombo
chore: move module packages to their own folder + move evmcrispr to `…
468b72b 
…/core`
@PJColombo
chore: move fixture creation scripts to test package
00babfd
@PJColombo
chore: move dev package deps to root package
5450215
@PJColombo
test(core): migrate to vitest
f14f3aa
@PJColombo
refactor(giveth): migrate tests to vitest + support custom handlers…
92cc63a 
… in mock server
@PJColombo
refactor: add the following:
5894bd0 
- migrate `aragonos` tests to `vitest`.
- use `hardhat` instead of `anvil` due to multiple forked chain processes execution.
- create chain manager express app to orchestrate multiple forked chains on concurrent tests executions.
@vercel
Copy link

vercel bot commented Jan 10, 2023 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
evmcrispr-evmcrispr-terminal ❌ Failed (Inspect) Jan 10, 2023 at 10:36PM (UTC)

@changeset-bot
Copy link

changeset-bot bot commented Jan 10, 2023 •
edited
Loading

⚠️ No Changeset found

Latest commit: 15fb72f

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@sembrestels sembrestels force-pushed the next branch 8 times, most recently from ce9c936 to ea27383 Compare June 25, 2024 04:37
@sembrestels sembrestels force-pushed the next branch 6 times, most recently from 7cede20 to 590607f Compare June 25, 2024 06:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@PJColombo