Edw/fix put config panic (#864)

* Ensure project name is set in set config call

* Add test to make sure project name is set in putconfigrequest

* Update src/pkg/cli/configSet_test.go

---------

Co-authored-by: Edward J <[email protected]>
Co-authored-by: Lio李歐 <[email protected]>

Author: 3 people

src/pkg/cli/configSet.go

@@ -19,5 +19,5 @@ func ConfigSet(ctx context.Context, loader client.Loader, provider client.Provid
 		return ErrDryRun
 	}
 
-	return provider.PutConfig(ctx, &defangv1.PutConfigRequest{Name: name, Value: value})
+	return provider.PutConfig(ctx, &defangv1.PutConfigRequest{Project: projectName, Name: name, Value: value})
 }

src/pkg/cli/configSet_test.go

@@ -0,0 +1,32 @@
+package cli
+
+import (
+	"context"
+	"errors"
+	"testing"
+
+	"github.com/DefangLabs/defang/src/pkg/cli/client"
+	"github.com/DefangLabs/defang/src/pkg/cli/compose"
+	defangv1 "github.com/DefangLabs/defang/src/protos/io/defang/v1"
+)
+
+func TestConfigSet(t *testing.T) {
+	ctx := context.Background()
+	loader := client.MockLoader{Project: &compose.Project{Name: "test"}}
+	provider := MustHaveProjectNamePutConfigProvider{}
+	err := ConfigSet(ctx, loader, provider, "test_name", "test_value")
+	if err != nil {
+		t.Fatalf("ConfigSet() error = %v", err)
+	}
+}
+
+type MustHaveProjectNamePutConfigProvider struct {
+	client.Provider
+}
+
+func (m MustHaveProjectNamePutConfigProvider) PutConfig(ctx context.Context, req *defangv1.PutConfigRequest) error {
+	if req.Project == "" {
+		return errors.New("project name is missing")
+	}
+	return nil
+}