fix(BUX-461): default fee

BuxOrg · Jan 12, 2024 · 2ebc639 · 2ebc639
1 parent 4f681e1
commit 2ebc639
Show file tree

Hide file tree

Showing 3 changed files with 29 additions and 13 deletions.
diff --git a/chainstate/definitions.go b/chainstate/definitions.go
@@ -56,7 +56,7 @@ const (
 func DefaultFee() *utils.FeeUnit {
 	return &utils.FeeUnit{
 		Satoshis: 1,
-		Bytes:    20,
+		Bytes:    1000,
 	}
 }
 

diff --git a/go.mod b/go.mod
diff --git a/utils/fees_test.go b/utils/fees_test.go
@@ -33,16 +33,30 @@ func TestGetOutputSizeForType(t *testing.T) {
 }
 
 func TestIsLowerThan(t *testing.T) {
-	one := FeeUnit{
-		Satoshis: 1,
-		Bytes:    20,
-	}
-	two := FeeUnit{
-		Satoshis: 2,
-		Bytes:    20,
-	}
-	assert.True(t, one.IsLowerThan(&two))
-	assert.False(t, two.IsLowerThan(&one))
+	t.Run("same satoshis, different bytes", func(t *testing.T) {
+		one := FeeUnit{
+			Satoshis: 1,
+			Bytes:    1000,
+		}
+		two := FeeUnit{
+			Satoshis: 1,
+			Bytes:    20,
+		}
+		assert.True(t, one.IsLowerThan(&two))
+		assert.False(t, two.IsLowerThan(&one))
+	})
+	t.Run("same bytes, different satoshis", func(t *testing.T) {
+		one := FeeUnit{
+			Satoshis: 1,
+			Bytes:    20,
+		}
+		two := FeeUnit{
+			Satoshis: 2,
+			Bytes:    20,
+		}
+		assert.True(t, one.IsLowerThan(&two))
+		assert.False(t, two.IsLowerThan(&one))
+	})
 }
 
 func TestLowestFee(t *testing.T) {
@@ -87,8 +101,8 @@ func TestLowestFee(t *testing.T) {
 	})
 
 	t.Run("lowest fee as defaultValue", func(t *testing.T) {
-		feeList, defaultFee := initTest()
-		feeList = []FeeUnit{}
+		_, defaultFee := initTest()
+		feeList := []FeeUnit{}
 		assert.Equal(t, defaultFee, *LowestFee(feeList, &defaultFee))
 	})
 }