refactoring and standard eslint

Author: pofider

.eslintrc

@@ -0,0 +1,3 @@
+{
+  "extends": ["standard"]
+}

.npmrc

@@ -0,0 +1 @@
+save-exact=true

index.js

@@ -1,5 +1,4 @@
 /*!
  * Copyright(c) 2014 Jan Blaha
  */
-
-module.exports = require("./lib/client.js");
+module.exports = require('./lib/client.js')

lib/client.js

@@ -5,80 +5,79 @@
  * able to render reports or do crud using jaydata context.
  */
 
-var request = require("request"),
-    url = require("url"),
-    concat = require('concat-stream');
+var request = require('request')
+var url = require('url')
+var concat = require('concat-stream')
+var assign = require('object-assign')
 
 module.exports = function (url, username, password) {
-    return new Client(url, username, password);
-};
+  return new Client(url, username, password)
+}
 
 var Client = function (url, username, password) {
-    this.url = url;
-    this.username = username;
-    this.password = password;
+  this.url = url
+  this.username = username
+  this.password = password
 }
 
 /**
  * Render report in temore server and return response
  * @returns object containing header property with response headers and body property with response body
  */
 Client.prototype.render = function (req, options, cb) {
-
-    if (typeof cb == 'undefined') {
-        cb = options;
-        options = {}
+  if (typeof cb === 'undefined') {
+    cb = options
+    options = {}
+  }
+
+  options = assign({
+    uri: url.resolve(this.url, 'api/report'),
+    body: JSON.stringify(req),
+    strictSSL: false,
+    headers: {
+      'Content-Type': 'application/json'
     }
+  }, options)
 
-
-    options.uri = options.uri || url.resolve(this.url, 'api/report');
-    options.body = options.body || JSON.stringify(req);
-    options.strictSSL = options.strictSSL || false;
-    options.encoding = options.encoding || null;
-    options.headers = options.headers || {
-            'Content-Type': 'application/json'
-        };
-
-
-    if (this.username) {
-        options.auth = {
-            username: this.username,
-            password: this.password
-        }
+  if (this.username) {
+    options.auth = {
+      username: this.username,
+      password: this.password
     }
-
-    var responseStream = request.post(options);
-
-    responseStream.on("error", function(err) {
-        cb(err);
-    });
-
-    responseStream.on("response", function (response) {
-        response.on("error", function (err) {
-            cb(err)
-        })
-
-        if (response.statusCode !== 200) {
-            return responseToBuffer(response, function(data) {
-                try {
-                    var errorMessage = JSON.parse(data.toString());
-                    var error = new Error(errorMessage.message);
-                    error.remoteStack = errorMessage.stack;
-                    cb(error)
-                }
-                catch(e) {
-                    cb(new Error(data));
-                }
-            });
+  }
+
+  var responseStream = request.post(options)
+
+  responseStream.on('error', function (err) {
+    cb(err)
+  })
+
+  responseStream.on('response', function (response) {
+    response.on('error', function (err) {
+      cb(err)
+    })
+
+    if (response.statusCode !== 200) {
+      return responseToBuffer(response, function (data) {
+        try {
+          var errorMessage = JSON.parse(data.toString())
+          var error = new Error(errorMessage.message)
+          error.remoteStack = errorMessage.stack
+          cb(error)
+        } catch (e) {
+          var err = new Error('Unknown error, status code ' + response.statusCode)
+          err.response = response
+          cb(err)
         }
+      })
+    }
 
-        response.body = function(cb) { responseToBuffer(response, cb); }
-        cb(null, response);
-    });
-};
-
+    response.body = function (cb) { responseToBuffer(response, cb) }
+    cb(null, response)
+  })
+}
 
-function responseToBuffer(response, cb) {
-    var writeStream = concat(cb);
-    response.pipe(writeStream);
-}
+function responseToBuffer (response, cb) {
+  var writeStream = concat(cb)
+  response.pipe(writeStream)
+}

package.json

@@ -13,16 +13,22 @@
   "description": "Remote node.js client to ",
   "main": "index.js",
   "scripts": {
-    "test": "mocha --reporter spec test"
+    "test": "mocha --reporter spec test && standard"
   },
   "licence": "MIT",
   "devDependencies": {
-    "jsreport": "jsreport/jsreport",
-    "mocha": "^1.21.4",
-    "should": "^4.0.4"
+    "eslint": "3.1.0",
+    "jsreport-authentication": "1.0.3",
+    "jsreport-core": "1.0.5",
+    "jsreport-express": "1.0.2",
+    "jsreport-jsrender": "1.0.1",
+    "mocha": "2.5.3",
+    "should": "9.0.2",
+    "standard": "7.1.2"
   },
   "dependencies": {
-    "concat-stream": "^1.4.6",
-    "request": "^2.40.0"
+    "concat-stream": "1.5.1",
+    "lodash.assign": "4.0.9",
+    "request": "2.73.0"
   }
 }