ci: ignore some files for test coverage (#284)

Pantani · lumtis · web-flow · commit 14cf84150dd3 · 2021-09-01T14:38:17.000+02:00
* ignore some files for test coverage

* add more files without test

* remove genesis.go from ignore and increase cli package coverage

* Update GetParams method comment

Co-authored-by: Lucas Bertrand &lt;lucas.bertrand.22@gmail.com&gt;

Co-authored-by: Lucas Bertrand &lt;lucas.bertrand.22@gmail.com&gt;
diff --git a/.codecov.yml b/.codecov.yml
@@ -15,7 +15,7 @@ coverage:
           - modules
       client:
         if_ci_failed: error
-        target: 35%
+        target: 70%
         flags:
           - client
 
@@ -29,21 +29,32 @@ flags:
     carryforward: true
     paths:
       - "x/"
-      - "!x/**/client/"
+      - "!x/**/client/**/*"
   client:
     carryforward: true
     paths:
-      - "x/**/client/"
+      - "x/**/client/**/*"
 
 ignore:
+  - "x/**/client/cli/query.go"
+  - "x/**/client/cli/query_params.go"
+  - "x/**/client/cli/tx.go"
+  - "x/**/client/cli/tx_*.go"
+  - "x/**/keeper/keeper.go"
+  - "x/**/keeper/msg_server.go"
+  - "x/**/keeper/grpc_query_params.go"
   - "x/**/types/codec.go"
   - "x/**/types/errors.go"
   - "x/**/types/keys.go"
+  - "x/**/types/key_*.go"
+  - "x/**/types/types.go"
+  - "x/**/module.go"
+  - "x/**/handler.go"
   - "*.proto"
   - "*.md"
   - "*.yml"
+  - "*.yaml"
   - "*.pb.go"
-  - "x/**/types/key_*.go"
   - ".github/"
   - "app/"
   - "cmd/"
@@ -53,4 +64,5 @@ ignore:
   - "scripts/"
   - "tests/"
   - "testutil/"
+  - "dart/"
   - "vue/"
diff --git a/x/launch/keeper/params.go b/x/launch/keeper/params.go
@@ -17,15 +17,15 @@ func (k Keeper) MaxLaunchTime(ctx sdk.Context) (res uint64) {
 	return
 }
 
-// Get all parameteras as types.Params
+// GetParams get all parameters as types.Params
 func (k Keeper) GetParams(ctx sdk.Context) types.Params {
 	return types.NewParams(
 		k.MinLaunchTime(ctx),
 		k.MaxLaunchTime(ctx),
 	)
 }
 
-// set the params
+// SetParams set the params
 func (k Keeper) SetParams(ctx sdk.Context, params types.Params) {
 	k.paramstore.SetParamSet(ctx, &params)
 }
diff --git a/x/launch/types/params.go b/x/launch/types/params.go
@@ -21,7 +21,7 @@ var (
 
 var _ paramtypes.ParamSet = (*Params)(nil)
 
-// ParamTable for launch module
+// ParamKeyTable the param key table for launch module
 func ParamKeyTable() paramtypes.KeyTable {
 	return paramtypes.NewKeyTable().RegisterParamSet(&Params{})
 }
@@ -42,7 +42,7 @@ func DefaultParams() Params {
 	)
 }
 
-// Implements params.ParamSet
+// ParamSetPairs get the params.ParamSet
 func (p *Params) ParamSetPairs() paramtypes.ParamSetPairs {
 	return paramtypes.ParamSetPairs{
 		paramtypes.NewParamSetPair(KeyMinLaunchTime, &p.MinLaunchTime, validateLaunchTime),

Original file line number	Diff line number	Diff line change
`@@ -17,15 +17,15 @@ func (k Keeper) MaxLaunchTime(ctx sdk.Context) (res uint64) {`
`17`	`17`	`return`
`18`	`18`	`}`
`19`	`19`
`20`		`-// Get all parameteras as types.Params`
	`20`	`+// GetParams get all parameters as types.Params`
`21`	`21`	`func (k Keeper) GetParams(ctx sdk.Context) types.Params {`
`22`	`22`	`return types.NewParams(`
`23`	`23`	`k.MinLaunchTime(ctx),`
`24`	`24`	`k.MaxLaunchTime(ctx),`
`25`	`25`	`)`
`26`	`26`	`}`
`27`	`27`
`28`		`-// set the params`
	`28`	`+// SetParams set the params`
`29`	`29`	`func (k Keeper) SetParams(ctx sdk.Context, params types.Params) {`
`30`	`30`	`k.paramstore.SetParamSet(ctx, &params)`
`31`	`31`	`}`
Original file line number	Diff line number	Diff line change
`@@ -21,7 +21,7 @@ var (`
`21`	`21`
`22`	`22`	`var _ paramtypes.ParamSet = (*Params)(nil)`
`23`	`23`
`24`		`-// ParamTable for launch module`
	`24`	`+// ParamKeyTable the param key table for launch module`
`25`	`25`	`func ParamKeyTable() paramtypes.KeyTable {`
`26`	`26`	`return paramtypes.NewKeyTable().RegisterParamSet(&Params{})`
`27`	`27`	`}`
`@@ -42,7 +42,7 @@ func DefaultParams() Params {`
`42`	`42`	`)`
`43`	`43`	`}`
`44`	`44`
`45`		`-// Implements params.ParamSet`
	`45`	`+// ParamSetPairs get the params.ParamSet`
`46`	`46`	`func (p *Params) ParamSetPairs() paramtypes.ParamSetPairs {`
`47`	`47`	`return paramtypes.ParamSetPairs{`
`48`	`48`	`paramtypes.NewParamSetPair(KeyMinLaunchTime, &p.MinLaunchTime, validateLaunchTime),`