Fix composite source encoding
This commit is contained in:
parent
c9fdbb55f9
commit
b482cdcc99
2 changed files with 32 additions and 1 deletions
|
@ -77,7 +77,9 @@ var Bing = function (options) {
|
||||||
+ "&$top=" + opts.top
|
+ "&$top=" + opts.top
|
||||||
+ "&$skip=" + opts.skip
|
+ "&$skip=" + opts.skip
|
||||||
+ "&Options=%27" + (opts.options || []).join('%2B') + "%27"
|
+ "&Options=%27" + (opts.options || []).join('%2B') + "%27"
|
||||||
+ (opts.sources ? "&Sources=%27" + opts.sources + "%27" : '')
|
+ (opts.sources
|
||||||
|
? "&Sources=%27" + encodeURIComponent(opts.sources) + "%27"
|
||||||
|
: '')
|
||||||
+ (opts.newsSortBy ? "&NewsSortBy=%27" + opts.newsSortBy + "%27" : '')
|
+ (opts.newsSortBy ? "&NewsSortBy=%27" + opts.newsSortBy + "%27" : '')
|
||||||
+ (opts.newsCategory ? "&NewsCategory=%27" + opts.newsCategory + "%27" : '')
|
+ (opts.newsCategory ? "&NewsCategory=%27" + opts.newsCategory + "%27" : '')
|
||||||
+ (opts.newsLocationOverride
|
+ (opts.newsLocationOverride
|
||||||
|
|
|
@ -136,6 +136,35 @@ describe("Bing News", function () {
|
||||||
});
|
});
|
||||||
|
|
||||||
|
|
||||||
|
describe("Bing Composite", function () {
|
||||||
|
|
||||||
|
this.timeout(1000 * 10);
|
||||||
|
|
||||||
|
it('finds composite search', function (done) {
|
||||||
|
|
||||||
|
Bing.composite('animal',
|
||||||
|
{
|
||||||
|
sources: 'web+news',
|
||||||
|
skip: 1,
|
||||||
|
newSortBy: 'Date'
|
||||||
|
},
|
||||||
|
function (err, res, body) {
|
||||||
|
|
||||||
|
//TODO try unaccepted options like imageFilters
|
||||||
|
|
||||||
|
should.not.exist(err);
|
||||||
|
should.exist(res);
|
||||||
|
should.exist(body);
|
||||||
|
|
||||||
|
body.d.results.should.have.length(1);
|
||||||
|
|
||||||
|
done();
|
||||||
|
});
|
||||||
|
});
|
||||||
|
|
||||||
|
});
|
||||||
|
|
||||||
|
|
||||||
describe("Bing Video", function () {
|
describe("Bing Video", function () {
|
||||||
|
|
||||||
this.timeout(1000 * 10);
|
this.timeout(1000 * 10);
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue