Skip to content

Commit 69c91d7

Browse files
fix: pr and test fixes
1 parent 93664e1 commit 69c91d7

File tree

5 files changed

+18
-25
lines changed

5 files changed

+18
-25
lines changed

lib-es5/api.js

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -384,13 +384,6 @@ exports.sub_folders = function sub_folders(path, callback) {
384384
return call_api("get", uri, params, callback, options);
385385
};
386386

387-
exports.search_folders = function search_folders(params, callback) {
388-
var options = arguments.length > 2 && arguments[2] !== undefined ? arguments[2] : {};
389-
390-
options.content_type = 'json';
391-
return call_api("post", "folders/search", params, callback, options);
392-
};
393-
394387
/**
395388
* Creates an empty folder
396389
*
@@ -551,6 +544,13 @@ exports.search = function search(params, callback) {
551544
return call_api("post", "resources/search", params, callback, options);
552545
};
553546

547+
exports.search_folders = function search_folders(params, callback) {
548+
var options = arguments.length > 2 && arguments[2] !== undefined ? arguments[2] : {};
549+
550+
options.content_type = 'json';
551+
return call_api("post", "folders/search", params, callback, options);
552+
};
553+
554554
exports.update_resources_access_mode_by_prefix = function update_resources_access_mode_by_prefix(access_mode, prefix, callback) {
555555
var options = arguments.length > 3 && arguments[3] !== undefined ? arguments[3] : {};
556556

lib-es5/v2/api.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,6 @@ v1_adapters(exports, api, {
3636
create_upload_preset: 0,
3737
root_folders: 0,
3838
sub_folders: 1,
39-
search_folders: 1,
4039
delete_folder: 1,
4140
create_folder: 1,
4241
upload_mappings: 0,
@@ -56,6 +55,7 @@ v1_adapters(exports, api, {
5655
update_resources_access_mode_by_tag: 2,
5756
update_resources_access_mode_by_ids: 2,
5857
search: 1,
58+
search_folders: 1,
5959
delete_derived_by_transformation: 2,
6060
add_metadata_field: 1,
6161
list_metadata_fields: 1,

lib/api.js

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -284,11 +284,6 @@ exports.sub_folders = function sub_folders(path, callback, options = {}) {
284284
return call_api("get", uri, params, callback, options);
285285
};
286286

287-
exports.search_folders = function search_folders(params, callback, options = {}) {
288-
options.content_type = 'json';
289-
return call_api("post", "folders/search", params, callback, options);
290-
}
291-
292287
/**
293288
* Creates an empty folder
294289
*
@@ -409,6 +404,11 @@ exports.search = function search(params, callback, options = {}) {
409404
return call_api("post", "resources/search", params, callback, options);
410405
};
411406

407+
exports.search_folders = function search_folders(params, callback, options = {}) {
408+
options.content_type = 'json';
409+
return call_api("post", "folders/search", params, callback, options);
410+
};
411+
412412
exports.update_resources_access_mode_by_prefix = function update_resources_access_mode_by_prefix(
413413
access_mode,
414414
prefix,

lib/v2/api.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,6 @@ v1_adapters(exports, api, {
3434
create_upload_preset: 0,
3535
root_folders: 0,
3636
sub_folders: 1,
37-
search_folders: 1,
3837
delete_folder: 1,
3938
create_folder: 1,
4039
upload_mappings: 0,
@@ -54,6 +53,7 @@ v1_adapters(exports, api, {
5453
update_resources_access_mode_by_tag: 2,
5554
update_resources_access_mode_by_ids: 2,
5655
search: 1,
56+
search_folders: 1,
5757
delete_derived_by_transformation: 2,
5858
add_metadata_field: 1,
5959
list_metadata_fields: 1,

test/integration/api/search/search_folders_spec.js

Lines changed: 4 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
const cloudinary = require('../../../../cloudinary');
22
const {TIMEOUT} = require('../../../testUtils/testConstants');
33
const describe = require('../../../testUtils/suite');
4+
const wait = require('../../../testUtils/helpers/wait');
45

56
const folderNames = ['testFolder1', 'testFolder2'];
67

@@ -51,7 +52,7 @@ describe('search_folders_api', function () {
5152
before(function () {
5253
return Promise.all(folderNames.map(folderName => {
5354
return cloudinary.v2.api.create_folder(folderName);
54-
}));
55+
})).then(wait(2));
5556
});
5657

5758
after(function () {
@@ -60,19 +61,11 @@ describe('search_folders_api', function () {
6061
}));
6162
});
6263

63-
it('should return all folders prefixed with testFolder', function () {
64+
it('should return a search response with folders', function () {
6465
return cloudinary.v2.search_folders.expression('name=testFolder*')
6566
.execute()
6667
.then(function (results) {
67-
expect(results.folders.length).to.eql(2);
68-
});
69-
});
70-
71-
it('should allow search by exact folder name', function () {
72-
return cloudinary.v2.search_folders.expression('name=testFolder1')
73-
.execute()
74-
.then(function (results) {
75-
expect(results.folders.length).to.eql(1);
68+
expect(results).to.have.key('folders');
7669
});
7770
});
7871
});

0 commit comments

Comments
 (0)