openclimatefix / pv-site-api

Site specific API for PV forecasting
5 stars 8 forks source link

Add enode link URL endpoint #86

Closed AndrewLester closed 1 year ago

AndrewLester commented 1 year ago

Pull Request

Description

Adds an endpoint for getting a valid Enode link URL for a user.

Fixes #72

Need to use env vars for Enode URLs, but coming in another PR

How Has This Been Tested?

Adds a test for both fake and mock Enode requests.

Checklist:

codecov[bot] commented 1 year ago

Codecov Report

Merging #86 (d26af24) into h4i/enode (1a23431) will increase coverage by 5.04%. The diff coverage is 100.00%.

:exclamation: Current head d26af24 differs from pull request most recent head baa189b. Consider uploading reports for the commit baa189b to get more accurate results

@@              Coverage Diff              @@
##           h4i/enode      #86      +/-   ##
=============================================
+ Coverage      86.93%   91.97%   +5.04%     
=============================================
  Files              9        8       -1     
  Lines            398      324      -74     
=============================================
- Hits             346      298      -48     
+ Misses            52       26      -26     
Impacted Files Coverage Δ
pv_site_api/fake.py 100.00% <100.00%> (ø)
pv_site_api/main.py 90.18% <100.00%> (+0.18%) :arrow_up:

... and 4 files with indirect coverage changes

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more