sixcolors / cs50-final-project / 26919931458

99%

main: 99%

Pull #541

github

Address PR review comments

- Locator.svelte: add response.ok checks in filterLocations and
  submitValue; add isFinite guard on lat/lon before dispatching
- Location.svelte: replace {@html itemLabel} with {itemLabel} to
  eliminate XSS vector from external geocoder display_name values
- Map.svelte: add escapeHtml helper and escape all external values
  in bindPopup HTML; switch WFS outputFormat from csv to
  application/json to avoid fragile split(',') CSV parsing; update
  data parsing to use GeoJSON FeatureCollection features; add
  .catch() to wildfire fetch chain; update stale CSV comment
- tests/locator.test.ts: add ok/status to fetch mock; replace fixed
  setTimeout sleeps with screen.findByText and waitFor
- README.md: remove stale NRCAN Open Data API reference in
  description

16 of 25 branches covered (64.0%)

Branch coverage included in aggregate %.

67 of 67 new or added lines in 4 files covered. (100.0%)

958 of 972 relevant lines covered (98.56%)

0.99 hits per line