mockaton 6.4.5 → 6.4.6

package/Tests.js

@@ -165,7 +165,7 @@ async function runTests() {
 		'api/alternative(comment-2).GET.200.json',
 		JSON.stringify({ comment: 2 }))
 
-	await test422WhenUpdatingNonExistingMockAlternative()
+	await testBadRequestWhenUpdatingNonExistingMockAlternative()
 
 	await testAutogenerates500(
 		'/api/alternative',
@@ -288,13 +288,15 @@ async function testItUpdatesDelayAndFile(url, file, expectedBody) {
 	})
 }
 
-async function test422WhenUpdatingNonExistingMockAlternative() {
+async function testBadRequestWhenUpdatingNonExistingMockAlternative() {
 	await it('There are mocks for /api/the-route but not this one', async () => {
+		const missingFile = 'api/the-route(non-existing-variant).GET.200.json'
 		const res = await request(API.edit, {
 			method: 'PATCH',
-			body: JSON.stringify({ [DF.file]: 'api/the-route(non-existing-variant).GET.200.json' })
+			body: JSON.stringify({ [DF.file]: missingFile })
 		})
-		equal(res.status, 422)
+		equal(res.status, 400)
+		equal(await res.text(), `Missing Mock: ${missingFile}`)
 	})
 }
 

package/package.json

@@ -2,7 +2,7 @@
 	"name": "mockaton",
 	"description": "A deterministic server-side for developing and testing frontend clients",
 	"type": "module",
-	"version": "6.4.5",
+	"version": "6.4.6",
 	"main": "index.js",
 	"types": "index.d.ts",
 	"license": "MIT",

package/src/Api.js

@@ -9,7 +9,7 @@ import { Config } from './Config.js'
 import { DF, API } from './ApiConstants.js'
 import { parseJSON } from './utils/http-request.js'
 import * as mockBrokersCollection from './mockBrokersCollection.js'
-import { sendOK, sendBadRequest, sendJSON, sendFile, sendUnprocessableContent } from './utils/http-response.js'
+import { sendOK, sendBadRequest, sendJSON, sendFile } from './utils/http-response.js'
 
 
 export const apiGetRequests = new Map([
@@ -60,10 +60,9 @@ async function updateBroker(req, response) {
 		const body = await parseJSON(req)
 		const file = body[DF.file]
 		const broker = mockBrokersCollection.getBrokerByFilename(file)
-		if (!broker || !broker.mockExists(file)) {
-			sendUnprocessableContent(response, `Missing Mock: ${file}`)
-			return
-		}
+		if (!broker || !broker.mockExists(file))
+			throw `Missing Mock: ${file}`
+
 		if (DF.delayed in body)
 			broker.updateDelay(body[DF.delayed])
 		broker.updateFile(file)

package/src/utils/http-response.js

@@ -51,7 +51,7 @@ export async function sendPartialContent(response, range, file) {
 export function sendBadRequest(response, error) {
 	console.error(error)
 	response.statusCode = 400
-	response.end()
+	response.end(error)
 }
 
 export function sendNotFound(response) {
@@ -59,12 +59,6 @@ export function sendNotFound(response) {
 	response.end()
 }
 
-export function sendUnprocessableContent(response, error) {
-	console.error(error)
-	response.statusCode = 422
-	response.end()
-}
-
 export function sendInternalServerError(response, error) {
 	console.error(error)
 	response.statusCode = 500