Improve test coverage of headers

To bypass issues with supertest/superagent, I have added
a test helper and written a single test to prove headers
are working as expected

Author: digitalsadhu

package.json

@@ -4,7 +4,7 @@
   "description": "JSONAPI support for loopback",
   "main": "lib/index.js",
   "scripts": {
-    "test": "npm run lint && mocha --reporter=spec ./test/**/*",
+    "test": "npm run lint && mocha --reporter=spec ./test/**/*.test.js",
     "lint": "eslint ."
   },
   "repository": {

test/create.test.js

@@ -1,6 +1,7 @@
 var request = require('supertest');
 var loopback = require('loopback');
 var expect = require('chai').expect;
+var query = require('./util/query');
 var JSONAPIComponent = require('../');
 var app;
 var Post;
@@ -38,25 +39,35 @@ describe('loopback json api component create method', function () {
         .end(done);
     });
     it('POST /models should have the JSON API Content-Type header set on response', function (done) {
-      //TODO: superagent/supertest breaks when trying to use JSON API Content-Type header
+      var data = {
+        data: {
+          type: 'posts',
+          attributes: {
+            title: 'my post',
+            content: 'my post content'
+          }
+        }
+      };
+
+      var options = {
+        headers: {
+          'Accept': 'application/vnd.api+json',
+          'Content-Type': 'application/vnd.api+json'
+        }
+      };
+
+      //use http module via util to make this post to check headers are working since
+      //superagent/supertest breaks when trying to use JSON API Content-Type header
       //waiting on a fix
       //see https://github.com/visionmedia/superagent/issues/753
-      //using Content-Type: application/json in the mean time.
-      //Have tested correct header using curl and all is well
-      // request(app).post('/posts')
-      //   .send({
-      //     data: {
-      //       type: 'posts',
-      //       attributes: {
-      //         title: 'my post',
-      //         content: 'my post content'
-      //       }
-      //     }
-      //   })
-      //   .set('Content-Type', 'application/vnd.api+json')
-      //   .expect('Content-Type', 'application/vnd.api+json; charset=utf-8')
-      //   .end(done);
-      done();
+      query(app).post('/posts', data, options, function (err, res) {
+        if (err) console.log(err);
+        expect(res.headers['content-type']).to.match(/application\/vnd\.api\+json/);
+        expect(res.statusCode).to.equal(201);
+        expect(res.body).to.have.all.keys('data');
+        expect(res.body.data).to.have.all.keys('type', 'id', 'links', 'attributes');
+        done();
+      });
     });
 
     it('POST /models should have the Location header set on response', function (done) {

test/util/query.js

@@ -0,0 +1,32 @@
+var http = require('http');
+
+module.exports = function (app) {
+  return {
+    post: function (path, data, options, cb) {
+      var s = app.listen(function () {
+        var req = http.request({
+          port: s.address().port,
+          path: path,
+          headers: options.headers || {},
+          method: 'POST'
+        }, function (res) {
+          res.setEncoding('utf8');
+          res.rawData = '';
+          res.on('data', function (chunk) {
+            res.rawData += chunk;
+          });
+          res.on('end', function () {
+            res.body = JSON.parse(res.rawData);
+            cb(null, res);
+          });
+        });
+        req.on('error', function (err) {
+          cb(err);
+        });
+        var postData = JSON.stringify(data || {});
+        req.write(postData);
+        req.end();
+      });
+    }
+  };
+};