-
Notifications
You must be signed in to change notification settings - Fork 9
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
Sweep: Add more query tests #26
Comments
Here's the PR! #27.⚡ Sweep Basic Tier: I'm using GPT-4. You have 5 GPT-4 tickets left for the month and 3 for the day.
For more GPT-4 tickets, visit our payment portal. For a one week free trial, try Sweep Pro (unlimited GPT-4 tickets). Actions (click)
Install Sweep Configs: Pull Request
Step 1: 🔎 SearchingI found the following snippets in your repository. I will now analyze these snippets and come up with a plan. Some code snippets I looked at (click to expand). If some file is missing from here, you can mention the path in the ticket description.blitz-orm/tests/mocks/testSchema.ts Lines 352 to 496 in 2abe786
Lines 1 to 79 in 2abe786
blitz-orm/tests/mocks/testSchema.ts Lines 15 to 201 in 2abe786
Step 2: ⌨️ Coding
Modify tests/unit/queries/query.test.ts with contents:
Modify package.json with contents: Ensure that the "@blitznocode/eslint-config" package is included in the "devDependencies" section of the package.json file. If it's not present, add it with the appropriate version. After modifying the package.json, run "npm install" or "yarn install" to install the package.
Run package.json through the sandbox.
Modify .eslintrc with contents: Check the .eslintrc file to ensure that it's correctly configured to extend from "@blitznocode/eslint-config". The "extends" field in the .eslintrc file should include "@blitznocode/eslint-config". If it's not present, add it.
Run .eslintrc through the sandbox.
Modify tests/mocks/data.tql with contents:
Step 3: 🔁 Code ReviewI have finished reviewing the code for completeness. I did not find errors for 🎉 Latest improvements to Sweep:
💡 To recreate the pull request edit the issue title or description. To tweak the pull request, leave a comment on the pull request. |
Todo:
Try to identify missing tests in the queries and add at least 10 new tests that cover typical DB queries that are not present in the current tests.
If you miss some data in the database for some particular tests (data is here (test/mocks/data.tql ) you can suggest new lines of data as well
Context files:
Files to change:
Checklist
tests/unit/queries/query.test.ts
✓ 73cd014tests/unit/queries/query.test.ts
✗package.json
! No changes made.eslintrc
! No changes madetests/unit/queries/query.test.ts
✗tests/mocks/data.tql
✓ d29d060tests/mocks/data.tql
✓The text was updated successfully, but these errors were encountered: