Skip to content

test: update vitest version & fix typo in url - #64

Open
jumpy-cat wants to merge 1 commit into
ci-tweakfrom
test-fix
Open

test: update vitest version & fix typo in url#64
jumpy-cat wants to merge 1 commit into
ci-tweakfrom
test-fix

Conversation

@jumpy-cat

@jumpy-cat jumpy-cat commented Aug 13, 2026

Copy link
Copy Markdown
Member

Description

Was debugging a cryptic test failure higher up the stack when I noticed the vitest version being used is 3 years old. Bumped that to the most recent stable version, which resulted in an better msg and allowed me to find the root cause: a typo in a url. The test passed historically because parseInt would take only the prefix and ignore the rest but the process of migrating to documented resulted in something more strict.

Type of Change

  • Bug fix (fix)
  • Dependency / build update

Related Issues

none

Changes Made

  • Backend (TypeScript):
    • updated vitest from 1.x version to 4.x version
    • fixed typo in path.test.ts

Testing Done

Flutter: n/a

  • the specific test the caused this higher in the stack now passes

Screenshots / Demo (if UI or notification change)

n/a

Checklist

  • Commit messages follow Conventional Commits
  • PR title follows [type](scope): short description
  • PR target branch is not main and is our current working update branch (e.g. maizebus2.1)
  • No print() / debugPrint() / console.log() left in production code
  • Secrets / keys not committed

Stack created with GitHub Stacks CLIGive Feedback 💬

@jumpy-cat
jumpy-cat marked this pull request as ready for review August 14, 2026 16:06
@jumpy-cat
jumpy-cat marked this pull request as draft August 14, 2026 16:09
@jumpy-cat
jumpy-cat marked this pull request as ready for review August 14, 2026 16:09
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