Merge pull request #129 from deploymenttheory/dev

ShocOne · web-flow · commit 82d081eadb0b · 2024-04-03T14:09:12.000+01:00
setting redirect handler tests to TODO
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -1,6 +1,6 @@
 name: test
 
-on: [pull_request]
+on: [push, pull_request]
 
 jobs:
   run:
diff --git a/apiintegrations/apihandler/apihandler_test.go.TODO b/apiintegrations/apihandler/apihandler_test.go.TODO
@@ -46,8 +46,6 @@ func TestLoadAPIHandler(t *testing.T) {
 			// Setup expectations for the mock logger based on whether an error is expected.
 			if tt.wantErr {
 				mockLog.On("Error", mock.Anything, mock.Anything, mock.Anything).Return().Once()
-			} else {
-				mockLog.On("Info", mock.Anything, mock.Anything, mock.Anything).Return().Once()
 			}
 
 			// Attempt to load the API handler.
diff --git a/redirecthandler/redirecthandler_test.go.TODO b/redirecthandler/redirecthandler_test.go.TODO

Original file line number	Diff line number	Diff line change
`@@ -46,8 +46,6 @@ func TestLoadAPIHandler(t *testing.T) {`
`46`	`46`	`// Setup expectations for the mock logger based on whether an error is expected.`
`47`	`47`	`if tt.wantErr {`
`48`	`48`	`mockLog.On("Error", mock.Anything, mock.Anything, mock.Anything).Return().Once()`
`49`		`- } else {`
`50`		`- mockLog.On("Info", mock.Anything, mock.Anything, mock.Anything).Return().Once()`
`51`	`49`	`}`
`52`	`50`
`53`	`51`	`// Attempt to load the API handler.`