Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add ut cases for command gen dao #2958

Merged
merged 10 commits into from
Sep 12, 2023
Merged

add ut cases for command gen dao #2958

merged 10 commits into from
Sep 12, 2023

Conversation

gqcn
Copy link
Member

@gqcn gqcn commented Sep 12, 2023

No description provided.

@codecov-commenter
Copy link

codecov-commenter commented Sep 12, 2023

Codecov Report

Patch coverage: 66.55% and project coverage change: -0.09% ⚠️

Comparison is base (a5a7d23) 79.18% compared to head (0c1ad96) 79.09%.

❗ Current head 0c1ad96 differs from pull request most recent head 5192a0a. Consider uploading reports for the commit 5192a0a to get more accurate results

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

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2958      +/-   ##
==========================================
- Coverage   79.18%   79.09%   -0.09%     
==========================================
  Files         642      643       +1     
  Lines       52597    52744     +147     
==========================================
+ Hits        41648    41718      +70     
- Misses       8876     8944      +68     
- Partials     2073     2082       +9     
Flag Coverage Δ
go-1.18-386 79.19% <66.55%> (+<0.01%) ⬆️
go-1.18-amd64 79.15% <66.55%> (-0.05%) ⬇️
go-1.19-386 79.02% <66.55%> (-0.09%) ⬇️
go-1.19-amd64 79.01% <66.55%> (-0.09%) ⬇️
go-1.20-386 78.52% <66.55%> (-0.74%) ⬇️
go-1.20-amd64 79.16% <66.55%> (-0.08%) ⬇️

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

Files Changed Coverage Δ
os/gfile/gfile.go 83.62% <ø> (ø)
os/gstructs/gstructs_type.go 100.00% <ø> (ø)
os/gfile/gfile_copy.go 42.19% <36.95%> (-3.07%) ⬇️
util/gutil/gutil_struct.go 55.55% <38.46%> (-44.45%) ⬇️
os/gstructs/gstructs_field.go 89.58% <50.00%> (-2.09%) ⬇️
os/gstructs/gstructs.go 93.47% <93.47%> (ø)
os/gstructs/gstructs_tag.go 77.85% <100.00%> (ø)
util/gconv/gconv_convert.go 92.80% <100.00%> (+0.37%) ⬆️

... and 2 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

os/gstructs/gstructs.go Outdated Show resolved Hide resolved
@gqcn gqcn merged commit 5219c5c into master Sep 12, 2023
21 checks passed
@gqcn gqcn deleted the feature/cli-ut branch September 12, 2023 14:00
@@ -86,7 +171,8 @@ func CopyFile(src, dst string) (err error) {
// CopyDir recursively copies a directory tree, attempting to preserve permissions.
//
// Note that, the Source directory must exist and symlinks are ignored and skipped.
func CopyDir(src string, dst string) (err error) {
func CopyDir(src string, dst string, option ...CopyOption) (err error) {
var usedOption = getCopyOption(option...)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这里是不是可以滞后处理呢?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants