ci: fix coverage report publishing

.github/workflows/integration-tests.yml

@@ -2,19 +2,20 @@ name: Integration tests
 
 on:
   workflow_dispatch: # Runs on manual calls
-  schedule: 
+  schedule:
     - cron: "0 0 * * *" # Runs automatically every day
   pull_request:
     # paths makes the action run only when the given paths are changed
     paths: ["**.go", "**.proto", "go.mod", "go.sum"]
 
-# Allow concurrent runs on main/release branches but isolates other branches 
+# Allow concurrent runs on main/release branches but isolates other branches
 concurrency:
   group: ${{ github.workflow }}-${{ github.ref }}-${{ github.head_ref }}
   cancel-in-progress: ${{ ! (github.ref == 'refs/heads/master' || startsWith(github.ref, 'refs/heads/release/')) }}
 
 jobs:
   integration-tests:
+    # Job that runs all tests and publishes code coverage reports.
     runs-on: ubuntu-latest
     timeout-minutes: 30
     steps:

contrib/make/test.mk

@@ -1,20 +1,21 @@
-###############################################################################
-###                            		  Tests 					                        ###
-###############################################################################
+#########################################################################
+# Tests
+#########################################################################
 
 PACKAGES_NOSIMULATION = ${shell go list ./... | grep -v simapp}
 
-# Used for CI by Codecov
 .PHONY: test-coverage
 test-coverage:
-	go test ./... -v $(PACKAGES_NOSIMULATION) -short \
+	go test ./... $(PACKAGES_NOSIMULATION) -short \
 		-coverprofile=coverage.txt \
 		-covermode=atomic \
 		-race | grep -v "no test" | grep -v "no statement"
 
+# NOTE: Using the verbose flag breaks the coverage reporting in CI.
+# Used for CI by Codecov
 .PHONY: test-coverage-integration
 test-coverage-integration:
-	go test ./... -v $(PACKAGES_NOSIMULATION) \
+	go test ./... \
 		-coverprofile=coverage.txt \
 		-covermode=atomic \
 		-race | grep -v "no test" | grep -v "no statement"

x/perp/v2/keeper/clearing_house.go

@@ -557,8 +557,8 @@
 	// update user volume
 	dnrEpoch, err := k.DnREpoch.Get(ctx)
 	if err != nil {
 		return err
 	}
 	k.IncreaseTraderVolume(ctx, dnrEpoch, traderAddr, positionResp.ExchangedNotionalValue.Abs().TruncateInt())
 
 	transferredFee, err := k.transferFee(ctx, market.Pair, traderAddr, positionResp.ExchangedNotionalValue,
@@ -838,7 +838,6 @@
 	pair asset.Pair,
 	traderAddr sdk.AccAddress,
 	sizeAmt sdk.Dec, // unsigned
-
 ) (*types.PositionResp, error) {
 	market, err := k.GetMarket(ctx, pair)
 	if err != nil {