luxolus
/
open-consul
2
0
Fork 0
Code Issues 1 Pull Requests Packages Releases Wiki Activity

ui: Change partitions to expect `[]` from the API (#11791)

This commit is contained in:
John Cowen 2021-12-10 14:41:08 +00:00 committed by GitHub
parent 374be91fa6
commit c6c1b9f13a
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
4 changed files with 29 additions and 28 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
.changelog/11791.txt Normal file
View File

@ -0,0 +1,3 @@
```release-note:bugfix
ui: Change partitions to expect [] from the listing API
```

2
ui/packages/consul-ui/app/serializers/partition.js
View File

@ -11,7 +11,7 @@ export default class PartitionSerializer extends Serializer {
respond((headers, body) => { respond((headers, body) => {
return cb( return cb(
headers, headers,
body.Partitions.map(item => { body.map(item => {
item.Partition = '*'; item.Partition = '*';
item.Namespace = '*'; item.Namespace = '*';
return item; return item;

12
ui/packages/consul-ui/mock-api/v1/partitions
View File

@ -1,5 +1,4 @@
{ [
"Partitions": [
{ {
"Name": "default", "Name": "default",
"Description": "Builtin Default Partition" "Description": "Builtin Default Partition"
@ -19,12 +18,11 @@ ${i === 0 ? `
` : ``} ` : ``}
{ {
"Name": "${fake.hacker.noun()}-partition-${i}", "Name": "${fake.hacker.noun()}-partition-${i}",
${fake.random.boolean() ? ` ${fake.random.boolean() ? `
"Description": "${fake.lorem.sentence()}", "Description": "${fake.lorem.sentence()}",
` : ``} ` : ``}
"CreateIndex": 12, "CreateIndex": 12,
"ModifyIndex": 16 "ModifyIndex": 16
} }
`)} `)}
] ]
}

2
ui/packages/consul-ui/tests/integration/serializers/partition-test.js
View File

@ -13,7 +13,7 @@ module('Integration | Serializer | partition', function(hooks) {
url: `/v1/partitions?dc=${dc}`, url: `/v1/partitions?dc=${dc}`,
}; };
return get(request.url).then(function(payload) { return get(request.url).then(function(payload) {
const expected = payload.Partitions.map(item => const expected = payload.map(item =>
Object.assign({}, item, { Object.assign({}, item, {
Datacenter: dc, Datacenter: dc,
Namespace: '*', Namespace: '*',