From d7e39dc54230428de4cfc33af72e6c538d844882 Mon Sep 17 00:00:00 2001 From: Neeko Date: Fri, 3 Jul 2020 12:04:17 +0200 Subject: [PATCH] On se simplifie la vie avec les mock, du moins on essaye --- package-lock.json | 17 +++++++++++++++++ package.json | 1 + test/test.js | 23 ++++++++++++----------- 3 files changed, 30 insertions(+), 11 deletions(-) diff --git a/package-lock.json b/package-lock.json index ced6282..9f48a26 100644 --- a/package-lock.json +++ b/package-lock.json @@ -1684,6 +1684,23 @@ "resolved": "https://registry.npmjs.org/negotiator/-/negotiator-0.6.2.tgz", "integrity": "sha512-hZXc7K2e+PgeI1eDBe/10Ard4ekbfrrqG8Ep+8Jmf4JID2bNg7NvCPOZN+kfF574pFQI7mum2AUqDidoKqcTOw==" }, + "node-mocks-http": { + "version": "1.8.1", + "resolved": "https://registry.npmjs.org/node-mocks-http/-/node-mocks-http-1.8.1.tgz", + "integrity": "sha512-qtd9YwXzCTdLfqjP7XSOtFei3TggwnjFIppmYEneQBaDIuknwgJTpItLskC5/pWOpU3lsK5aqdo+5CfIKHkXLg==", + "dev": true, + "requires": { + "accepts": "^1.3.7", + "depd": "^1.1.0", + "fresh": "^0.5.2", + "merge-descriptors": "^1.0.1", + "methods": "^1.1.2", + "mime": "^1.3.4", + "parseurl": "^1.3.3", + "range-parser": "^1.2.0", + "type-is": "^1.6.18" + } + }, "nodemon": { "version": "2.0.4", "resolved": "https://registry.npmjs.org/nodemon/-/nodemon-2.0.4.tgz", diff --git a/package.json b/package.json index 3bcb88b..e47f79d 100644 --- a/package.json +++ b/package.json @@ -17,6 +17,7 @@ "devDependencies": { "assert": "^2.0.0", "mocha": "^8.0.1", + "node-mocks-http": "^1.8.1", "nodemon": "^2.0.4", "prettier": "2.0.5" } diff --git a/test/test.js b/test/test.js index f6b203a..a6777cb 100644 --- a/test/test.js +++ b/test/test.js @@ -141,12 +141,18 @@ describe('APIController', () => { it("Doit retouner un code HTTP 404 si l'id est incorrect", () => { //Arrange - const error = { status: 404 } - const req = { - query: { - id: '9999999999', + var req = httpMocks.createRequest({ + method: 'GET', + url: '/?id=999999999', + params: { + id: 999999999, }, - } + }) + + var res = httpMocks.createResponse() + + const error = { status: 404 } + const db = { getByID: (id) => { return null @@ -155,16 +161,11 @@ describe('APIController', () => { const apiController = new ApiController(db) let result = null - const res = { - status: (value) => { - result = value - }, - } // Act apiController.single(req, res) // Assert - assert.deepEqual(result, error) + assert.deepEqual(result, error.status) }) })