feat: dependencies option for module create

integration/cmd_app_test.go

@@ -110,5 +110,36 @@ func TestGenerateAStargateAppWithEmptyModule(t *testing.T) {
 		ExecShouldError(),
 	))
 
+	env.Must(env.Exec("create a module with dependencies",
+		step.NewSteps(step.New(
+			step.Exec(
+				"starport",
+				"module",
+				"create",
+				"example_with_dep",
+				"--dep",
+				"account,bank,staking,slashing,example",
+				"--require-registration",
+			),
+			step.Workdir(path),
+		)),
+	))
+
+	env.Must(env.Exec("should prevent creating a module with invalid dependencies",
+		step.NewSteps(step.New(
+			step.Exec(
+				"starport",
+				"module",
+				"create",
+				"example_with_wrong_dep",
+				"--dep",
+				"dup,dup",
+				"--require-registration",
+			),
+			step.Workdir(path),
+		)),
+		ExecShouldError(),
+	))
+
 	env.EnsureAppIsSteady(path)
 }

integration/cmd_ibc_test.go

@@ -31,25 +31,59 @@ func TestCreateModuleWithIBC(t *testing.T) {
 
 	env.Must(env.Exec("create an IBC module with an ordered channel",
 		step.NewSteps(step.New(
-			step.Exec("starport", "module", "create", "--ibc", "orderedfoo", "--ordering", "ordered", "--require-registration"),
+			step.Exec(
+				"starport",
+				"module",
+				"create",
+				"--ibc",
+				"orderedfoo",
+				"--ordering",
+				"ordered",
+				"--require-registration",
+			),
 			step.Workdir(path),
 		)),
 	))
 
 	env.Must(env.Exec("create an IBC module with an unordered channel",
 		step.NewSteps(step.New(
-			step.Exec("starport", "module", "create", "--ibc", "unorderedfoo", "--ordering", "unordered", "--require-registration"),
+			step.Exec(
+				"starport",
+				"module",
+				"create",
+				"--ibc",
+				"unorderedfoo",
+				"--ordering",
+				"unordered",
+				"--require-registration",
+			),
 			step.Workdir(path),
 		)),
 	))
 
-	env.Must(env.Exec("create an non IBC module",
+	env.Must(env.Exec("create a non IBC module",
 		step.NewSteps(step.New(
 			step.Exec("starport", "module", "create", "foobar", "--require-registration"),
 			step.Workdir(path),
 		)),
 	))
 
+	env.Must(env.Exec("create an IBC module with dependencies",
+		step.NewSteps(step.New(
+			step.Exec(
+				"starport",
+				"module",
+				"create",
+				"example_with_dep",
+				"--ibc",
+				"--dep",
+				"account,bank,staking,slashing",
+				"--require-registration",
+			),
+			step.Workdir(path),
+		)),
+	))
+
 	env.EnsureAppIsSteady(path)
 }
 
@@ -69,7 +103,16 @@ func TestCreateIBCPacket(t *testing.T) {
 
 	env.Must(env.Exec("create a packet",
 		step.NewSteps(step.New(
-			step.Exec("starport", "packet", "bar", "text", "--module", "foo", "--ack", "foo:string,bar:int,foobar:bool"),
+			step.Exec(
+				"starport",
+				"packet",
+				"bar",
+				"text",
+				"--module",
+				"foo",
+				"--ack",
+				"foo:string,bar:int,foobar:bool",
+			),
 			step.Workdir(path),
 		)),
 	))

integration/cmd_message_test.go

@@ -16,7 +16,16 @@ func TestGenerateAnAppWithMessage(t *testing.T) {
 
 	env.Must(env.Exec("create a message",
 		step.NewSteps(step.New(
-			step.Exec("starport", "message", "do-foo", "text", "vote:int", "like:bool", "-r", "foo,bar:int,foobar:bool"),
+			step.Exec(
+				"starport",
+				"message",
+				"do-foo",
+				"text",
+				"vote:int",
+				"like:bool",
+				"-r",
+				"foo,bar:int,foobar:bool",
+			),
 			step.Workdir(path),
 		)),
 	))
@@ -45,7 +54,18 @@ func TestGenerateAnAppWithMessage(t *testing.T) {
 
 	env.Must(env.Exec("create a message in a module",
 		step.NewSteps(step.New(
-			step.Exec("starport", "message", "do-foo", "text", "--module", "foo", "--desc", "foo bar foobar", "--response", "foo,bar:int,foobar:bool"),
+			step.Exec(
+				"starport",
+				"message",
+				"do-foo",
+				"text",
+				"--module",
+				"foo",
+				"--desc",
+				"foo bar foobar",
+				"--response",
+				"foo,bar:int,foobar:bool",
+			),
 			step.Workdir(path),
 		)),
 	))

integration/cmd_query_test.go

@@ -14,14 +14,33 @@ func TestGenerateAnAppWithQuery(t *testing.T) {
 
 	env.Must(env.Exec("create a query",
 		step.NewSteps(step.New(
-			step.Exec("starport", "query", "foo", "text", "vote:int", "like:bool", "-r", "foo,bar:int,foobar:bool"),
+			step.Exec(
+				"starport",
+				"query",
+				"foo",
+				"text",
+				"vote:int",
+				"like:bool",
+				"-r",
+				"foo,bar:int,foobar:bool",
+			),
 			step.Workdir(path),
 		)),
 	))
 
 	env.Must(env.Exec("create a paginated query",
 		step.NewSteps(step.New(
-			step.Exec("starport", "query", "bar", "text", "vote:int", "like:bool", "-r", "foo,bar:int,foobar:bool", "--paginated"),
+			step.Exec(
+				"starport",
+				"query",
+				"bar",
+				"text",
+				"vote:int",
+				"like:bool",
+				"-r",
+				"foo,bar:int,foobar:bool",
+				"--paginated",
+			),
 			step.Workdir(path),
 		)),
 	))
@@ -50,7 +69,18 @@ func TestGenerateAnAppWithQuery(t *testing.T) {
 
 	env.Must(env.Exec("create a query in a module",
 		step.NewSteps(step.New(
-			step.Exec("starport", "query", "foo", "text", "--module", "foo", "--desc", "foo bar foobar", "--response", "foo,bar:int,foobar:bool"),
+			step.Exec(
+				"starport",
+				"query",
+				"foo",
+				"text",
+				"--module",
+				"foo",
+				"--desc",
+				"foo bar foobar",
+				"--response",
+				"foo,bar:int,foobar:bool",
+			),
 			step.Workdir(path),
 		)),
 	))

starport/cmd/module-create.go

@@ -15,6 +15,7 @@ import (
 )
 
 const (
+	flagDep                 = "dep"
 	flagIBC                 = "ibc"
 	flagIBCOrdering         = "ordering"
 	flagRequireRegistration = "require-registration"
@@ -53,6 +54,7 @@ func NewModuleCreate() *cobra.Command {
 		Args:  cobra.MinimumNArgs(1),
 		RunE:  createModuleHandler,
 	}
+	c.Flags().StringSlice(flagDep, []string{}, "module dependencies (e.g. --dep account,bank)")
 	c.Flags().Bool(flagIBC, false, "scaffold an IBC module")
 	c.Flags().String(flagIBCOrdering, "none", "channel ordering of the IBC module [none|ordered|unordered]")
 	c.Flags().Bool(flagRequireRegistration, false, "if true command will fail if module can't be registered")
@@ -86,6 +88,12 @@ func createModuleHandler(cmd *cobra.Command, args []string) error {
 		options = append(options, scaffolder.WithIBCChannelOrdering(ibcOrdering), scaffolder.WithIBC())
 	}
 
+	// Get module dependencies
+	dependencies, err := cmd.Flags().GetStringSlice(flagDep)
+	if err != nil {
+		return err
+	}
+
 	sc, err := scaffolder.New(appPath)
 	if err != nil {
 		return err
@@ -111,6 +119,30 @@ func createModuleHandler(cmd *cobra.Command, args []string) error {
 		fmt.Println(sourceModificationToString(sm))
 	}
 
+	if len(dependencies) > 0 {
+		dependencyWarning(dependencies)
+	}
+
 	io.Copy(cmd.OutOrStdout(), &msg)
 	return nil
 }
+
+// in previously scaffolded apps gov keeper is defined below the scaffolded module keeper definition
+// therefore we must warn the user to manually move the definition if it's the case
+// https://github.com/tendermint/starport/issues/818#issuecomment-865736052
+const govWarning = `⚠️ If your app has been scaffolded with Starport 0.16.x or below
+Please make sure that your module keeper definition is defined after gov module keeper definition in app/app.go:
+
+app.GovKeeper = ...
+...
+[your module keeper definition]
+`
+
+// dependencyWarning is used to print a warning if gov is provided as a dependency
+func dependencyWarning(dependencies []string) {
+	for _, dep := range dependencies {
+		if dep == "gov" {
+			fmt.Print(govWarning)
+		}
+	}
+}

starport/pkg/cosmosanalysis/app/app.go

@@ -0,0 +1,52 @@
+package app
+
+import (
+	"fmt"
+	"go/ast"
+	"go/parser"
+	"go/token"
+)
+
+const (
+	appTypeName = "App"
+)
+
+// CheckKeeper checks for the existence of the keeper with the provided name in the app structure
+func CheckKeeper(appPath, keeperName string) error {
+	fileSet := token.NewFileSet()
+	f, err := parser.ParseFile(fileSet, appPath, nil, 0)
+	if err != nil {
+		return err
+	}
+
+	// Inspect the file for app struct
+	var found bool
+	ast.Inspect(f, func(n ast.Node) bool {
+		// look for struct methods.
+		appType, ok := n.(*ast.TypeSpec)
+		if !ok || appType.Name.Name != appTypeName {
+			return true
+		}
+
+		appStruct, ok := appType.Type.(*ast.StructType)
+		if !ok {
+			return true
+		}
+
+		// Search for the keeper specific field
+		for _, field := range appStruct.Fields.List {
+			for _, fieldName := range field.Names {
+				if fieldName.Name == keeperName {
+					found = true
+				}
+			}
+		}
+
+		return false
+	})
+
+	if !found {
+		return fmt.Errorf("app doesn't contain %s", keeperName)
+	}
+	return nil
+}