Skip to content

test(ModelManager): increase time.tick to harden flaky tests#99

Merged
remarkablemark merged 2 commits into
masterfrom
test/unit
May 20, 2026
Merged

test(ModelManager): increase time.tick to harden flaky tests#99
remarkablemark merged 2 commits into
masterfrom
test/unit

Conversation

@remarkablemark
Copy link
Copy Markdown
Member

@remarkablemark remarkablemark commented May 19, 2026

What is the motivation for this pull request?

Split the ModelManager tests from 1 large test file into multiple focused test files for better maintainability and readability.

What is the current behavior?

All ModelManager component tests were in a single large test file (ModelManager.test.tsx), making it difficult to navigate and maintain.

What is the new behavior?

Tests are now organized into separate focused files:

  • ModelManager.test.tsx (main component)
  • ModelDeleteConfirmView.test.tsx
  • ModelDeleteView.test.tsx
  • ModelDownloadView.test.tsx
  • ModelDownloadingView.test.tsx
  • ModelSwitchView.test.tsx
  • ModelCustomDownloadView.test.tsx
  • test-utils.tsx (shared utilities)

Checklist:

@remarkablemark remarkablemark self-assigned this May 19, 2026
@remarkablemark remarkablemark force-pushed the test/unit branch 4 times, most recently from 4f98c49 to fb0dad2 Compare May 20, 2026 00:33
@codecov
Copy link
Copy Markdown

codecov Bot commented May 20, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

Files with missing lines Coverage Δ
src/components/ModelManager/ModelDeleteView.tsx 100.00% <ø> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@remarkablemark remarkablemark enabled auto-merge May 20, 2026 01:56
@remarkablemark remarkablemark merged commit f00844a into master May 20, 2026
16 checks passed
@remarkablemark remarkablemark deleted the test/unit branch May 20, 2026 01:58
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