Fix test runners so all tests are automatically run #88

Merged

annadavi merged 4 commits from fix-test-runner into ui_change 2026-10-26 02:36:08 +00:00

Conversation 1 Commits 4 Files changed 21 +227 -61

annadavi commented 2026-10-26 00:59:01 +00:00

Member

Copy link

Description

Create separate test/unit and test/integration folders, add automatic test discovery, migrate npm tests to vitest, consolidate test commands to just npm run test:unit and npm run test:integration, updated docs and ci to actually run all tests

Related Issue

Closes #

Type of change

Type: Bug fix

Testing

• I added/updated tests.
• I tested my changes locally.
  • I tested my changes with a laptop-size screen.
  • I tested my changes with a phone-size screen.
  • I tested my changes with a tablet-size screen.
• I tested on the kennel preview deployment for this pull request.

Screenshots/Recordings

Checklist

• My code follows the project's existing style.
• I have reviewed my changes.
• Only the necessary files and lines have been changed.
• This PR is ready for review.

## Description <!-- Please provide a description of the issue, the changes you made to fix it, and why they work. It is expected that you understand why your changes work and if you do not understand why at least say as much so a maintainer knows how much to value the PR. No AI slop. --> Create separate test/unit and test/integration folders, add automatic test discovery, migrate npm tests to vitest, consolidate test commands to just npm run test:unit and npm run test:integration, updated docs and ci to actually run all tests ## Related Issue <!-- Link the issue this PR addresses, if applicable. --> Closes #<!-- Add your issue number here --> ## Type of change **Type:** Bug fix<!-- Bug fix / New feature / Refactor / Test coverage / Documentation --> ## Testing - [X] I added/updated tests. - [ ] I tested my changes locally. - [ ] I tested my changes with a laptop-size screen. - [ ] I tested my changes with a phone-size screen. - [ ] I tested my changes with a tablet-size screen. - [ ] I tested on the kennel preview deployment for this pull request. ## Screenshots/Recordings <!-- If this includes a UI change, attach screenshots on both laptop and mobile. --> ## Checklist - [X] My code follows the project's existing style. - [X] I have reviewed my changes. - [X] Only the necessary files and lines have been changed. - [X] This PR is ready for review.

annadavi added 2 commits 2026-10-26 00:59:01 +00:00

test: create separate test/unit and test/integration folders, add automatic test discovery, migrate npm tests to vitest, consolidate test commands to just npm run test:unit and npm run test:integration, updated docs and ci to actually run all tests c6c59f86df

fix: removed 2 unused variables bccf901098

scottylabs-bot commented 2026-10-26 01:00:35 +00:00

Owner

Copy link

Kennel Deployments

Torn down at 2026-10-26 02:36:11 UTC.

<!-- kennel-deployment --> ### Kennel Deployments Torn down at 2026-10-26 02:36:11 UTC.

annadavi added 1 commit 2026-10-26 02:05:39 +00:00

fix(ci): prefer public Nix binary cache 05bacb52d0

annadavi added 1 commit 2026-10-26 02:23:11 +00:00

fix: try switching workflow from nix to docker 2337df5f08

annadavi force-pushed fix-test-runner from 2337df5f08 to 7de282239f 2026-10-26 02:31:29 +00:00 Compare

annadavi merged commit 7de282239f into ui_change 2026-10-26 02:36:08 +00:00

annadavi deleted branch fix-test-runner 2026-10-26 02:36:08 +00:00

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels   

first-task

  

needs/docs

Must have documentation written

  

needs/research

Discuss the ideal implementation with the tech lead

  

needs/upstream

Needs a contribution to an upstream project

  

type/bug

Something not working as intended

  

type/external

Lives outside of code

  

type/feature

New functionality

  

type/refactor

Improving existing code

  

urgency

high

This is currently causing issues

  

urgency

immediate

This is making a project or feature unusable

  

urgency

low

This can happen whenever

  

urgency

medium

This is expected to cause issues soon

No labels

first-task

needs/docs

needs/research

needs/upstream

type/bug

type/external

type/feature

type/refactor

urgency

high

urgency

immediate

urgency

low

urgency

medium

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

2 participants

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference

ScottyLabs/study!88

No description provided.