public_git/kubernetes-operator
1
0
Fork 0
mirror of https://github.com/jenkinsci/kubernetes-operator.git synced 2026-10-08 09:01:15 +02:00
Code Issues Releases Wiki Activity

#72 Add support for plugin names with underscore

Browse Source
This commit is contained in:
Jakub Al-Khalili
2019-08-14 09:51:55 +02:00
parent aecfd500f1
commit 5c55db489f
2 changed files with 13 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
pkg/controller/jenkins/plugins/plugin.go
+1 -1
View File
@@ -22,7 +22,7 @@ func (p Plugin) String() string {
}
var (
namePattern = regexp.MustCompile(`^[0-9a-z-]+$`)
namePattern = regexp.MustCompile(`^[0-9a-z-_]+$`)
versionPattern = regexp.MustCompile(`^[0-9\\.]+$`)
)
pkg/controller/jenkins/plugins/plugin_test.go
+12
View File
@@ -32,6 +32,18 @@ func TestVerifyDependencies(t *testing.T) {
got := VerifyDependencies(basePlugins)
assert.Equal(t, true, got)
})
t.Run("happy, two plugin names with names with underscores", func(t *testing.T) {
basePlugins := map[Plugin][]Plugin{
Must(New("first_root_plugin:1.0.0")): {
Must(New("first_plugin:0.0.1")),
},
Must(New("second_root_plugin:1.0.0")): {
Must(New("first_plugin:0.0.1")),
},
}
got := VerifyDependencies(basePlugins)
assert.Equal(t, true, got)
})
t.Run("fail, two root plugins have different versions", func(t *testing.T) {
basePlugins := map[Plugin][]Plugin{
Must(New("first-root-plugin:1.0.0")): {