-
Notifications
You must be signed in to change notification settings - Fork 0
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.
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
feat: Add e2e testing #34
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need a Github action for this.
@@ -0,0 +1,7 @@ | |||
DB_NAME=squid |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could we add .env to gitignore
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
.env
is used for interpolation in docker-compose.yml
so it is needed. Same as ./envs
files are needed inside containers.
import { Transfer, Deposit, Execution, TransferStatus } from "../../src/model"; | ||
import { Network, ResourceType } from "@buildwithsygma/core"; | ||
|
||
const NUMBER_OF_TRANSFERS = 31; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is missmatch between NUMBER_OF_TRANSFERS and NUMBER_OF_SUBSTRATE_DEPOSITS + NUMBER_OF_FUNGIBLE_DEPOSITS + NUMBER_OF_PERMISSIONLESS_DEPOSITS + NUMBER_OF_NFT_DEPOSITS
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Substrate deposit is both a "Substrate deposit" and a "fungible deposit" so it shouldn't be counted twice in that equation.
Added e2e tests for the whole indexing functionality. Also set up the testing environment in
docker-compose.yml
.closes #10