Skip to content

Conversation

@nlebovits
Copy link
Contributor

@nlebovits nlebovits commented Oct 4, 2025

Related Issue(s):

Description:
A small PR to get conformance.py to 100% coverage.

  • Add tests for get_by_name() method including error cases and valid lookups
  • Add tests for valid_uri property that generates STAC API conformance URIs
  • Add tests for pattern property that creates regex patterns for URI matching

PR Checklist:

  • Code is formatted
  • Tests pass
  • Changes are added to CHANGELOG.md

@codecov-commenter
Copy link

codecov-commenter commented Oct 4, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.74%. Comparing base (21435b0) to head (5360f26).
⚠️ Report is 150 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #834      +/-   ##
==========================================
+ Coverage   93.43%   94.74%   +1.30%     
==========================================
  Files          13       15       +2     
  Lines         990     1217     +227     
==========================================
+ Hits          925     1153     +228     
+ Misses         65       64       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

- Add tests for get_by_name() method including error cases and valid lookups
- Add tests for valid_uri property that generates STAC API conformance URIs
- Add tests for pattern property that creates regex patterns for URI matching
- Achieve 100% test coverage for conformance.py module
@nlebovits nlebovits force-pushed the lebovits/590-conformance-tests branch from fe6550b to 5ac25da Compare October 4, 2025 18:07
@gadomski gadomski requested a review from jsignell October 6, 2025 12:48
Copy link
Member

@jsignell jsignell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Thank you for these PRs!

@jsignell jsignell enabled auto-merge (squash) October 6, 2025 20:37
@jsignell jsignell merged commit 107fe2b into stac-utils:main Oct 6, 2025
14 checks passed
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.

3 participants