Skip to content

Reland "Support subscription crud api"#389

Merged
pohlm01 merged 1 commit into
mainfrom
subscription_crud
Mar 13, 2026
Merged

Reland "Support subscription crud api"#389
pohlm01 merged 1 commit into
mainfrom
subscription_crud

Conversation

@bjorn3

@bjorn3 bjorn3 commented Mar 13, 2026

Copy link
Copy Markdown
Collaborator

This relands #386 which was temporarily reverted for the 0.2.0 release.

* Run cargo sqlx prepare

Signed-off-by: Björn Baron <bjorn@tweedegolf.com>

* Misc tweaks to the subscription wire definitions

Signed-off-by: Björn Baron <bjorn@tweedegolf.com>

* Support subscription crud api

Signed-off-by: Björn Baron <bjorn@tweedegolf.com>

* Add test for empty objectOperations in Subscription

Signed-off-by: Stefan van Ieperen <stefan@tweedegolf.com>
Signed-off-by: Björn Baron <bjorn@tweedegolf.com>

* Add clientName query param

Signed-off-by: Björn Baron <bjorn@tweedegolf.com>

* Support filtering subscriptions by target object type

Signed-off-by: Björn Baron <bjorn@tweedegolf.com>

* Support filtering subscriptions by program id

Signed-off-by: Björn Baron <bjorn@tweedegolf.com>

* Add MockSubscriptionSource to fix MockDataSource

Signed-off-by: Björn Baron <bjorn@tweedegolf.com>

* Review comments

Signed-off-by: Björn Baron <bjorn@tweedegolf.com>

* Add a test for an unspecified edge case

Signed-off-by: Björn Baron <bjorn@tweedegolf.com>

---------

Signed-off-by: Björn Baron <bjorn@tweedegolf.com>
Signed-off-by: Stefan van Ieperen <stefan@tweedegolf.com>
Co-authored-by: Stefan van Ieperen <stefan@tweedegolf.com>
@bjorn3 bjorn3 requested a review from pohlm01 March 13, 2026 13:33
@codecov

codecov Bot commented Mar 13, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 55.65217% with 102 lines in your changes missing coverage. Please review.
✅ Project coverage is 79.75%. Comparing base (547144c) to head (0a399fe).
⚠️ Report is 3 commits behind head on main.

Files with missing lines Patch % Lines
openleadr-vtn/src/api/subscription.rs 48.17% 71 Missing ⚠️
...leadr-vtn/src/data_source/postgres/subscription.rs 75.00% 11 Missing ⚠️
openleadr-wire/src/subscription.rs 0.00% 9 Missing ⚠️
openleadr-vtn/src/state.rs 71.42% 8 Missing ⚠️
openleadr-wire/src/lib.rs 66.66% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #389      +/-   ##
==========================================
- Coverage   80.57%   79.75%   -0.83%     
==========================================
  Files          45       47       +2     
  Lines        5242     5472     +230     
==========================================
+ Hits         4224     4364     +140     
- Misses       1018     1108      +90     

☔ 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.

@pohlm01 pohlm01 added this pull request to the merge queue Mar 13, 2026
Merged via the queue into main with commit cc85b33 Mar 13, 2026
20 checks passed
@pohlm01 pohlm01 deleted the subscription_crud branch March 13, 2026 14:04
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