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.

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

Update README.md for running single Indexer test file #1562

Merged
Merged
Changes from all commits
Commits
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: 2 additions & 0 deletions indexer/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -107,6 +107,8 @@ pnpm run test:all

If you change any logic, you'll have to re-build the services and packages before running unit tests.

### To run a single test file:
`cd services/{service_name} && pnpm build && pnpm test -- {test_name}`
Comment on lines +110 to +111
Copy link
Contributor

Choose a reason for hiding this comment

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

Update the instructions for running a single test file to include more specific details.

- `cd services/{service_name} && pnpm build && pnpm test -- {test_name}`
+ `cd services/{service_name} && pnpm run build && pnpm run test -- {test_name}`

This change ensures that the commands are consistent with other parts of the document where pnpm run is used for executing scripts.


Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation.

Suggested change
### To run a single test file:
`cd services/{service_name} && pnpm build && pnpm test -- {test_name}`
### To run a single test file:
`cd services/{service_name} && pnpm run build && pnpm run test -- {test_name}`


# Running Dockerfile locally
TODO(DEC-671): Add e2e tests
Expand Down
Loading