Merge pull request #73 from vivki/main

add unit test for js_periodic
nginxinc · Oct 11, 2023 · 6bf177f · 6bf177f
2 parents badc29f + 3a98076
commit 6bf177f
Showing 1 changed file with 10 additions and 1 deletion.
diff --git a/analyze_test.go b/analyze_test.go
@@ -234,7 +234,7 @@ func TestAnalyze_njs(t *testing.T) {
 			blockCtx{"http", "location"},
 			false,
 		},
-		"js_periodic not ok in http": {
+		"js_periodic not ok in http if": {
 			&Directive{
 				Directive: "js_periodic",
 				Args:      []string{"function"},
@@ -252,6 +252,15 @@ func TestAnalyze_njs(t *testing.T) {
 			blockCtx{"stream", "server"},
 			false,
 		},
+		"js_periodic not ok in stream": {
+			&Directive{
+				Directive: "js_periodic",
+				Args:      []string{"function"},
+				Line:      5,
+			},
+			blockCtx{"stream"},
+			true,
+		},
 	}
 
 	for name, tc := range testcases {