Improve unit test logs (#797)

<!-- < < < < < < < < < < < < < < < < < < < < < < < < < < < < < < < < < ☺
v           ✰  Thanks for creating a PR! You're awesome! ✰
v Please note that maintainers will only review those PRs with a
completed PR template.
☺ > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >  -->

## Purpose of Changes and their Description
This PR refactors the logging setup used in unit tests to produce
cleaner, more readable output.

### Before:
<img width="1381" alt="Screenshot 2025-04-10 at 16 01 19"
src="https://github.com/user-attachments/assets/393f7d25-140a-4e1b-8160-fe15534c2d85"
/>

### After:
![Screenshot 2025-04-10 at 13 10
57](https://github.com/user-attachments/assets/8d485047-0225-4012-bd03-a8fee4f65736)

## Are these changes tested and documented?

- [x] If tested, please describe how. If not, why tests are not needed.
- [ ] If documented, please describe where. If not, describe why docs
are not needed.
- [x] Added to `Unreleased` section of `CHANGELOG.md`?

Author: guilherme-brandao

CHANGELOG.md

@@ -103,6 +103,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 * [#775](https://github.com/allora-network/allora-chain/pull/775) Pipe a "y" to key add command in l1_node.sh script
 * [#783](https://github.com/allora-network/allora-chain/pull/783) Add API Breaking Changes section to CHANGELOG.md
 * [#787](https://github.com/allora-network/allora-chain/pull/785) Bounded Dec: Utility functions to cap value to bounds
+* [#797](https://github.com/allora-network/allora-chain/pull/797) Improve unit test logs
 
 ### Changed
 

log/logger.go

@@ -0,0 +1,41 @@
+package log
+
+import (
+	"fmt"
+	"os"
+	"path/filepath"
+	"time"
+
+	"cosmossdk.io/log"
+	"github.com/rs/zerolog"
+)
+
+// nolint: exhaustruct
+func NewTestLogger(t zerolog.TestingLog) log.Logger {
+	lvl := zerolog.DebugLevel
+	cw := zerolog.ConsoleWriter{
+		Out:           os.Stderr,
+		TimeFormat:    time.RFC3339Nano,
+		NoColor:       true,
+		FieldsExclude: []string{"module"},
+	}
+
+	// Clean up formatter: raw message only
+	cw.FormatLevel = func(i interface{}) string { return "" }
+	cw.FormatCaller = func(i interface{}) string {
+		if fullPath, ok := i.(string); ok {
+			if cwd, err := os.Getwd(); err == nil {
+				if relPath, err := filepath.Rel(cwd, fullPath); err == nil {
+					return fmt.Sprintf("[%s]", relPath)
+				}
+			}
+			return fmt.Sprintf("[%s]", fullPath)
+		}
+		return "[no-caller]"
+	}
+	cw.FormatTimestamp = func(i interface{}) string {
+		return fmt.Sprintf("[%s]", i)
+	}
+	zerolog.TimestampFieldName = "time"
+	return log.NewCustomLogger(zerolog.New(cw).With().CallerWithSkipFrameCount(4).Timestamp().Logger().Level(lvl))
+}

x/emissions/keeper/actor_utils/worker_test.go

@@ -9,6 +9,7 @@ import (
 	cosmosMath "cosmossdk.io/math"
 	storetypes "cosmossdk.io/store/types"
 	"github.com/allora-network/allora-chain/app/params"
+	alloralog "github.com/allora-network/allora-chain/log"
 	alloraMath "github.com/allora-network/allora-chain/math"
 	alloratestutil "github.com/allora-network/allora-chain/test/testutil"
 	"github.com/allora-network/allora-chain/x/emissions/keeper"
@@ -57,7 +58,7 @@ func (s *WorkerTestSuite) SetupTest() {
 	storeService := runtime.NewKVStoreService(key)
 	testCtx := testutil.DefaultContextWithDB(s.T(), key, storetypes.NewTransientStoreKey("transient_test"))
 	// Set logger to show logs from the rewards module too
-	logger := log.NewTestLogger(s.T()).With("module", "rewards")
+	logger := alloralog.NewTestLogger(s.T()).With("module", "rewards")
 	ctx := testCtx.Ctx.WithHeaderInfo(header.Info{Time: time.Now()}).WithLogger(logger) // nolint: exhaustruct
 	encCfg := moduletestutil.MakeTestEncodingConfig(auth.AppModuleBasic{}, bank.AppModuleBasic{}, module.AppModule{})
 

x/emissions/keeper/inference_synthesis/network_inference_builder_test.go

@@ -13,6 +13,7 @@ import (
 
 	storetypes "cosmossdk.io/store/types"
 	"github.com/allora-network/allora-chain/app/params"
+	alloralog "github.com/allora-network/allora-chain/log"
 	alloraMath "github.com/allora-network/allora-chain/math"
 	alloratestutil "github.com/allora-network/allora-chain/test/testutil"
 	"github.com/allora-network/allora-chain/x/emissions/keeper"
@@ -59,7 +60,7 @@ func (s *InferenceSynthesisTestSuite) SetupTest() {
 	storeService := runtime.NewKVStoreService(key)
 	testCtx := testutil.DefaultContextWithDB(s.T(), key, storetypes.NewTransientStoreKey("transient_test"))
 	// Set logger to show logs from the rewards module too
-	logger := log.NewTestLogger(s.T()).With("module", "inference_synthesis")
+	logger := alloralog.NewTestLogger(s.T()).With("module", "inference_synthesis")
 	ctx := testCtx.Ctx.WithHeaderInfo(header.Info{
 		Height:  1,
 		Hash:    []byte("1"),

x/emissions/keeper/inference_synthesis/weights_test.go

@@ -8,6 +8,7 @@ import (
 	"cosmossdk.io/log"
 	storetypes "cosmossdk.io/store/types"
 	"github.com/allora-network/allora-chain/app/params"
+	alloralog "github.com/allora-network/allora-chain/log"
 	alloraMath "github.com/allora-network/allora-chain/math"
 	alloratestutil "github.com/allora-network/allora-chain/test/testutil"
 	"github.com/allora-network/allora-chain/x/emissions/keeper"
@@ -56,7 +57,7 @@ func (s *WeightsTestSuite) SetupTest() {
 	storeService := runtime.NewKVStoreService(key)
 	testCtx := testutil.DefaultContextWithDB(s.T(), key, storetypes.NewTransientStoreKey("transient_test"))
 	// Set logger to show logs from the rewards module too
-	logger := log.NewTestLogger(s.T()).With("module", "inference_synthesis")
+	logger := alloralog.NewTestLogger(s.T()).With("module", "inference_synthesis")
 	ctx := testCtx.Ctx.WithHeaderInfo(header.Info{
 		Height:  1,
 		Hash:    []byte("1"),

x/emissions/module/rewards/rewards_test.go

@@ -5,24 +5,25 @@ import (
 	"testing"
 	"time"
 
-	actorutils "github.com/allora-network/allora-chain/x/emissions/keeper/actor_utils"
-	inferencesynthesis "github.com/allora-network/allora-chain/x/emissions/keeper/inference_synthesis"
-
 	"cosmossdk.io/core/header"
 	"cosmossdk.io/log"
 	cosmosMath "cosmossdk.io/math"
 	storetypes "cosmossdk.io/store/types"
 	"github.com/allora-network/allora-chain/app/params"
+	alloralog "github.com/allora-network/allora-chain/log"
 	alloraMath "github.com/allora-network/allora-chain/math"
 	alloratestutil "github.com/allora-network/allora-chain/test/testutil"
 	"github.com/allora-network/allora-chain/x/emissions/keeper"
+	actorutils "github.com/allora-network/allora-chain/x/emissions/keeper/actor_utils"
+	inferencesynthesis "github.com/allora-network/allora-chain/x/emissions/keeper/inference_synthesis"
 	"github.com/allora-network/allora-chain/x/emissions/keeper/msgserver"
 	"github.com/allora-network/allora-chain/x/emissions/module"
 	"github.com/allora-network/allora-chain/x/emissions/module/rewards"
 	"github.com/allora-network/allora-chain/x/emissions/types"
 	mintkeeper "github.com/allora-network/allora-chain/x/mint/keeper"
 	mint "github.com/allora-network/allora-chain/x/mint/module"
 	minttypes "github.com/allora-network/allora-chain/x/mint/types"
+
 	"github.com/cometbft/cometbft/crypto/secp256k1"
 	codecAddress "github.com/cosmos/cosmos-sdk/codec/address"
 	"github.com/cosmos/cosmos-sdk/runtime"
@@ -67,7 +68,8 @@ func (s *RewardsTestSuite) SetupTest() {
 	storeService := runtime.NewKVStoreService(key)
 	testCtx := testutil.DefaultContextWithDB(s.T(), key, storetypes.NewTransientStoreKey("transient_test"))
 	// Set logger to show logs from the rewards module too
-	logger := log.NewTestLogger(s.T()).With("module", "rewards")
+
+	logger := alloralog.NewTestLogger(s.T()).With("module", "rewards")
 	ctx := testCtx.Ctx.WithHeaderInfo(header.Info{Time: time.Now()}).WithLogger(logger) // nolint: exhaustruct
 	encCfg := moduletestutil.MakeTestEncodingConfig(auth.AppModuleBasic{}, bank.AppModuleBasic{}, module.AppModule{})
 
@@ -1118,7 +1120,7 @@ func (s *RewardsTestSuite) TestIncreasingTaskRewardAlphaIncreasesImportanceOfPre
 // We increase alpha between the trials to prove that their worsening performance decreases regret.
 // This is somewhat counterintuitive, but can be explained by the following passage from the litepaper:
 // "A positive regret implies that the inference of worker j is expected by worker k to outperform
-// the network’s previously reported accuracy, whereas a negative regret indicates that the network
+// the network's previously reported accuracy, whereas a negative regret indicates that the network
 // is expected to be more accurate."
 func (s *RewardsTestSuite) TestIncreasingAlphaRegretIncreasesPresentEffectOnRegret() {
 	/// SETUP