Removing unnecessary refactor

stripe · Feb 16, 2024 · 31190ca · 31190ca
1 parent 81393a5
commit 31190ca
Showing 1 changed file with 1 addition and 5 deletions.
diff --git a/pkg/smokescreen/smokescreen_test.go b/pkg/smokescreen/smokescreen_test.go
@@ -1328,10 +1328,6 @@ func findCanonicalProxyClose(logs []*logrus.Entry) *logrus.Entry {
 }
 
 func testConfig(role string) (*Config, error) {
-	return testConfigFromACLFile(role, "testdata/acl.yaml")
-}
-
-func testConfigFromACLFile(role string, filepath string) (*Config, error) {
 	conf := NewConfig()
 
 	if err := conf.SetAllowRanges(allowRanges); err != nil {
@@ -1341,7 +1337,7 @@ func testConfigFromACLFile(role string, filepath string) (*Config, error) {
 	conf.ExitTimeout = 10 * time.Second
 	conf.AdditionalErrorMessageOnDeny = "Proxy denied"
 	conf.Resolver = &net.Resolver{}
-	conf.SetupEgressAcl(filepath)
+	conf.SetupEgressAcl("testdata/acl.yaml")
 	conf.RoleFromRequest = func(req *http.Request) (string, error) {
 		return role, nil
 	}