fabric8-services / fabric8-auth

Identity and Access Management for fabric8 services
https://auth.openshift.io/api/status
Apache License 2.0
14 stars 26 forks source link

Delete user on Che #813

Closed xcoulon closed 5 years ago

xcoulon commented 5 years ago

Call Che to delete user account, but do not delete tenants if something wrong happened on Che side.

Fixes #ODC140

codecov[bot] commented 5 years ago

Codecov Report

Merging #813 into master will decrease coverage by 0.08%. The diff coverage is 58.1%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #813      +/-   ##
==========================================
- Coverage   78.51%   78.43%   -0.09%     
==========================================
  Files          92       93       +1     
  Lines        8966     8995      +29     
==========================================
+ Hits         7040     7055      +15     
- Misses       1413     1420       +7     
- Partials      513      520       +7
Impacted Files Coverage Δ
authorization/token/service/token_service.go 71.84% <26.31%> (-0.9%) :arrow_down:
authentication/account/service/user.go 76.38% <33.33%> (-0.66%) :arrow_down:
authorization/token/manager/token_manager.go 81.9% <66.66%> (-0.26%) :arrow_down:
che/service/che_service.go 69.69% <69.69%> (ø)
configuration/configuration.go 83.65% <85.71%> (+0.03%) :arrow_up:
...rovider/service/authentication_provider_service.go 72.22% <0%> (ø) :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 b3aa93c...c04ef06. Read the comment docs.

xcoulon commented 5 years ago

[test]

xcoulon commented 5 years ago

[test]