Properly use angular.toJson for msearch bodies
This commit is contained in:
parent
bd7dc1a1fe
commit
e783fb7401
2 changed files with 10 additions and 2 deletions
|
@ -20,7 +20,7 @@ describe('ui/courier/fetch/strategy/search', () => {
|
||||||
index: ['logstash-123'],
|
index: ['logstash-123'],
|
||||||
type: 'blah',
|
type: 'blah',
|
||||||
search_type: 'blah2',
|
search_type: 'blah2',
|
||||||
body: 'hm this is the body'
|
body: { foo: 'bar', $foo: 'bar' }
|
||||||
}
|
}
|
||||||
];
|
];
|
||||||
}));
|
}));
|
||||||
|
@ -32,6 +32,14 @@ describe('ui/courier/fetch/strategy/search', () => {
|
||||||
});
|
});
|
||||||
|
|
||||||
describe('#reqsFetchParamsToBody()', () => {
|
describe('#reqsFetchParamsToBody()', () => {
|
||||||
|
it('filters out any body properties that begin with $', () => {
|
||||||
|
let value;
|
||||||
|
search.reqsFetchParamsToBody(reqsFetchParams).then(val => value = val);
|
||||||
|
$rootScope.$apply();
|
||||||
|
expect(_.includes(value, 'foo')).to.be(true);
|
||||||
|
expect(_.includes(value, '$foo')).to.be(false);
|
||||||
|
});
|
||||||
|
|
||||||
context('when indexList is not empty', () => {
|
context('when indexList is not empty', () => {
|
||||||
it('includes the index', () => {
|
it('includes the index', () => {
|
||||||
let value;
|
let value;
|
||||||
|
|
|
@ -43,7 +43,7 @@ define(function (require) {
|
||||||
ignore_unavailable: true
|
ignore_unavailable: true
|
||||||
})
|
})
|
||||||
+ '\n'
|
+ '\n'
|
||||||
+ toJson(fetchParams.body || {}, angular.json);
|
+ toJson(fetchParams.body || {}, angular.toJson);
|
||||||
});
|
});
|
||||||
})
|
})
|
||||||
.then(function (requests) {
|
.then(function (requests) {
|
||||||
|
|
Loading…
Reference in a new issue