Skip to content

Commit

Permalink
Merge pull request #87 from PlayNetwork/v2.2.2
Browse files Browse the repository at this point in the history
v2.2.2 - Added support for filtering out assets that don't exist
  • Loading branch information
tsmith1985 committed Mar 8, 2019
2 parents e57046a + 8d0a266 commit 07eedd7
Show file tree
Hide file tree
Showing 4 changed files with 55 additions and 1 deletion.
4 changes: 4 additions & 0 deletions history.md
Original file line number Diff line number Diff line change
@@ -1,3 +1,7 @@
# v2.2.2 - 2019/03/06

* Added support for filtering out assets that don't exist

# v2.2.2 - 2019/03/01

* Adjusted playback module to accept `isrc` or `trackId` as a content identifier
Expand Down
23 changes: 23 additions & 0 deletions lib/asset.js
Original file line number Diff line number Diff line change
Expand Up @@ -207,6 +207,29 @@ module.exports = function (assetOptions, ensureAuthHeaders, self) {
return validation.promiseOrCallback(exec, callback);
};

self.filterAssets = (assets, callback) => {
// handle any non-specified input params
if (typeof assets === 'function') {
callback = assets;
original = undefined;
}

let exec = co(function *() {
if (validation.isEmpty(assets)) {
return yield Promise.reject(new Error('assets are required'));
}

let headers = yield ensureAuthHeaders();

return yield req.post({
headers : headers,
pathname : '/v1/assets/bulk'
}, assets);
});

return validation.promiseOrCallback(exec, callback);
};

self.getAssetStream = (asset, options, callback) => {
// handle any non-specified input params
if (typeof options === 'function') {
Expand Down
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "playnetwork-sdk",
"version": "2.2.1",
"version": "2.2.2",
"contributors": [
{
"name": "Joshua Thomas",
Expand Down
27 changes: 27 additions & 0 deletions test/lib/asset.js
Original file line number Diff line number Diff line change
Expand Up @@ -272,6 +272,33 @@ describe('asset', function () {
});
});

describe('#filterAssets', function (done) {
it('should require assets (promise)', function (done) {
asset.filterAssets()
.then(() => {
return done(new Error('should require assets'));
})
.catch((err) => {
should.exist(err);
should.exist(err.message);
err.message.should.contain('assets are required');

return done();
});
});

it('should require assets (callback)', function (done) {
asset.filterAssets(function (err, result) {
should.exist(err);
should.exist(err.message);
err.message.should.contain('assets are required');
should.not.exist(result);

return done();
});
});
});

describe('#getAssetStream', function () {
it('should require asset (promise)', function (done) {
asset.getAssetStream()
Expand Down

0 comments on commit 07eedd7

Please sign in to comment.