Skip to content

Improve using aliases for wekeo-cds source and update docs and tests - #1104

Open
sandorkertesz wants to merge 2 commits into
developfrom
feature/wekeo-cds-source-alias
Open

Improve using aliases for wekeo-cds source and update docs and tests#1104
sandorkertesz wants to merge 2 commits into
developfrom
feature/wekeo-cds-source-alias

Conversation

@sandorkertesz

@sandorkertesz sandorkertesz commented Aug 17, 2026

Copy link
Copy Markdown
Collaborator

Description

Makes the wekeocds source to deprecation explicit instead of inferred from the alias table.

This involves the following changes in ClassMaker:

  • Alias inverted: ALIASES was {"wekeo-cds": "wekeocds"}, now {"wekeocds": "wekeo-cds"} — the dashed name is canonical and SOURCES is keyed by it.
  • New DEPRECATED set: deprecation. A source can now be deprecated without being an alias.
  1. Improved source alias testing
  • These tests now rely the newly added SourceMaker._lookup() method to avoid using/modifying SourceMaker class variables (i.e. global resources)
  1. Updated docs

Contributor Declaration

By opening this pull request, I affirm the following:

  • All authors agree to the Contributor License Agreement.
  • The code follows the project's coding standards.
  • I have performed self-review and added comments where needed.
  • I have added or updated tests to verify that my changes are effective and functional.
  • I have run all existing tests and confirmed they pass.

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.27%. Comparing base (03497a1) to head (25f9bb8).
⚠️ Report is 1 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #1104      +/-   ##
===========================================
- Coverage    82.27%   82.27%   -0.01%     
===========================================
  Files          239      239              
  Lines        17025    17018       -7     
  Branches       823      823              
===========================================
- Hits         14007    14001       -6     
+ Misses        2771     2770       -1     
  Partials       247      247              

☔ View full report in Codecov by Harness.
📢 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.

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.

2 participants