search

elixir-cloud-aai / tesk-api

GA4GH TES API Service that translates tasks into Kubernetes Batch API calls
Apache License 2.0
7 stars 18 forks source link

Improved IT and adding ignore group membership to build User #14

Open navisatebi opened 4 years ago

navisatebi commented 4 years ago

Refined IT to reflect changes. Consolidated building user to retrieve label selector - logic moved to one single place.

codecov[bot] commented 4 years ago

Codecov Report

Merging #14 into master will increase coverage by 0.16%. The diff coverage is 90.90%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #14      +/-   ##
==========================================
+ Coverage   83.81%   83.97%   +0.16%     
==========================================
  Files          37       37              
  Lines         964      980      +16     
  Branches      101      103       +2     
==========================================
+ Hits          808      823      +15     
- Misses        107      108       +1     
  Partials       49       49
Impacted Files Coverage Δ
.../java/uk/ac/ebi/tsc/tesk/config/security/User.java 64.16% <88.00%> (+1.43%) :arrow_up:
.../tesk/config/security/AuthorisationProperties.java 100.00% <100.00%> (ø)
...tesk/config/security/ElixirPrincipalExtractor.java 96.55% <100.00%> (+0.25%) :arrow_up:
...c/tesk/util/component/KubernetesClientWrapper.java 63.23% <100.00%> (+0.54%) :arrow_up:
...sc/tesk/util/component/TesKubernetesConverter.java 90.37% <0.00%> (+0.74%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 389a559...49aec2f. Read the comment docs.