Merge pull request #95 from uber/lu.module

Update module system initializer signature
uber · May 11, 2017 · a89ab21 · a89ab21
2 parents a0ba255 + b2b7099
commit a89ab21
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 12 deletions.
diff --git a/codegen/module_system.go b/codegen/module_system.go
@@ -92,8 +92,6 @@ type ClientStub struct {
 // NewDefaultModuleSystem creates a fresh instance of the default zanzibar
 // module system (clients, endpoints)
 func NewDefaultModuleSystem(
-	configDirName string,
-	middlewareConfig string,
 	h *PackageHelper,
 ) (*module.System, error) {
 	system := module.NewSystem()

diff --git a/codegen/runner/runner.go b/codegen/runner/runner.go
@@ -82,11 +82,7 @@ func main() {
 		err, fmt.Sprintf("Can't build package helper %s", configDirName),
 	)
 
-	moduleSystem, err := codegen.NewDefaultModuleSystem(
-		configDirName,
-		config.MustGetString("middlewareConfig"),
-		packageHelper,
-	)
+	moduleSystem, err := codegen.NewDefaultModuleSystem(packageHelper)
 	checkError(
 		err, fmt.Sprintf("Error creating module system %s", configDirName),
 	)

diff --git a/codegen/template_test.go b/codegen/template_test.go
@@ -71,11 +71,7 @@ func TestGenerateBar(t *testing.T) {
 		return
 	}
 
-	moduleSystem, err := codegen.NewDefaultModuleSystem(
-		absGatewayPath,
-		"./middlewares/middleware-config.json",
-		packageHelper,
-	)
+	moduleSystem, err := codegen.NewDefaultModuleSystem(packageHelper)
 	if !assert.NoError(t, err, "failed to create module system", err) {
 		return
 	}