Skip to content
This repository has been archived by the owner on Oct 3, 2024. It is now read-only.

VEN and Resource API #112

Merged
merged 14 commits into from
Sep 24, 2024
Merged

VEN and Resource API #112

merged 14 commits into from
Sep 24, 2024

Conversation

marlonbaeten
Copy link
Member

See #106

Copy link

codecov bot commented Sep 17, 2024

Codecov Report

Attention: Patch coverage is 55.20111% with 323 lines in your changes missing coverage. Please review.

Project coverage is 70.41%. Comparing base (5cd85b3) to head (eea94f1).
Report is 13 commits behind head on main.

Files with missing lines Patch % Lines
openadr-vtn/src/data_source/postgres/resource.rs 23.39% 167 Missing ⚠️
openadr-vtn/src/api/resource.rs 0.00% 75 Missing ⚠️
openadr-vtn/src/api/ven.rs 51.68% 43 Missing ⚠️
openadr-vtn/src/data_source/postgres/ven.rs 93.67% 16 Missing ⚠️
openadr-vtn/src/jwt.rs 65.38% 9 Missing ⚠️
openadr-vtn/src/data_source/postgres/report.rs 0.00% 4 Missing ⚠️
openadr-vtn/src/data_source/mod.rs 80.00% 3 Missing ⚠️
openadr-vtn/src/data_source/postgres/mod.rs 50.00% 3 Missing ⚠️
openadr-vtn/src/state.rs 86.36% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #112      +/-   ##
==========================================
- Coverage   72.85%   70.41%   -2.44%     
==========================================
  Files          34       39       +5     
  Lines        4140     4841     +701     
==========================================
+ Hits         3016     3409     +393     
- Misses       1124     1432     +308     

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

@marlonbaeten marlonbaeten marked this pull request as ready for review September 24, 2024 12:00
@pohlm01 pohlm01 self-requested a review September 24, 2024 12:49
@pohlm01 pohlm01 merged commit 3f67f84 into main Sep 24, 2024
10 of 11 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants