fix: code style (#743)

Co-authored-by: Rak Laptudirm <[email protected]>

dynamic/subsetsum_test.go

@@ -1,9 +1,6 @@
 package dynamic
 
-import (
-	"fmt"
-	"testing"
-)
+import "testing"
 
 func TestSubsetSum(t *testing.T) {
 
@@ -74,7 +71,7 @@ func TestSubsetSum(t *testing.T) {
 
 	for i := range subsetSumTestData {
 
-		t.Run(fmt.Sprintf(subsetSumTestData[i].description), func(t *testing.T) {
+		t.Run(subsetSumTestData[i].description, func(t *testing.T) {
 			array := subsetSumTestData[i].array
 			sum := subsetSumTestData[i].sum
 

math/matrix/matrix_test.go

@@ -32,7 +32,7 @@ func TestNewFromElements(t *testing.T) {
 		for j := 0; j < len(validElements[0]); j++ {
 			err := expectedm1.Set(i, j, validElements[i][j])
 			if err != nil {
-				t.Errorf("copyMatrix.Set error: " + err.Error())
+				t.Errorf("copyMatrix.Set error: %s", err.Error())
 			}
 		}
 	}
@@ -73,7 +73,7 @@ func TestMatrixGet(t *testing.T) {
 	matrix := matrix.New(3, 3, 0)
 	err := matrix.Set(1, 1, 42) // Set a specific value for testing
 	if err != nil {
-		t.Errorf("copyMatrix.Set error: " + err.Error())
+		t.Errorf("copyMatrix.Set error: %s", err.Error())
 	}
 	// Test case 1: Valid Get
 	val1, err1 := matrix.Get(1, 1)

math/matrix/strassenmatrixmultiply_test.go

@@ -66,7 +66,7 @@ func TestStrassenMatrixMultiply(t *testing.T) {
 	}
 }
 func TestMatrixMultiplication(t *testing.T) {
-	rand.Seed(time.Now().UnixNano())
+	rand.New(rand.NewSource(time.Now().UnixNano()))
 
 	// Generate random matrices for testing
 	size := 1 << (rand.Intn(8) + 1) // tests for matrix with n as power of 2
@@ -103,7 +103,7 @@ func TestMatrixMultiplication(t *testing.T) {
 }
 
 func MakeRandomMatrix[T constraints.Integer](rows, columns int) matrix.Matrix[T] {
-	rand.Seed(time.Now().UnixNano())
+	rand.New(rand.NewSource(time.Now().UnixNano()))
 
 	matrixData := make([][]T, rows)
 	for i := 0; i < rows; i++ {