Merge pull request #94 from PlayNetwork/feature/bms-370

BMS-370: Add ability to get profile based on authentication headers

lib/provision.js

@@ -7,6 +7,7 @@ var
 	validation = require('./validation');
 
 const
+	CLIENT_ID = 'x-client-id',
 	DEFAULT_HOST = 'provision-api.apps.playnetwork.com',
 	DEFAULT_SECURE = true,
 	EVENT_ERROR = 'error',
@@ -364,12 +365,13 @@ module.exports = function (provisionOptions, ensureAuthHeaders, self) {
 		}
 
 		let exec = co(function *() {
+			let headers = yield ensureAuthHeaders();
+
 			if (validation.isEmpty(profileAlias)) {
-				return yield Promise.reject(new Error('profileAlias is required'));
+				const clientId = headers[CLIENT_ID];
+				profileAlias = `clientId:${clientId}`;
 			}
 
-			let headers = yield ensureAuthHeaders();
-
 			return yield req.get({
 				headers : headers,
 				pathname : `/v2/profiles/${profileAlias}`,

package.json

@@ -1,6 +1,6 @@
 {
   "name": "playnetwork-sdk",
-  "version": "1.4.6",
+  "version": "1.4.7",
   "contributors": [
     {
       "name": "Joshua Thomas",

test/lib/provision.js

@@ -550,27 +550,30 @@ describe('provision', () => {
 	});
 
 	describe('#getProfile', () => {
-		it('should require a profileAlias (promise)', (done) => {
-			provision.getProfile()
-				.then(() => {
-					return done(new Error('should require profileAlias'));
-				})
-				.catch((err) => {
-					should.exist(err);
-					should.exist(err.message);
-					err.message.should.contain('profileAlias is required');
+		it('should properly retrieve profile even without profileAlias', function (done) {
+			// intercept outbound request
+			nock('https://provision-api.apps.playnetwork.com')
+				.get('/v2/profiles/clientId:test')
+				.reply(200, { total : 0 });
 
+			provision.getProfile()
+				.then(function () {
 					return done();
-				});
+				})
+				.catch(function (err) {
+					return done(err);
+				})
 		});
 
-		it('should require profileAlias (callback)', (done) => {
-			provision.getProfile(function (err, result) {
-				should.exist(err);
-				should.exist(err.message);
-				err.message.should.contain('profileAlias is required');
-				should.not.exist(result);
+		it('should properly retrieve profile even without profileAlias (callback)', function (done) {
+			// intercept outbound request
+			nock('https://provision-api.apps.playnetwork.com')
+				.get('/v2/profiles/clientId:test')
+				.reply(200, { total : 0 });
 
+			provision.getProfile(function (err, result) {
+				should.not.exist(err);
+				should.exist(result);
 				return done();
 			});
 		});