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

WIP - Rewriting tests to use vitest #2178

Draft
wants to merge 22 commits into
base: master
Choose a base branch
from
Draft

WIP - Rewriting tests to use vitest #2178

wants to merge 22 commits into from

Conversation

alexcottner
Copy link

@alexcottner alexcottner commented Jul 26, 2024

Work in progress. First pass is done! tests at least work in node and unit tests work in the browser.
Need to still do some tidying and figure out what to do about our integration tests that do a lot of non-browser friendly things.

Notes:

  • This will be a staggering number of lines changed. I will try to call out the important bits with comments.
  • I will also try to keep the commits single-focus. First one being getting the first test collection working. Each after that being a new test collection's functionality.
  • Feel free to ignore this until it's out of draft, unless you have some strong opinions on patterns you're already seeing.

Progress:

  • groundwork
  • collection_test.ts
  • index_test.ts
  • integration_test.ts
  • kintobase_test.ts
  • utils_test.ts
  • adapters/base_test.ts
  • adapters/IDB_test.ts
  • http/api_test.ts
  • http/batch_test.ts
  • http/bucket_test.ts
  • http/collection_test.ts
  • http/endpont_test.ts
  • http/http_test.ts
  • http/integration_test.ts
  • http/requests_test.ts
  • Make browser tests work in firefox
  • Figure out what to do about integration tests in browser

Copy link

github-actions bot commented Jul 26, 2024

Size Change: -32 B (-0.07%)

Total Size: 46.5 kB

ℹ️ View Unchanged
Filename Size Change
dist/kinto.js 33.2 kB -32 B (-0.1%)
dist/kinto.min.js 13.2 kB 0 B

compressed-size-action

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant