Fix some unreachable code in unit tests (#1068)

Abirdcfly · Song Jian · commit 89926ce3ff48 · 2024-12-29T15:24:45.000+08:00
Signed-off-by: Abirdcfly &lt;fp544037857@gmail.com&gt;
Signed-off-by: Song Jian &lt;songjian@yf-networks.com&gt;
diff --git a/bfe_basic/condition/primitive_test.go b/bfe_basic/condition/primitive_test.go
@@ -234,7 +234,6 @@ func TestContextValueFetcher(t *testing.T) {
 	contextVal, err := hf.Fetch(req)
 	if err != nil {
 		t.Fatalf("Fetch(): %v", err)
-		t.FailNow()
 	}
 
 	// check
diff --git a/bfe_http2/server_test.go b/bfe_http2/server_test.go
@@ -3000,7 +3000,6 @@ func TestNoRstPostAfterGOAWAY(t *testing.T) {
 		}
 		if gf, ok := f.(*RSTStreamFrame); ok && gf.StreamID == 1 {
 			t.Fatal("got rst but want no ret")
-			break
 		}
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -234,7 +234,6 @@ func TestContextValueFetcher(t *testing.T) {`
`234`	`234`	`contextVal, err := hf.Fetch(req)`
`235`	`235`	`if err != nil {`
`236`	`236`	`t.Fatalf("Fetch(): %v", err)`
`237`		`- t.FailNow()`
`238`	`237`	`}`
`239`	`238`
`240`	`239`	`// check`
Original file line number	Diff line number	Diff line change
`@@ -3000,7 +3000,6 @@ func TestNoRstPostAfterGOAWAY(t *testing.T) {`
`3000`	`3000`	`}`
`3001`	`3001`	`if gf, ok := f.(*RSTStreamFrame); ok && gf.StreamID == 1 {`
`3002`	`3002`	`t.Fatal("got rst but want no ret")`
`3003`		`- break`
`3004`	`3003`	`}`
`3005`	`3004`	`}`
`3006`	`3005`