kratos icon indicating copy to clipboard operation
kratos copied to clipboard

fix: remove duplicate test codes.

Open kvii opened this issue 2 years ago • 1 comments

Description (what this PR does / why we need it):

有两个 test case 重了,我给删除了。再就是 test 失败打印日志的时候不需要再运行一遍函数。

Which issue(s) this PR fixes (resolves / be part of):

Other special notes for the reviewers:

kvii avatar Feb 03 '24 09:02 kvii

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 84.72%. Comparing base (f566bdc) to head (40f2c0d).

:exclamation: Current head 40f2c0d differs from pull request most recent head bcb8006. Consider uploading reports for the commit bcb8006 to get more accurate results

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3187      +/-   ##
==========================================
+ Coverage   84.62%   84.72%   +0.10%     
==========================================
  Files          88       88              
  Lines        3993     3993              
==========================================
+ Hits         3379     3383       +4     
+ Misses        440      438       -2     
+ Partials      174      172       -2     

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

codecov-commenter avatar Feb 03 '24 09:02 codecov-commenter

need to resolve the code conflict

demoManito avatar Mar 12 '24 12:03 demoManito

@demoManito Conflict solved.

kvii avatar Mar 13 '24 00:03 kvii