Skip to content

Commit

Permalink
fix(plugins): Load ApiExtension config property from service yml in g…
Browse files Browse the repository at this point in the history
…ate-plugins-test (#1323)
  • Loading branch information
jonsie committed Sep 2, 2020
1 parent cba3174 commit 2695d36
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
Expand Up @@ -19,7 +19,6 @@ package com.netflix.spinnaker.gate.plugins
import com.netflix.spinnaker.gate.api.extension.ApiExtension
import com.netflix.spinnaker.gate.api.extension.HttpRequest
import com.netflix.spinnaker.gate.api.extension.HttpResponse
import com.netflix.spinnaker.kork.plugins.api.ExtensionConfiguration
import com.netflix.spinnaker.kork.plugins.api.PluginConfiguration
import com.netflix.spinnaker.kork.plugins.api.PluginSdks
import javax.annotation.Nonnull
Expand Down Expand Up @@ -68,5 +67,5 @@ class GateApiExtension(

@PluginConfiguration("api-extension")
class ApiExtensionConfigProperties {
var id: String = "test"
var id: String = ""
}
2 changes: 2 additions & 0 deletions gate-plugins-test/src/test/resources/gate-plugins-test.yml
Expand Up @@ -39,6 +39,8 @@ spinnaker:
plugins:
com.netflix.gate.enabled.plugin:
enabled: true
api-extension.config:
id: test
com.netflix.gate.disabled.plugin:
enabled: false
com.netflix.gate.version.not.supported.plugin:
Expand Down

0 comments on commit 2695d36

Please sign in to comment.