Skip to content

Reach 100% test coverage with 96% CI guardrail#148

Merged
coolsam726 merged 2 commits into
mainfrom
feature/coverage-96
Jun 13, 2026
Merged

Reach 100% test coverage with 96% CI guardrail#148
coolsam726 merged 2 commits into
mainfrom
feature/coverage-96

Conversation

@coolsam726

@coolsam726 coolsam726 commented Jun 13, 2026

Copy link
Copy Markdown
Owner

Summary

  • Expand the Pest suite with focused coverage for Flatpickr, FlatpickrServiceProvider, hydration/dehydration edge cases, validation, themes, commands, and the facade.
  • Add test doubles (StatefulFlatpickr, ParseThrowingFlatpickr) so hydration and exception paths can be exercised without a mounted Filament form container.
  • Add minimal package fixtures (stubs/flatpickr-field.stub, database/migrations/) so service-provider boot paths are covered.
  • Enforce a 96% coverage floor via pest --coverage --min=96 in CI and the composer test-coverage script.

Coverage

Area Before After
Total 58.1% 100%
Forms/Components/Flatpickr 57.9% 100%
FlatpickrServiceProvider 62.3% 100%

Notable test additions

  • Hydration: single/range/multiple, native vs non-native, time-only and datetime paths
  • Dehydration: native formatting, brute-force range splitting, parse exceptions
  • Validation: array/null states for range and multiple pickers
  • Service provider: install force-publish, stub publishing, migration registration
  • Attributes, view helpers, theme assets, console commands

Test plan

  • vendor/bin/pest --coverage --min=96 passes locally (114 tests)
  • vendor/bin/pint --test passes
  • CI matrix passes with the new coverage gate

Add targeted tests for hydration, attributes, native dehydration, themes, commands, and the facade, then gate pest --coverage at 96% locally and in the test workflow.
Add edge-case tests for hydration, dehydration, validation, and range splitting; add package stubs and migration directory fixtures; cover install force-publish and stub publishing paths.
@coolsam726 coolsam726 changed the title Raise coverage to 96% and enforce in CI Reach 100% test coverage with 96% CI guardrail Jun 13, 2026
@coolsam726 coolsam726 merged commit 34aa5b3 into main Jun 13, 2026
46 checks passed
@coolsam726 coolsam726 deleted the feature/coverage-96 branch June 13, 2026 22:27
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