2018-05-11 12:47:21 +00:00
|
|
|
@setupApplicationTest
|
2018-06-04 13:53:51 +00:00
|
|
|
Feature: dc / kvs / update: KV Update
|
2018-07-05 12:43:03 +00:00
|
|
|
Background:
|
2018-05-11 12:47:21 +00:00
|
|
|
Given 1 datacenter model with the value "datacenter"
|
2018-07-05 12:43:03 +00:00
|
|
|
Scenario: Update to [Name] change value to [Value]
|
2018-05-11 12:47:21 +00:00
|
|
|
And 1 kv model from yaml
|
|
|
|
|
---
|
UI: [BUGFIX] Decode/encode urls (#5206)
In https://github.com/hashicorp/consul/commit/858b05fc3127d3d20d9554e932353d767c7b5fdc#diff-46ef88aa04507fb9b039344277531584
we removed encoding values in pathnames as we thought they were
eventually being encoded by `ember`. It looks like this isn't the case.
Turns out sometimes they are encoded sometimes they aren't. It's complicated.
If at all possible refer to the PR https://github.com/hashicorp/consul/pull/5206.
It's related to the difference between `dynamic` routes and `wildcard` routes.
Partly related to this is a decision on whether we urlencode the slashes within service names or not. Whilst historically we haven't done this, we feel its a good time to change this behaviour, so we'll also be changing services to use dynamic routes instead of wildcard routes. So service links will then look like /ui/dc-1/services/application%2Fservice rather than /ui/dc-1/services/application/service
Here, we define our routes in a declarative format (for the moment at least JSON) outside of Router.map, and loop through this within Router.map to set all our routes using the standard this.route method. We essentially configure our Router from the outside. As this configuration is now done declaratively outside of Router.map we can also make this data available to href-to and paramsFor, allowing us to detect wildcard routes and therefore apply urlencoding/decoding.
Where I mention 'conditionally' below, this is detection is what is used for the decision.
We conditionally add url encoding to the `{{href-to}}` helper/addon. The
reasoning here is, if we are asking for a 'href/url' then whatever we
receive back should always be urlencoded. We've done this by reusing as much
code from the original `ember-href-to` addon as possible, after this
change every call to the `{{href-to}}` helper will be urlencoded.
As all links using `{{href-to}}` are now properly urlencoded. We also
need to decode them in the correct place 'on the other end', so..
We also override the default `Route.paramsFor` method to conditionally decode all
params before passing them to the `Route.model` hook.
Lastly (the revert), as we almost consistently use url params to
construct API calls, we make sure we re-encode any slugs that have been
passed in by the user/developer. The original API for the `createURL`
function was to allow you to pass values that didn't need encoding,
values that **did** need encoding, followed by query params (which again
require url encoding)
All in all this should make the entire ember app url encode/decode safe.
2019-01-23 13:46:59 +00:00
|
|
|
Key: "[Name]"
|
2020-02-05 09:37:45 +00:00
|
|
|
Flags: 12
|
2018-05-11 12:47:21 +00:00
|
|
|
---
|
|
|
|
|
When I visit the kv page for yaml
|
|
|
|
|
---
|
|
|
|
|
dc: datacenter
|
UI: [BUGFIX] Decode/encode urls (#5206)
In https://github.com/hashicorp/consul/commit/858b05fc3127d3d20d9554e932353d767c7b5fdc#diff-46ef88aa04507fb9b039344277531584
we removed encoding values in pathnames as we thought they were
eventually being encoded by `ember`. It looks like this isn't the case.
Turns out sometimes they are encoded sometimes they aren't. It's complicated.
If at all possible refer to the PR https://github.com/hashicorp/consul/pull/5206.
It's related to the difference between `dynamic` routes and `wildcard` routes.
Partly related to this is a decision on whether we urlencode the slashes within service names or not. Whilst historically we haven't done this, we feel its a good time to change this behaviour, so we'll also be changing services to use dynamic routes instead of wildcard routes. So service links will then look like /ui/dc-1/services/application%2Fservice rather than /ui/dc-1/services/application/service
Here, we define our routes in a declarative format (for the moment at least JSON) outside of Router.map, and loop through this within Router.map to set all our routes using the standard this.route method. We essentially configure our Router from the outside. As this configuration is now done declaratively outside of Router.map we can also make this data available to href-to and paramsFor, allowing us to detect wildcard routes and therefore apply urlencoding/decoding.
Where I mention 'conditionally' below, this is detection is what is used for the decision.
We conditionally add url encoding to the `{{href-to}}` helper/addon. The
reasoning here is, if we are asking for a 'href/url' then whatever we
receive back should always be urlencoded. We've done this by reusing as much
code from the original `ember-href-to` addon as possible, after this
change every call to the `{{href-to}}` helper will be urlencoded.
As all links using `{{href-to}}` are now properly urlencoded. We also
need to decode them in the correct place 'on the other end', so..
We also override the default `Route.paramsFor` method to conditionally decode all
params before passing them to the `Route.model` hook.
Lastly (the revert), as we almost consistently use url params to
construct API calls, we make sure we re-encode any slugs that have been
passed in by the user/developer. The original API for the `createURL`
function was to allow you to pass values that didn't need encoding,
values that **did** need encoding, followed by query params (which again
require url encoding)
All in all this should make the entire ember app url encode/decode safe.
2019-01-23 13:46:59 +00:00
|
|
|
kv: "[Name]"
|
2018-05-11 12:47:21 +00:00
|
|
|
---
|
UI: [BUGFIX] Decode/encode urls (#5206)
In https://github.com/hashicorp/consul/commit/858b05fc3127d3d20d9554e932353d767c7b5fdc#diff-46ef88aa04507fb9b039344277531584
we removed encoding values in pathnames as we thought they were
eventually being encoded by `ember`. It looks like this isn't the case.
Turns out sometimes they are encoded sometimes they aren't. It's complicated.
If at all possible refer to the PR https://github.com/hashicorp/consul/pull/5206.
It's related to the difference between `dynamic` routes and `wildcard` routes.
Partly related to this is a decision on whether we urlencode the slashes within service names or not. Whilst historically we haven't done this, we feel its a good time to change this behaviour, so we'll also be changing services to use dynamic routes instead of wildcard routes. So service links will then look like /ui/dc-1/services/application%2Fservice rather than /ui/dc-1/services/application/service
Here, we define our routes in a declarative format (for the moment at least JSON) outside of Router.map, and loop through this within Router.map to set all our routes using the standard this.route method. We essentially configure our Router from the outside. As this configuration is now done declaratively outside of Router.map we can also make this data available to href-to and paramsFor, allowing us to detect wildcard routes and therefore apply urlencoding/decoding.
Where I mention 'conditionally' below, this is detection is what is used for the decision.
We conditionally add url encoding to the `{{href-to}}` helper/addon. The
reasoning here is, if we are asking for a 'href/url' then whatever we
receive back should always be urlencoded. We've done this by reusing as much
code from the original `ember-href-to` addon as possible, after this
change every call to the `{{href-to}}` helper will be urlencoded.
As all links using `{{href-to}}` are now properly urlencoded. We also
need to decode them in the correct place 'on the other end', so..
We also override the default `Route.paramsFor` method to conditionally decode all
params before passing them to the `Route.model` hook.
Lastly (the revert), as we almost consistently use url params to
construct API calls, we make sure we re-encode any slugs that have been
passed in by the user/developer. The original API for the `createURL`
function was to allow you to pass values that didn't need encoding,
values that **did** need encoding, followed by query params (which again
require url encoding)
All in all this should make the entire ember app url encode/decode safe.
2019-01-23 13:46:59 +00:00
|
|
|
Then the url should be /datacenter/kv/[EncodedName]/edit
|
2020-01-28 14:25:52 +00:00
|
|
|
And the title should be "Edit Key/Value - Consul"
|
2018-09-20 09:36:30 +00:00
|
|
|
# Turn the Code Editor off so we can fill the value easier
|
|
|
|
|
And I click "[name=json]"
|
2018-07-03 14:40:15 +00:00
|
|
|
Then I fill in with yaml
|
2018-05-11 12:47:21 +00:00
|
|
|
---
|
|
|
|
|
value: [Value]
|
|
|
|
|
---
|
|
|
|
|
And I submit
|
ui: Adds Partitions to the HTTP layer (#10447)
This PR mainly adds partition to our HTTP adapter. Additionally and perhaps most importantly, we've also taken the opportunity to move our 'conditional namespaces' deeper into the app.
The reason for doing this was, we like that namespaces should be thought of as required instead of conditional, 'special' things and would like the same thinking to be applied to partitions.
Now, instead of using code throughout the app throughout the adapters to add/remove namespaces or partitions depending on whether they are enabled or not. As a UI engineer you just pretend that namespaces and partitions are always enabled, and we remove them for you deeper in the app, out of the way of you forgetting to treat these properties as a special case.
Notes:
Added a PartitionAbility while we were there (not used as yet)
Started to remove the CONSTANT variables we had just for property names. I prefer that our adapters are as readable and straightforwards as possible, it just looks like HTTP.
We'll probably remove our formatDatacenter method we use also at some point, it was mainly too make it look the same as our previous formatNspace, but now we don't have that, it instead now looks different!
We enable parsing of partition in the UIs URL, but this is feature flagged so still does nothing just yet.
All of the test changes were related to the fact that we were treating client.url as a function rather than a method, and now that we reference this in client.url (etc) it needs binding to client.
2021-09-15 17:09:55 +00:00
|
|
|
Then a PUT request was made to "/v1/kv/[EncodedName]?dc=datacenter&ns=@!namespace&flags=12" with the body "[Value]"
|
2018-08-29 18:14:31 +00:00
|
|
|
And "[data-notification]" has the "notification-update" class
|
|
|
|
|
And "[data-notification]" has the "success" class
|
2018-05-11 12:47:21 +00:00
|
|
|
Where:
|
UI: [BUGFIX] Decode/encode urls (#5206)
In https://github.com/hashicorp/consul/commit/858b05fc3127d3d20d9554e932353d767c7b5fdc#diff-46ef88aa04507fb9b039344277531584
we removed encoding values in pathnames as we thought they were
eventually being encoded by `ember`. It looks like this isn't the case.
Turns out sometimes they are encoded sometimes they aren't. It's complicated.
If at all possible refer to the PR https://github.com/hashicorp/consul/pull/5206.
It's related to the difference between `dynamic` routes and `wildcard` routes.
Partly related to this is a decision on whether we urlencode the slashes within service names or not. Whilst historically we haven't done this, we feel its a good time to change this behaviour, so we'll also be changing services to use dynamic routes instead of wildcard routes. So service links will then look like /ui/dc-1/services/application%2Fservice rather than /ui/dc-1/services/application/service
Here, we define our routes in a declarative format (for the moment at least JSON) outside of Router.map, and loop through this within Router.map to set all our routes using the standard this.route method. We essentially configure our Router from the outside. As this configuration is now done declaratively outside of Router.map we can also make this data available to href-to and paramsFor, allowing us to detect wildcard routes and therefore apply urlencoding/decoding.
Where I mention 'conditionally' below, this is detection is what is used for the decision.
We conditionally add url encoding to the `{{href-to}}` helper/addon. The
reasoning here is, if we are asking for a 'href/url' then whatever we
receive back should always be urlencoded. We've done this by reusing as much
code from the original `ember-href-to` addon as possible, after this
change every call to the `{{href-to}}` helper will be urlencoded.
As all links using `{{href-to}}` are now properly urlencoded. We also
need to decode them in the correct place 'on the other end', so..
We also override the default `Route.paramsFor` method to conditionally decode all
params before passing them to the `Route.model` hook.
Lastly (the revert), as we almost consistently use url params to
construct API calls, we make sure we re-encode any slugs that have been
passed in by the user/developer. The original API for the `createURL`
function was to allow you to pass values that didn't need encoding,
values that **did** need encoding, followed by query params (which again
require url encoding)
All in all this should make the entire ember app url encode/decode safe.
2019-01-23 13:46:59 +00:00
|
|
|
---------------------------------------------------------
|
|
|
|
|
| Name | EncodedName | Value |
|
|
|
|
|
| key | key | value |
|
|
|
|
|
| #key | %23key | value |
|
|
|
|
|
| key-name | key-name | a value |
|
|
|
|
|
| key name | key%20name | a value |
|
|
|
|
|
| folder/key-name | folder/key-name | a value |
|
|
|
|
|
---------------------------------------------------------
|
2018-07-05 12:43:03 +00:00
|
|
|
Scenario: Update to a key change value to ' '
|
|
|
|
|
And 1 kv model from yaml
|
|
|
|
|
---
|
|
|
|
|
Key: key
|
2020-02-05 09:37:45 +00:00
|
|
|
Flags: 12
|
2018-07-05 12:43:03 +00:00
|
|
|
---
|
|
|
|
|
When I visit the kv page for yaml
|
|
|
|
|
---
|
|
|
|
|
dc: datacenter
|
|
|
|
|
kv: key
|
|
|
|
|
---
|
|
|
|
|
Then the url should be /datacenter/kv/key/edit
|
2018-09-20 09:36:30 +00:00
|
|
|
# Turn the Code Editor off so we can fill the value easier
|
|
|
|
|
And I click "[name=json]"
|
2018-07-05 12:43:03 +00:00
|
|
|
Then I fill in with yaml
|
|
|
|
|
---
|
|
|
|
|
value: ' '
|
|
|
|
|
---
|
|
|
|
|
And I submit
|
ui: Adds Partitions to the HTTP layer (#10447)
This PR mainly adds partition to our HTTP adapter. Additionally and perhaps most importantly, we've also taken the opportunity to move our 'conditional namespaces' deeper into the app.
The reason for doing this was, we like that namespaces should be thought of as required instead of conditional, 'special' things and would like the same thinking to be applied to partitions.
Now, instead of using code throughout the app throughout the adapters to add/remove namespaces or partitions depending on whether they are enabled or not. As a UI engineer you just pretend that namespaces and partitions are always enabled, and we remove them for you deeper in the app, out of the way of you forgetting to treat these properties as a special case.
Notes:
Added a PartitionAbility while we were there (not used as yet)
Started to remove the CONSTANT variables we had just for property names. I prefer that our adapters are as readable and straightforwards as possible, it just looks like HTTP.
We'll probably remove our formatDatacenter method we use also at some point, it was mainly too make it look the same as our previous formatNspace, but now we don't have that, it instead now looks different!
We enable parsing of partition in the UIs URL, but this is feature flagged so still does nothing just yet.
All of the test changes were related to the fact that we were treating client.url as a function rather than a method, and now that we reference this in client.url (etc) it needs binding to client.
2021-09-15 17:09:55 +00:00
|
|
|
Then a PUT request was made to "/v1/kv/key?dc=datacenter&ns=@!namespace&flags=12" with the body " "
|
2018-08-29 18:14:31 +00:00
|
|
|
Then the url should be /datacenter/kv
|
2020-01-28 14:25:52 +00:00
|
|
|
And the title should be "Key/Value - Consul"
|
2018-08-29 18:14:31 +00:00
|
|
|
And "[data-notification]" has the "notification-update" class
|
|
|
|
|
And "[data-notification]" has the "success" class
|
2018-07-05 12:43:03 +00:00
|
|
|
Scenario: Update to a key change value to ''
|
|
|
|
|
And 1 kv model from yaml
|
|
|
|
|
---
|
|
|
|
|
Key: key
|
2020-02-05 09:37:45 +00:00
|
|
|
Flags: 12
|
2018-07-05 12:43:03 +00:00
|
|
|
---
|
|
|
|
|
When I visit the kv page for yaml
|
|
|
|
|
---
|
|
|
|
|
dc: datacenter
|
|
|
|
|
kv: key
|
|
|
|
|
---
|
|
|
|
|
Then the url should be /datacenter/kv/key/edit
|
2018-09-20 09:36:30 +00:00
|
|
|
# Turn the Code Editor off so we can fill the value easier
|
|
|
|
|
And I click "[name=json]"
|
2018-07-05 12:43:03 +00:00
|
|
|
Then I fill in with yaml
|
|
|
|
|
---
|
|
|
|
|
value: ''
|
|
|
|
|
---
|
|
|
|
|
And I submit
|
ui: Adds Partitions to the HTTP layer (#10447)
This PR mainly adds partition to our HTTP adapter. Additionally and perhaps most importantly, we've also taken the opportunity to move our 'conditional namespaces' deeper into the app.
The reason for doing this was, we like that namespaces should be thought of as required instead of conditional, 'special' things and would like the same thinking to be applied to partitions.
Now, instead of using code throughout the app throughout the adapters to add/remove namespaces or partitions depending on whether they are enabled or not. As a UI engineer you just pretend that namespaces and partitions are always enabled, and we remove them for you deeper in the app, out of the way of you forgetting to treat these properties as a special case.
Notes:
Added a PartitionAbility while we were there (not used as yet)
Started to remove the CONSTANT variables we had just for property names. I prefer that our adapters are as readable and straightforwards as possible, it just looks like HTTP.
We'll probably remove our formatDatacenter method we use also at some point, it was mainly too make it look the same as our previous formatNspace, but now we don't have that, it instead now looks different!
We enable parsing of partition in the UIs URL, but this is feature flagged so still does nothing just yet.
All of the test changes were related to the fact that we were treating client.url as a function rather than a method, and now that we reference this in client.url (etc) it needs binding to client.
2021-09-15 17:09:55 +00:00
|
|
|
Then a PUT request was made to "/v1/kv/key?dc=datacenter&ns=@!namespace&flags=12" with no body
|
2018-08-29 18:14:31 +00:00
|
|
|
Then the url should be /datacenter/kv
|
|
|
|
|
And "[data-notification]" has the "notification-update" class
|
|
|
|
|
And "[data-notification]" has the "success" class
|
2018-07-05 12:43:03 +00:00
|
|
|
Scenario: Update to a key when the value is empty
|
|
|
|
|
And 1 kv model from yaml
|
|
|
|
|
---
|
2020-02-05 09:37:45 +00:00
|
|
|
Key: key
|
|
|
|
|
Value: ~
|
|
|
|
|
Flags: 12
|
2018-07-05 12:43:03 +00:00
|
|
|
---
|
|
|
|
|
When I visit the kv page for yaml
|
|
|
|
|
---
|
|
|
|
|
dc: datacenter
|
|
|
|
|
kv: key
|
|
|
|
|
---
|
|
|
|
|
Then the url should be /datacenter/kv/key/edit
|
|
|
|
|
And I submit
|
ui: Adds Partitions to the HTTP layer (#10447)
This PR mainly adds partition to our HTTP adapter. Additionally and perhaps most importantly, we've also taken the opportunity to move our 'conditional namespaces' deeper into the app.
The reason for doing this was, we like that namespaces should be thought of as required instead of conditional, 'special' things and would like the same thinking to be applied to partitions.
Now, instead of using code throughout the app throughout the adapters to add/remove namespaces or partitions depending on whether they are enabled or not. As a UI engineer you just pretend that namespaces and partitions are always enabled, and we remove them for you deeper in the app, out of the way of you forgetting to treat these properties as a special case.
Notes:
Added a PartitionAbility while we were there (not used as yet)
Started to remove the CONSTANT variables we had just for property names. I prefer that our adapters are as readable and straightforwards as possible, it just looks like HTTP.
We'll probably remove our formatDatacenter method we use also at some point, it was mainly too make it look the same as our previous formatNspace, but now we don't have that, it instead now looks different!
We enable parsing of partition in the UIs URL, but this is feature flagged so still does nothing just yet.
All of the test changes were related to the fact that we were treating client.url as a function rather than a method, and now that we reference this in client.url (etc) it needs binding to client.
2021-09-15 17:09:55 +00:00
|
|
|
Then a PUT request was made to "/v1/kv/key?dc=datacenter&ns=@!namespace&flags=12" with no body
|
2018-08-29 18:14:31 +00:00
|
|
|
Then the url should be /datacenter/kv
|
|
|
|
|
And "[data-notification]" has the "notification-update" class
|
|
|
|
|
And "[data-notification]" has the "success" class
|
|
|
|
|
Scenario: There was an error saving the key
|
|
|
|
|
When I visit the kv page for yaml
|
|
|
|
|
---
|
|
|
|
|
dc: datacenter
|
|
|
|
|
kv: key
|
|
|
|
|
---
|
|
|
|
|
Then the url should be /datacenter/kv/key/edit
|
|
|
|
|
|
|
|
|
|
Given the url "/v1/kv/key" responds with a 500 status
|
|
|
|
|
And I submit
|
|
|
|
|
Then the url should be /datacenter/kv/key/edit
|
|
|
|
|
Then "[data-notification]" has the "notification-update" class
|
|
|
|
|
And "[data-notification]" has the "error" class
|
2018-06-04 13:53:51 +00:00
|
|
|
@ignore
|
|
|
|
|
Scenario: KV's with spaces are saved correctly
|
|
|
|
|
Then ok
|
|
|
|
|
@ignore
|
|
|
|
|
Scenario: KV's with returns are saved correctly
|
|
|
|
|
Then ok
|