Merge pull request #49 from kuzzleio/kuz-345-sdk-add-login-method

bugfix: login doesn't store jwtToken / I've tested this PR and it works. Going to merge.

Author: xbill82

dist/kuzzle.js

@@ -602,7 +602,34 @@ Kuzzle.prototype.connect = function () {
   return this;
 };
 
+/**
+ * Set the jwtToken used to query kuzzle
+ * @param token
+ * @returns {Kuzzle}
+ */
+Kuzzle.prototype.setJwtToken = function(token) {
+  this.jwtToken = token;
+  return this;
+};
+
+/**
+ * Get the jwtToken used by kuzzle
+ * @returns {Kuzzle}
+ */
+Kuzzle.prototype.getJwtToken = function() {
+  return this.jwtToken;
+};
 
+/**
+ * Send login request to kuzzle with credentials
+ * If login success, store the jwtToken into kuzzle object
+ *
+ * @param strategy
+ * @param credentials
+ * @param expiresIn
+ * @param cb
+ * @returns {Kuzzle}
+ */
 Kuzzle.prototype.login = function (strategy, credentials, expiresIn, cb) {
   var
     self = this,
@@ -625,7 +652,7 @@ Kuzzle.prototype.login = function (strategy, credentials, expiresIn, cb) {
 
   this.query({controller: 'auth', action: 'login'}, {body: request}, {}, function(error, response) {
     if (error === null) {
-      self.jwtToken = response.jwt;
+      self.setJwtToken(response.result.jwt);
       renewAllSubscriptions.call(self);
 
       if (typeof cb === 'function') {
@@ -643,6 +670,12 @@ Kuzzle.prototype.login = function (strategy, credentials, expiresIn, cb) {
   return self;
 };
 
+/**
+ * Send logout request to kuzzle with jwtToken.
+ *
+ * @param cb
+ * @returns {Kuzzle}
+ */
 Kuzzle.prototype.logout = function (cb) {
   var
     self = this,
@@ -655,7 +688,7 @@ Kuzzle.prototype.logout = function (cb) {
 
   this.query({controller: 'auth', action: 'logout'}, request, {}, function(error) {
     if (error === null) {
-      self.jwtToken = undefined;
+      self.setJwtToken(undefined);
 
       if (typeof cb === 'function') {
         cb(null, self);

dist/kuzzle.min.js

@@ -327,7 +327,34 @@ Kuzzle.prototype.connect = function () {
   return this;
 };
 
+/**
+ * Set the jwtToken used to query kuzzle
+ * @param token
+ * @returns {Kuzzle}
+ */
+Kuzzle.prototype.setJwtToken = function(token) {
+  this.jwtToken = token;
+  return this;
+};
+
+/**
+ * Get the jwtToken used by kuzzle
+ * @returns {Kuzzle}
+ */
+Kuzzle.prototype.getJwtToken = function() {
+  return this.jwtToken;
+};
 
+/**
+ * Send login request to kuzzle with credentials
+ * If login success, store the jwtToken into kuzzle object
+ *
+ * @param strategy
+ * @param credentials
+ * @param expiresIn
+ * @param cb
+ * @returns {Kuzzle}
+ */
 Kuzzle.prototype.login = function (strategy, credentials, expiresIn, cb) {
   var
     self = this,
@@ -350,7 +377,7 @@ Kuzzle.prototype.login = function (strategy, credentials, expiresIn, cb) {
 
   this.query({controller: 'auth', action: 'login'}, {body: request}, {}, function(error, response) {
     if (error === null) {
-      self.jwtToken = response.jwt;
+      self.setJwtToken(response.result.jwt);
       renewAllSubscriptions.call(self);
 
       if (typeof cb === 'function') {
@@ -368,6 +395,12 @@ Kuzzle.prototype.login = function (strategy, credentials, expiresIn, cb) {
   return self;
 };
 
+/**
+ * Send logout request to kuzzle with jwtToken.
+ *
+ * @param cb
+ * @returns {Kuzzle}
+ */
 Kuzzle.prototype.logout = function (cb) {
   var
     self = this,
@@ -380,7 +413,7 @@ Kuzzle.prototype.logout = function (cb) {
 
   this.query({controller: 'auth', action: 'logout'}, request, {}, function(error) {
     if (error === null) {
-      self.jwtToken = undefined;
+      self.setJwtToken(undefined);
 
       if (typeof cb === 'function') {
         cb(null, self);

dist/kuzzle.min.map

@@ -708,7 +708,7 @@ describe('Kuzzle constructor', () => {
         });
 
         kuzzle.query = function(queryArgs, query, options, cb) {
-          cb(null, { jwt: 'test-toto'});
+          cb(null, { result: {jwt: 'test-toto'}});
         };
 
         kuzzle.login('local', loginCredentials, '1h', function(error, k) {