Fix: Make 'flytectl compile' consider launchplans used within workflows

flytectl/cmd/compile/compile.go

@@ -80,16 +80,20 @@ func compileFromPackage(packagePath string) error {
 		return err
 	}
 
+	var providers []common.InterfaceProvider
+	for _, plan := range plans {
+		providers = append(providers, compiler.NewLaunchPlanInterfaceProvider(*plan))
+	}
+
 	// compile workflows
 	for wfName, workflow := range workflows {
 
 		fmt.Println("\nCompiling workflow:", wfName)
-		plan := plans[wfName]
 
 		_, err := compiler.CompileWorkflow(workflow.Template,
 			workflow.SubWorkflows,
 			compiledTasks,
-			[]common.InterfaceProvider{compiler.NewLaunchPlanInterfaceProvider(*plan)})
+			providers)
 		if err != nil {
 			fmt.Println(":( Error Compiling workflow:", wfName)
 			return err

flytectl/cmd/compile/compile_test.go

@@ -69,4 +69,7 @@ func TestCompilePackage(t *testing.T) {
 	err = compileFromPackage("testdata/invalidworkflow.tgz")
 	assert.NotNil(t, err, "unable to handle invalid workflow")
 
+	// testing workflows with launchplans used within workflow
+	err = compileFromPackage("testdata/launchplan-in-wf.tgz")
+	assert.Nil(t, err, "unable to compile workflow with launchplans used within workflow")
 }