-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
add not implemented errors where needed #32
Conversation
billiford
commented
Oct 13, 2020
- add errors when trying to scale or perform a rolling restart on an unsupported kind
- add error when trying to patch using an unsupported merge type
- add error when trying to resolve unsupported artifact type
@@ -73,6 +73,17 @@ var _ = Describe("Restart", func() { | |||
}) | |||
}) | |||
|
|||
When("the kind is not supported to restarted", func() { | |||
BeforeEach(func() { | |||
actionConfig.Operation.RollingRestartManifest.ManifestName = "not-supported-kind test-name" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shuldnt this be?:
actionConfig.Operation.RollingRestartManifest.ManifestName = “not-supported-kind”?
to match the message in Line #83
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is one aspect of the Spinnaker API that I've always found weird. When referencing a manifest by name it is almost always referenced as <kind> <name>
, which is what I'm doing here.
@@ -84,6 +84,17 @@ var _ = Describe("Scale", func() { | |||
}) | |||
}) | |||
|
|||
When("the kind is not supported to scale", func() { | |||
BeforeEach(func() { | |||
actionConfig.Operation.ScaleManifest.ManifestName = "not-supported-kind test-name" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shuldnt this be?:
actionConfig.Operation.ScaleManifest.ManifestName = “not-supported-kind”?
to match the message in Line #94
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added some questions.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
* associate server groups and managers using UID * fix linter