Merge pull request #53 from Financial-Times/add-logging-in-getLicence…

…s-function

increase logs in getLicences function

lib/accessLicenceClient.js

@@ -21,17 +21,26 @@ function getLicences (query) {
 	const operation = 'accessLicenceClient.getLicences';
 	log.debug({operation, query: JSON.stringify(query)});
 
-	if (query.adminuserid || query.linkid || query.linktype || query.status || query.userid) {
-		const queryString = helpers.createParams(query);
-		const url = `${config.API_GATEWAY_HOST}/licences${queryString}`;
-		return fetch(url, Object.assign({}, options))
-			.then(res => helpers.parseJsonRes(res, `${operation} ${queryString}`))
-			.then(res => {
-				log.debug({operation, queryString, res: 'success'});
-				return res.accessLicences;
-			});
+	if (!(query.adminuserid || query.linkid || query.linktype || query.status || query.userid)) {
+		const error = new Error('At least one of adminuserid | linkid | linktype | status | userid must be specified');
+		return Promise.reject(error);
 	}
-	return Promise.reject(Error('At least one of adminuserid | linkid | linktype | status | userid must be specified'));
+
+	const queryString = helpers.createParams(query);
+	const url = `${config.API_GATEWAY_HOST}/licences${queryString}`;
+
+	log.info({operation, msg: 'Requesting licences user administrates form Access Licence Service', url });
+
+	return fetch(url, Object.assign({}, options))
+		.then(res => {
+			log.info({ operation, msg: `Recieved status ${res.status} from Access Licence Service` });
+			const message = `${operation} ${queryString}`;
+			return helpers.parseJsonRes(res, message);
+		})
+		.then(res => {
+			log.info({ operation, queryString, status: res.status, res: JSON.stringify(res) });
+			return res.accessLicences;
+		});
 }
 
 /**