Skip to content
This repository has been archived by the owner on Jul 12, 2022. It is now read-only.

Fix regen system token #1112

Merged
merged 2 commits into from
Apr 28, 2021
Merged

Fix regen system token #1112

merged 2 commits into from
Apr 28, 2021

Conversation

Leandroqo
Copy link
Contributor

Issue Description

when trying to regenerate the token it does not update the database

Signed-off-by: Leandro Queiroz <leandroqueiroz92@gmail.com>
@Leandroqo Leandroqo added the gate label Apr 27, 2021
@Leandroqo Leandroqo self-assigned this Apr 27, 2021
@codecov
Copy link

codecov bot commented Apr 27, 2021

Codecov Report

Merging #1112 (0cad9e1) into system-token (c556b5b) will increase coverage by 0.12%.
The diff coverage is n/a.

❗ Current head 0cad9e1 differs from pull request most recent head d9fb6cd. Consider uploading reports for the commit d9fb6cd to get more accurate results
Impacted file tree graph

@@                Coverage Diff                 @@
##             system-token    #1112      +/-   ##
==================================================
+ Coverage           77.74%   77.87%   +0.12%     
  Complexity           2172     2172              
==================================================
  Files                1303     1313      +10     
  Lines               18134    18313     +179     
  Branches             1685     1685              
==================================================
+ Hits                14099    14261     +162     
- Misses               3535     3544       +9     
- Partials              500      508       +8     
Flag Coverage Δ Complexity Δ
gate 90.50% <ø> (?) 0.00 <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ Complexity Δ
...ernal/use_case/system_token/revoke_system_token.go 100.00% <0.00%> (ø) 0.00% <0.00%> (?%)
...internal/use_case/system_token/get_system_token.go 100.00% <0.00%> (ø) 0.00% <0.00%> (?%)
...l/use_case/authorization/authorize_system_token.go 76.31% <0.00%> (ø) 0.00% <0.00%> (?%)
...ernal/use_case/system_token/create_system_token.go 94.59% <0.00%> (ø) 0.00% <0.00%> (?%)
gate/internal/domain/page.go 100.00% <0.00%> (ø) 0.00% <0.00%> (?%)
gate/internal/domain/system_token.go 100.00% <0.00%> (ø) 0.00% <0.00%> (?%)
...l/use_case/system_token/regenerate_system_token.go 100.00% <0.00%> (ø) 0.00% <0.00%> (?%)
...use_case/system_token/create_system_token_input.go 100.00% <0.00%> (ø) 0.00% <0.00%> (?%)
...rnal/use_case/system_token/get_all_system_token.go 100.00% <0.00%> (ø) 0.00% <0.00%> (?%)
...nal/use_case/authorization/authorize_user_token.go 81.25% <0.00%> (ø) 0.00% <0.00%> (?%)

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 c556b5b...d9fb6cd. Read the comment docs.

@emiteze emiteze merged commit 77d7713 into system-token Apr 28, 2021
@emiteze emiteze deleted the fix-regen-system-token branch April 28, 2021 19:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants