Skip to content

Commit 05d173f

Browse files
committedApr 26, 2023
Use ArrayMap for Parameter and Credential Sets
Signed-off-by: Carolyn Van Slyck <me@carolynvanslyck.com>
1 parent 9629bbb commit 05d173f

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

54 files changed

+677
-603
lines changed
 

‎pkg/cnab/config-adapter/adapter.go

+3-6
Original file line numberDiff line numberDiff line change
@@ -540,11 +540,11 @@ func (c *ManifestConverter) generateParameterSources(b *cnab.ExtendedBundle) cna
540540
ps["porter-state"] = c.generateOutputParameterSource("porter-state")
541541

542542
// bundle.outputs.OUTPUT
543+
if b.Parameters == nil {
544+
b.Parameters = make(map[string]bundle.Parameter)
545+
}
543546
for _, outputDef := range c.Manifest.GetTemplatedOutputs() {
544547
wiringName, p, def := c.generateOutputWiringParameter(*b, outputDef.Name)
545-
if b.Parameters == nil {
546-
b.Parameters = make(map[string]bundle.Parameter, 1)
547-
}
548548
b.Parameters[wiringName] = p
549549
b.Definitions[wiringName] = &def
550550

@@ -555,9 +555,6 @@ func (c *ManifestConverter) generateParameterSources(b *cnab.ExtendedBundle) cna
555555
// bundle.dependencies.DEP.outputs.OUTPUT
556556
for _, ref := range c.Manifest.GetTemplatedDependencyOutputs() {
557557
wiringName, p, def := c.generateDependencyOutputWiringParameter(ref)
558-
if b.Parameters == nil {
559-
b.Parameters = make(map[string]bundle.Parameter, 1)
560-
}
561558
b.Parameters[wiringName] = p
562559
b.Definitions[wiringName] = &def
563560

‎pkg/cnab/provider/credentials_test.go

+8-12
Original file line numberDiff line numberDiff line change
@@ -23,12 +23,10 @@ func TestRuntime_loadCredentials(t *testing.T) {
2323

2424
r.TestConfig.TestContext.AddTestFile("testdata/db-creds.json", "/db-creds.json")
2525

26-
cs1 := storage.NewCredentialSet("", "mycreds", secrets.SourceMap{
27-
Name: "password",
28-
Source: secrets.Source{
29-
Strategy: secrets.SourceSecret,
30-
Hint: "password",
31-
},
26+
cs1 := storage.NewCredentialSet("", "mycreds")
27+
cs1.SetStrategy("password", secrets.Source{
28+
Strategy: secrets.SourceSecret,
29+
Hint: "password",
3230
})
3331

3432
err := r.credentials.InsertCredentialSet(context.Background(), cs1)
@@ -135,12 +133,10 @@ func TestRuntime_loadCredentials_WithApplyTo(t *testing.T) {
135133

136134
r.TestCredentials.AddSecret("password", "mypassword")
137135

138-
cs1 := storage.NewCredentialSet("", "mycreds", secrets.SourceMap{
139-
Name: "password",
140-
Source: secrets.Source{
141-
Strategy: secrets.SourceSecret,
142-
Hint: "password",
143-
},
136+
cs1 := storage.NewCredentialSet("", "mycreds")
137+
cs1.SetStrategy("password", secrets.Source{
138+
Strategy: secrets.SourceSecret,
139+
Hint: "password",
144140
})
145141

146142
err := r.credentials.InsertCredentialSet(context.Background(), cs1)

0 commit comments

Comments
 (0)