graham/prop - Forgejo: Beyond coding. We forge.

graham/prop

Fork 0

Commit graph

Author	SHA1	Message	Date
Graham McIntire	079346a1b9	fix: resolve all 281 credo issues across source and test files Some checks failed Build and Push / Build and Push Docker Image (push) Failing after 4m38s Details - F-level (228): replace length/1 with Enum.count_until/2 or pattern matching; convert Enum.flat_map+if to Enum.filter+Enum.map; fix identity case in narr_client - W-level (46): normalize dual atom/string key access in weather_layers, beacon_measurements, surface, skewt_live, contact_live/show; add :data_provider and weather-map assigns to ignored_assigns in credo config (consumed by child components credo can't trace); remove weak is_list assertion; remove explicit assert_receive timeout - R-level (6): replace 'This module provides...' moduledocs with meaningful descriptions - Also fix 4 compile-connected xref issues by deferring BandConfig.band_options() from module attribute to runtime Co-Authored-By: Claude <noreply@anthropic.com>	2026-07-29 09:04:21 -05:00
Graham McIntire	f82dfb4d00	fix(test): GridCache Valkey fetch_point bug + data-path tests (GridCache 49%→84%) - Fix production bug in valkey_fetch_point: Map.get(chunk_map, {lat, lon}) was looking up lat/lon in the outer map keyed by chunk tuple, not the inner cells map. Added chunk_key unwrap before lat/lon lookup. - DataStubAdapter returns real chunk data for fetch/fetch_bounds/fetch_point - Tests cover: fetch with decoded rows, fetch_bounds filtering, fetch_point with chunk unwrap, put with large row sets, claim_fill/release_fill Coverage: 79.30% → 79.43% (GridCache 74% → 84%)	2026-05-07 16:15:02 -05:00
Graham McIntire	5d90d52a45	test: format all test files + GridCache Valkey backend (49%->74%) Coverage: 79.09% -> 79.30% (GridCache 49% -> 74%)	2026-05-07 16:04:32 -05:00

Author

SHA1

Message

Date

Graham McIntire

079346a1b9

fix: resolve all 281 credo issues across source and test files

Build and Push / Build and Push Docker Image (push) Failing after 4m38s

Details

- F-level (228): replace length/1 with Enum.count_until/2 or pattern
  matching; convert Enum.flat_map+if to Enum.filter+Enum.map; fix
  identity case in narr_client
- W-level (46): normalize dual atom/string key access in weather_layers,
  beacon_measurements, surface, skewt_live, contact_live/show; add
  :data_provider and weather-map assigns to ignored_assigns in credo
  config (consumed by child components credo can't trace); remove
  weak is_list assertion; remove explicit assert_receive timeout
- R-level (6): replace 'This module provides...' moduledocs with
  meaningful descriptions
- Also fix 4 compile-connected xref issues by deferring
  BandConfig.band_options() from module attribute to runtime

Co-Authored-By: Claude <noreply@anthropic.com>

2026-07-29 09:04:21 -05:00

Graham McIntire

f82dfb4d00

fix(test): GridCache Valkey fetch_point bug + data-path tests (GridCache 49%→84%)

- Fix production bug in valkey_fetch_point: Map.get(chunk_map, {lat, lon})
  was looking up lat/lon in the outer map keyed by chunk tuple, not the
  inner cells map. Added chunk_key unwrap before lat/lon lookup.
- DataStubAdapter returns real chunk data for fetch/fetch_bounds/fetch_point
- Tests cover: fetch with decoded rows, fetch_bounds filtering, fetch_point
  with chunk unwrap, put with large row sets, claim_fill/release_fill

Coverage: 79.30% → 79.43% (GridCache 74% → 84%)

2026-05-07 16:15:02 -05:00

Graham McIntire

5d90d52a45

test: format all test files + GridCache Valkey backend (49%->74%)

Coverage: 79.09% -> 79.30% (GridCache 49% -> 74%)

2026-05-07 16:04:32 -05:00

3 commits