Merge pull request #87 from Smeat/minimum_curve_fix

Fix "minimum" fan curve
markusressel · Feb 3, 2022 · 5ee84eb · 5ee84eb
2 parents 16f134f + c764d00
commit 5ee84eb
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/internal/curves/curve.go b/internal/curves/curve.go
@@ -116,7 +116,7 @@ func (c functionSpeedCurve) Evaluate() (value int, err error) {
 		delta := dmax - dmin
 		return int(delta), nil
 	case configuration.FunctionMinimum:
-		var min float64
+		var min float64 = 255
 		for _, v := range values {
 			min = math.Min(min, float64(v))
 		}

diff --git a/internal/curves/curve_test.go b/internal/curves/curve_test.go
@@ -266,7 +266,7 @@ func TestFunctionCurveDelta(t *testing.T) {
 
 func TestFunctionCurveMinimum(t *testing.T) {
 	// GIVEN
-	temp1 := 40000.0
+	temp1 := 60000.0
 	temp2 := 80000.0
 
 	s1 := MockSensor{
@@ -319,7 +319,7 @@ func TestFunctionCurveMinimum(t *testing.T) {
 	}
 
 	// THEN
-	assert.Equal(t, 0, result)
+	assert.Equal(t, 127, result)
 }
 
 func TestFunctionCurveMaximum(t *testing.T) {