AlchemyLanguage - Sentiment Analysis - Add support for targets #155

Author: germanattanasio

services/alchemy_language/v1.js

@@ -17,6 +17,7 @@
 'use strict';
 
 var extend         = require('extend');
+var util           = require('util');
 var requestFactory = require('../../lib/requestwrapper');
 var endpoints      = require('../../lib/alchemy_endpoints.json');
 var helper         = require('../../lib/helper');
@@ -93,8 +94,12 @@ AlchemyLanguage.prototype.concepts = createRequest('concepts');
  * Calculates the sentiment for text, a URL or HTML.
  */
 AlchemyLanguage.prototype.sentiment = function(params, callback) {
-  var service = (params && (params.target || params.targets )) ? 'sentiment_targeted' : 'sentiment';
-  return createRequest(service).call(this, params, callback);
+  var _params = extend({}, params);
+  var service = (params.target || params.targets) ? 'sentiment_targeted' : 'sentiment';
+  if (util.isArray(_params.targets))
+    _params.targets = _params.targets.join('|');
+
+  return createRequest(service).call(this, _params, callback);
 };
 /**
  * Extracts the cleaned text (removes ads, navigation, etc.) for a URL or HTML.

test/test.integration-all-services.js

@@ -450,13 +450,20 @@ describe('integration-all-services', function() {
       }, failIfError.bind(failIfError, done));
     });
 
-    it('sentiment_multiple_targets()', function(done) {
+    it('sentiment_multiple_targets_with_pipe()', function(done) {
       alchemy_language.sentiment({
         text: text,
         targets: 'United States|Peter Higgs'
       }, failIfError.bind(failIfError, done));
     });
 
+    it('sentiment_multiple_targets_with_array()', function(done) {
+      alchemy_language.sentiment({
+        text: text,
+        targets: ['United States','Peter Higgs']
+      }, failIfError.bind(failIfError, done));
+    });
+
     it('category()', function(done) {
       alchemy_language.category({
         text: text