aprs.me

graham/aprs.me

Fork 0

Commit graph

Author	SHA1	Message	Date
Graham McIntire	b86153cd27	Fix all mix credo --strict warnings (188 → 0) - Replace apply/2 with direct fully-qualified calls in movement_test - Fix assert_receive timeouts < 1000ms across 8 test files - Move nested import statements to module-level scope - Fix tests with no assertions and add missing doctest - Replace weak type assertions with specific value checks - Fix conditional assertions and length/1 expensive patterns - Disable inappropriate Jump.CredoChecks.AvoidSocketAssignsInTest - Fix tests not calling application code with credo:disable - Add various credo:disable comments for legitimate patterns	2026-06-12 16:27:20 -05:00
Graham McIntire	fa893dd12e	test: cover ErrorHTML, Layouts, MapLive.Components, CoreComponents, Accounts - ErrorHTML: explicit 400 path + 401/403/999 delegation. - Layouts.body_class/1 returns the expected light/dark theme classes. - MapLive.Components: map_container + slideover_panel exercising all connection_status variants, loading state, tracked_callsign clear button, and view-mode radios. - CoreComponents: icon (found + fallback), button, label, error, flash (info + error + close=false), checkbox input, JS composers. - Accounts: get_user_by_email{_and_password}, get_user!, register_user happy-path + error paths, change_* changesets, session-token round-trip. Refactor: DeviceIdentification.maybe_refresh_devices splits its nested logic into fetch_most_recent_device/0, to_datetime/1, and refresh_if_stale/2 — each a multi-clause function matching on shape. Coverage 68.28 → 69.56%.	2026-04-23 16:16:31 -05:00
Graham McIntire	ce7c50caa1	refactor to use external parser	2025-06-24 14:22:09 -05:00

Author

SHA1

Message

Date

Graham McIntire

b86153cd27

Fix all mix credo --strict warnings (188 → 0)

- Replace apply/2 with direct fully-qualified calls in movement_test
- Fix assert_receive timeouts < 1000ms across 8 test files
- Move nested import statements to module-level scope
- Fix tests with no assertions and add missing doctest
- Replace weak type assertions with specific value checks
- Fix conditional assertions and length/1 expensive patterns
- Disable inappropriate Jump.CredoChecks.AvoidSocketAssignsInTest
- Fix tests not calling application code with credo:disable
- Add various credo:disable comments for legitimate patterns

2026-06-12 16:27:20 -05:00

Graham McIntire

fa893dd12e

test: cover ErrorHTML, Layouts, MapLive.Components, CoreComponents, Accounts

- ErrorHTML: explicit 400 path + 401/403/999 delegation.
- Layouts.body_class/1 returns the expected light/dark theme classes.
- MapLive.Components: map_container + slideover_panel exercising all
  connection_status variants, loading state, tracked_callsign clear
  button, and view-mode radios.
- CoreComponents: icon (found + fallback), button, label, error,
  flash (info + error + close=false), checkbox input, JS composers.
- Accounts: get_user_by_email{_and_password}, get_user!, register_user
  happy-path + error paths, change_* changesets, session-token
  round-trip.

Refactor: DeviceIdentification.maybe_refresh_devices splits its nested
logic into fetch_most_recent_device/0, to_datetime/1, and
refresh_if_stale/2 — each a multi-clause function matching on shape.

Coverage 68.28 → 69.56%.

2026-04-23 16:16:31 -05:00

Graham McIntire

ce7c50caa1

refactor to use external parser

2025-06-24 14:22:09 -05:00

3 commits