kbase / dts

A data transfer service
https://kbase.github.io/dts/
MIT License
0 stars 0 forks source link

Implemented preliminary KBase ORCID -> local user federation #37

Closed jeff-cohere closed 9 months ago

jeff-cohere commented 9 months ago

This PR adds a mechanism for mapping ORCID IDs to local KBase users for our end-to-end demo. This is accomplished by assuming the existence of a kbase_users.json file in the current working directory for the DTS. The JSON file contains a single object whose fields are ORCIDs with string values containing associated local usernames. There's also a bit of logging cleanup.

This should be merged after #32.

codecov[bot] commented 9 months ago

Codecov Report

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

Comparison is base (b56fb34) 72.84% compared to head (5196d82) 72.71%. Report is 1 commits behind head on main.

Files Patch % Lines
databases/jdp/database.go 50.00% 7 Missing and 1 partial :warning:
core/task_manager.go 25.00% 2 Missing and 1 partial :warning:
databases/jdp/jamo.go 50.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #37 +/- ## ========================================== - Coverage 72.84% 72.71% -0.13% ========================================== Files 13 13 Lines 1609 1616 +7 ========================================== + Hits 1172 1175 +3 - Misses 361 364 +3 - Partials 76 77 +1 ```

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