Add fetch test

Author: pik

fetch/karma.conf.js

@@ -52,6 +52,7 @@ module.exports = function (config) {
     browsers: browsers,
     files: [
       'node_modules/babel-polyfill/dist/polyfill.js',
+      'node_modules/whatwg-fetch/fetch.js',
       'node_modules/js-data/dist/js-data.js',
       'fetch/dist/js-data-fetch.js',
       'fetch/karma.start.js',

fetch/package.json

@@ -18,8 +18,7 @@
     "axios",
     "rest",
     "adapter",
-    "http",
-    "fetch"
+    "http"
   ],
   "dependencies": {
     "js-data-adapter": "~0.8.1"

karma.conf.js

@@ -52,6 +52,7 @@ module.exports = function (config) {
     browsers: browsers,
     files: [
       'node_modules/babel-polyfill/dist/polyfill.js',
+      'node_modules/whatwg-fetch/fetch.js',
       'node_modules/js-data/dist/js-data.js',
       'dist/js-data-http.js',
       'karma.start.js',

karma.start.js

@@ -1,4 +1,5 @@
 /* global JSData:true, JSDataHttp:true, sinon:true, chai:true */
+
 before(function () {
   var Test = this
   Test.fail = function (msg) {

node/mocha.start.js

@@ -1,5 +1,6 @@
 'use strict'
 
+var { Headers, Request, fetch, Response } = require('whatwg-fetch') // eslint-disable-line
 var querystring = require('querystring')
 require('source-map-support').install()
 

package.json

@@ -30,9 +30,6 @@
   "standard": {
     "parser": "babel-eslint",
     "globals": [
-      "Headers",
-      "fetch",
-      "Request",
       "action",
       "describe",
       "it",
@@ -85,6 +82,7 @@
     "babel-plugin-transform-regenerator": "6.11.4",
     "babel-preset-stage-0": "6.5.0",
     "istanbul": "0.4.4",
+    "js-data-fetch": "file:fetch",
     "js-data-repo-tools": "0.5.6",
     "karma": "1.2.0",
     "karma-browserstack-launcher": "1.0.1",
@@ -95,6 +93,7 @@
     "phantomjs-prebuilt": "2.1.12",
     "rollup-plugin-commonjs": "3.3.1",
     "rollup-plugin-replace": "1.1.1",
-    "uglify-js": "2.7.0"
+    "uglify-js": "2.7.0",
+    "whatwg-fetch": "^1.0.0"
   }
 }

src/index.js

@@ -1,4 +1,4 @@
-/* global fetch:true Headers:true Request:true */
+/* global fetch:true Headers:true */
 
 import {utils} from 'js-data'
 import axios from '../node_modules/axios/dist/axios'
@@ -558,20 +558,19 @@ Adapter.extend({
    * @param {Object} config.headers Headers for the request.
    * @param {Object} config.params Querystring for the request.
    * @param {string} config.url Url for the request.
-   * @param {Object} [opts] Configuration options.
    */
-  fetch (config, opts) {
+  fetch (config) {
     const requestConfig = {
       method: config.method,
       // turn the plain headers object into the Fetch Headers object
-      headers: new Headers(config.headers)
+      headers: new Headers(config.headers || {})
     }
 
     if (config.data) {
       requestConfig.body = utils.toJson(config.data)
     }
 
-    return fetch(new Request(buildUrl(config.url, config.params), requestConfig))
+    return fetch(buildUrl(config.url, config.params), requestConfig)
       .then((response) => {
         response.config = {
           method: config.method,

test/fetch.test.js

@@ -1,3 +1,19 @@
 describe('fetch', function () {
-  it('should fetch')
+  it('should fetch from a URL', function (done) {
+    var Test = this
+
+    setTimeout(function () {
+      Test.requests[0].respond(200, { 'Content-Type': 'application/json' }, '{}')
+    }, 5)
+
+    Test.adapter.fetch({
+      method: 'get',
+      params: { active: true },
+      url: '/api/foos'
+    }).then(function (response) {
+      var request = Test.requests[0]
+      Test.assert.equal(request.method, 'GET')
+      Test.assert.equal(request.url, '/api/foos?active=true')
+      done()
+    })
 })