Add MockPost class

It exposes more options for mocking post requests.
embermap · May 19, 2019 · 0960bf7 · 0960bf7
1 parent 61065f5
commit 0960bf7
Show file tree

Hide file tree

Showing 4 changed files with 50 additions and 24 deletions.
diff --git a/addon-test-support/-private/mock-server.js b/addon-test-support/-private/mock-server.js
@@ -1,42 +1,63 @@
 import { fetch } from 'whatwg-fetch';
 
-let createMock = async function(path, method, statusCode, response) {
+let createMock = async function({ path, method, body, statusCode, response }) {
   return await fetch('/__mock-request', {
     method: 'post',
     headers: {
       "Content-Type": "application/json",
     },
     body: JSON.stringify({
       path,
+      body,
       method,
       statusCode,
       response
     }),
   });
 }
 
-export let mockServer = {
-  async get(path, response, status = 200) {
-    return createMock(path, "GET", status, response);
-  },
-
-  async post(path, response, status = 200) {
-    return createMock(path, "POST", status, response);
-  },
-
-  async patch(path, response, status = 200) {
-    return createMock(path, "PATCH", status, response);
-  },
-
-  async put(path, response, status = 200) {
-    return createMock(path, "PUT", status, response);
+let mockServer = {
+  get(path, response, statusCode = 200) {
+    // when needed we'll change this to use something like MockGet
+    // see MockPost for the idea
+    return createMock({
+      path,
+      method: "GET",
+      statusCode,
+      response
+    });
   },
 
-  async delete(path, response, status = 200) {
-    return createMock(path, "DELETE", status, response);
+  post(path, body = {}) {
+    return new MockPost({ path, body });
   },
 
   async cleanUp() {
     return fetch('/__cleanup-mocks');
   }
 };
+
+class MockPost {
+  constructor({ path, body }) {
+    this.path = path;
+    this.body = body;
+    this.status = 200;
+  }
+
+  statusCode(status) {
+    this.status = status;
+    return this;
+  }
+
+  reply(response) {
+    return createMock({
+      path: this.path,
+      body: this.body,
+      method: "POST",
+      statusCode: this.status,
+      response
+    });
+  }
+}
+
+export { mockServer };
diff --git a/tests/dummy/app/pods/examples/network/other/get-request/route.js b/tests/dummy/app/pods/examples/network/other/get-request/route.js
@@ -4,7 +4,7 @@ import fetch from 'fetch';
 export default Route.extend({
 
   async model() {
-    let response = await fetch('/api/posts', { method: 'post' });
+    let response = await fetch('/api/posts');
     return await response.json();
   }
 

diff --git a/tests/fastboot/network-mocking-test.js b/tests/fastboot/network-mocking-test.js
@@ -68,7 +68,7 @@ module('Fastboot | network mocking', function(hooks) {
   });
 
   test('it can mock a get request', async function(assert) {
-    await mockServer.post('/api/posts', [
+    await mockServer.get('/api/posts', [
       { id: 1, title: 'get post'},
     ]);
 
@@ -78,9 +78,14 @@ module('Fastboot | network mocking', function(hooks) {
   });
 
   test('it can mock a post request', async function(assert) {
-    await mockServer.post('/api/posts', [
-      { id: 1, title: 'post post'},
-    ]);
+    await mockServer
+      .post('/api/posts')
+      .reply([
+        {
+          id: 1,
+          title: 'post post'
+        }
+      ]);
 
     await visit('/examples/network/other/post-request');
 

diff --git a/tests/index.html b/tests/index.html
@@ -17,7 +17,7 @@
     {{content-for "head-footer"}}
     {{content-for "test-head-footer"}}
   </head>
-  <body>
+  <body style="background-color: white !important;">
     {{content-for "body"}}
     {{content-for "test-body"}}