engine/tracker: add CI apply and compare detection tests

Covers change application and baseline comparison helpers.

Made-with: Cursor

Author: AlexsanderHamir

engine/tracker/ci_apply_test.go

@@ -0,0 +1,47 @@
+package tracker
+
+import (
+	"testing"
+
+	"github.com/AlexsanderHamir/prof/internal"
+)
+
+func TestApplyCommandLineThresholdsNoFail(t *testing.T) {
+	r := &ProfileChangeReport{
+		FunctionChanges: []*FunctionChangeResult{
+			{FunctionName: "f", FlatChangePercent: 50, ChangeType: internal.REGRESSION},
+		},
+	}
+	sel := &Selections{UseThreshold: false}
+	if err := applyCommandLineThresholds(r, sel); err != nil {
+		t.Fatal(err)
+	}
+}
+
+func TestApplyCommandLineThresholdsTriggers(t *testing.T) {
+	r := &ProfileChangeReport{
+		FunctionChanges: []*FunctionChangeResult{
+			{FunctionName: "f", FlatChangePercent: 99, ChangeType: internal.REGRESSION},
+		},
+	}
+	sel := &Selections{UseThreshold: true, RegressionThreshold: 10}
+	if err := applyCommandLineThresholds(r, sel); err == nil {
+		t.Fatal("expected regression failure")
+	}
+}
+
+func TestShouldIgnoreFunctionByConfig(t *testing.T) {
+	cfg := &internal.CITrackingConfig{
+		IgnoreFunctions: []string{"x"},
+		IgnorePrefixes:  []string{"runtime."},
+	}
+	if !shouldIgnoreFunctionByConfig(cfg, "x") {
+		t.Fatal()
+	}
+	if !shouldIgnoreFunctionByConfig(cfg, "runtime.gc") {
+		t.Fatal()
+	}
+	if shouldIgnoreFunctionByConfig(cfg, "other") {
+		t.Fatal()
+	}
+}

engine/tracker/compare_detect_test.go

@@ -0,0 +1,48 @@
+package tracker
+
+import (
+	"testing"
+
+	"github.com/AlexsanderHamir/prof/internal"
+	"github.com/AlexsanderHamir/prof/parser"
+)
+
+func TestDetectChangeBetweenTwoObjectsRegression(t *testing.T) {
+	base := &parser.LineObj{FnName: "f", Flat: 10, Cum: 10}
+	cur := &parser.LineObj{FnName: "f", Flat: 20, Cum: 20}
+	r, err := detectChangeBetweenTwoObjects(base, cur)
+	if err != nil {
+		t.Fatal(err)
+	}
+	if r.ChangeType != internal.REGRESSION {
+		t.Fatalf("got %s", r.ChangeType)
+	}
+}
+
+func TestDetectChangeBetweenTwoObjectsImprovement(t *testing.T) {
+	base := &parser.LineObj{FnName: "f", Flat: 20, Cum: 20}
+	cur := &parser.LineObj{FnName: "f", Flat: 10, Cum: 10}
+	r, err := detectChangeBetweenTwoObjects(base, cur)
+	if err != nil {
+		t.Fatal(err)
+	}
+	if r.ChangeType != internal.IMPROVEMENT {
+		t.Fatalf("got %s", r.ChangeType)
+	}
+}
+
+func TestDetectChangeNilBaseline(t *testing.T) {
+	cur := &parser.LineObj{FnName: "f", Flat: 1, Cum: 1}
+	_, err := detectChangeBetweenTwoObjects(nil, cur)
+	if err == nil {
+		t.Fatal("expected error")
+	}
+}
+
+func TestLineObjByShortName(t *testing.T) {
+	objs := []*parser.LineObj{{FnName: "a", Flat: 1}, {FnName: "b", Flat: 2}}
+	m := lineObjByShortName(objs)
+	if len(m) != 2 || m["a"].Flat != 1 {
+		t.Fatal(m)
+	}
+}