flatironinstitute / dendro-old

Analyze neuroscience data in the cloud
https://flatironinstitute.github.io/dendro-docs/
Apache License 2.0
19 stars 2 forks source link

raise on not authenticated #68

Closed magland closed 12 months ago

magland commented 12 months ago

This PR implements the raise_on_not_authenticated option in _authenticate_gui_request

codecov-commenter commented 12 months ago

Codecov Report

Attention: 5 lines in your changes are missing coverage. Please review.

Comparison is base (97a0244) 80.82% compared to head (a4ef34b) 80.79%. Report is 5 commits behind head on main.

Files Patch % Lines
...i_helpers/routers/gui/_authenticate_gui_request.py 86.95% 3 Missing :warning:
...thon/dendro/api_helpers/routers/gui/file_routes.py 66.66% 1 Missing :warning:
python/dendro/client/_interim/NwbSorting.py 0.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #68 +/- ## ========================================== - Coverage 80.82% 80.79% -0.03% ========================================== Files 62 62 Lines 3577 3593 +16 ========================================== + Hits 2891 2903 +12 - Misses 686 690 +4 ``` | [Flag](https://app.codecov.io/gh/flatironinstitute/dendro/pull/68/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=flatironinstitute) | Coverage Δ | | |---|---|---| | [unittests](https://app.codecov.io/gh/flatironinstitute/dendro/pull/68/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=flatironinstitute) | `80.79% <84.84%> (-0.03%)` | :arrow_down: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=flatironinstitute#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.