Skip to content

Commit 82cb75c

Browse files
committed
Parse custom plugin identifier
1 parent 4b01e4a commit 82cb75c

File tree

2 files changed

+12
-4
lines changed

2 files changed

+12
-4
lines changed

_tests/integration/validate_test.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -94,7 +94,7 @@ func Test_ValidateTest(t *testing.T) {
9494
cmd := command.New(binPath(), "validate", "-c", "trigger_params_test_bitrise.yml")
9595
out, err = cmd.RunAndReturnTrimmedCombinedOutput()
9696
})
97-
require.NoError(t, err)
97+
require.NoError(t, err, out)
9898
require.Equal(t, "Config is valid: \x1b[32;1mtrue\x1b[0m", out)
9999
require.Equal(t, true, elapsed < runtimeLimit, runningTimeMsg, elapsed, elapsed-runtimeLimit)
100100
}
@@ -251,7 +251,7 @@ func Test_ValidToolConfigValidateTest(t *testing.T) {
251251
cmd := command.New(binPath(), "validate", "-c", configPth)
252252
out, err = cmd.RunAndReturnTrimmedCombinedOutput()
253253
})
254-
require.NoError(t, err)
254+
require.NoError(t, err, out)
255255
require.Equal(t, "Config is valid: \x1b[32;1mtrue\x1b[0m", out)
256256
require.Equal(t, true, elapsed < runtimeLimit, runningTimeMsg, elapsed, elapsed-runtimeLimit)
257257
}
@@ -269,7 +269,7 @@ func Test_InvalidToolConfigValidateTest(t *testing.T) {
269269
})
270270
require.Error(t, err, out)
271271
require.Contains(t, out, "Config is valid: \x1b[31;1mfalse\x1b[0m")
272-
require.Contains(t, out, "invalid provider")
272+
require.Contains(t, out, "URL of extra plugin empty-url-tool is empty")
273273
require.Equal(t, true, elapsed < runtimeLimit, runningTimeMsg, elapsed, elapsed-runtimeLimit)
274274
}
275275

toolprovider/config.go

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,11 +16,19 @@ func getToolRequests(config models.BitriseDataModel) ([]provider.ToolRequest, er
1616
if err != nil {
1717
return nil, fmt.Errorf("parse %s version: %w", toolID, err)
1818
}
19+
20+
var pluginIdentifier *string
21+
if config.ToolConfig != nil && config.ToolConfig.ExtraPlugins != nil {
22+
if pluginID, ok := config.ToolConfig.ExtraPlugins[toolID]; ok {
23+
pluginIdentifier = &pluginID
24+
}
25+
}
26+
1927
toolRequests = append(toolRequests, provider.ToolRequest{
2028
ToolName: provider.ToolID(toolID),
2129
UnparsedVersion: v,
2230
ResolutionStrategy: strategy,
23-
// TODO: plugin identifier
31+
PluginIdentifier: pluginIdentifier,
2432
})
2533
}
2634

0 commit comments

Comments
 (0)