Skip to content

chore: vitest testing in the shared workspace - #205

Merged
robsimmons merged 2 commits into
mainfrom
vitest-intro
Oct 31, 2026
Merged

robsimmons merged 2 commits into
mainfrom
vitest-intro

Conversation

@robsimmons

Copy link
Copy Markdown
Collaborator

No description provided.

Comment thread .github/workflows/test.yml Resolved
Comment thread shared/shared.test.ts
Comment on lines +40 to +42
test('bwrapProjectDir', () => {
expect(bwrapProjectDir('abc')).toBe('/workspace/abc/')
})

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
test('bwrapProjectDir', () => {
expect(bwrapProjectDir('abc')).toBe('/workspace/abc/')
})

Maybe this is test bloat?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

More like coverage checking extremism. Switched to v8-ignoring it, which is a more correct move (most correct move is probably not to care so much about coverage but I was close 😄 )

Co-authored-by: Wojciech Nawrocki <13901751+Vtec234@users.noreply.github.com>
@robsimmons
robsimmons merged commit 5251648 into main Oct 31, 2026
1 check passed
robsimmons added a commit that referenced this pull request Nov 1, 2026
I meant to merge this as part of #205 but messed up my commit stack
@robsimmons
robsimmons deleted the vitest-intro branch November 1, 2026 20:45
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.

3 participants