mockaton 6.3.9 → 6.3.10

package/Tests.js

@@ -17,6 +17,7 @@ import { API, DF, DEFAULT_500_COMMENT } from './src/ApiConstants.js'
 
 const tmpDir = mkdtempSync(tmpdir()) + '/'
 const staticTmpDir = mkdtempSync(tmpdir()) + '/'
+console.log(tmpDir)
 
 const fixtureCustomMime = [
 	'/api/custom-mime',
@@ -380,12 +381,12 @@ async function testInvalidFilenamesAreIgnored() {
 		consoleErrorSpy.mock.mockImplementation(() => {}) // so they don’t render in the test report
 
 		write('api/_INVALID_FILENAME_CONVENTION_.json', '')
-		write('api/bad-filename.GET._INVALID_STATUS_.json', '')
-		write('api/bad-filename._INVALID_METHOD_.200.json', '')
+		write('api/bad-filename-method._INVALID_METHOD_.200.json', '')
+		write('api/bad-filename-status.GET._INVALID_STATUS_.json', '')
 		await reset()
 		equal(consoleErrorSpy.mock.calls[0].arguments[0], 'Invalid Filename Convention')
-		equal(consoleErrorSpy.mock.calls[1].arguments[0], 'Invalid HTTP Response Status: "NaN"')
-		equal(consoleErrorSpy.mock.calls[2].arguments[0], 'Unrecognized HTTP Method: "_INVALID_METHOD_"')
+		equal(consoleErrorSpy.mock.calls[1].arguments[0], 'Unrecognized HTTP Method: "_INVALID_METHOD_"')
+		equal(consoleErrorSpy.mock.calls[2].arguments[0], 'Invalid HTTP Response Status: "NaN"')
 	})
 }
 

package/package.json

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

package/src/Filename.js

@@ -14,7 +14,14 @@ export const includesComment = (filename, search) =>
 	extractComments(filename).some(comment => comment.includes(search))
 
 
-export function validateFilename(file) {
+export function filenameIsValid(file) {
+	const error = validateFilename(file)
+	if (error)
+		console.error(error, file)
+	return !error
+}
+
+function validateFilename(file) {
 	const tokens = file.replace(reComments, '').split('.')
 	if (tokens.length < 4)
 		return 'Invalid Filename Convention'

package/src/MockBroker.js

@@ -41,7 +41,6 @@ export class MockBroker {
 	get file() { return this.currentMock.file }
 	get delay() { return this.currentMock.delay }
 	get status() { return parseFilename(this.file).status }
-	get isTemp500() { return includesComment(this.file, DEFAULT_500_COMMENT) }
 
 	updateFile(filename) {
 		this.currentMock.file = filename
@@ -78,6 +77,9 @@ export class MockBroker {
 		const file = urlMask.replace(/^\//, '') // Removes leading slash TESTME
 		this.register(`${file}${DEFAULT_500_COMMENT}.${method}.500.txt`)
 	}
+	get isTemp500() {
+		return includesComment(this.file, DEFAULT_500_COMMENT)
+	}
 }
 
 // Stars out (for regex) all the paths that are in square brackets

package/src/mockBrokersCollection.js

@@ -1,11 +1,8 @@
-import { join } from 'node:path'
-import { readdirSync as readDir } from 'node:fs'
-
 import { Config } from './Config.js'
 import { cookie } from './cookie.js'
-import { isFile } from './utils/fs.js'
 import { MockBroker } from './MockBroker.js'
-import { parseFilename, validateFilename } from './Filename.js'
+import { listFilesRecursively } from './utils/fs.js'
+import { parseFilename, filenameIsValid } from './Filename.js'
 
 
 /**
@@ -24,16 +21,11 @@ export function init() {
 	collection = {}
 	cookie.init(Config.cookies)
 
-	const files = readDir(Config.mocksDir, { recursive: true })
-		.filter(f => !Config.ignore.test(f) && isFile(join(Config.mocksDir, f)))
+	const files = listFilesRecursively(Config.mocksDir)
 		.sort()
+		.filter(f => !Config.ignore.test(f) && filenameIsValid(f))
 
 	for (const file of files) {
-		const error = validateFilename(file)
-		if (error) {
-			console.error(error, file)
-			continue
-		}
 		const { method, urlMask } = parseFilename(file)
 		collection[method] ??= {}
 		if (!collection[method][urlMask])
@@ -75,8 +67,8 @@ export function getBrokerForUrl(method, url) {
 export function extractAllComments() {
 	const comments = new Set()
 	forEachBroker(broker => {
-		for (const comment of broker.extractComments())
-			comments.add(comment)
+		for (const c of broker.extractComments())
+			comments.add(c)
 	})
 	return Array.from(comments)
 }

package/src/utils/fs.js

@@ -1,7 +1,11 @@
-import { lstatSync, readFileSync } from 'node:fs'
+import { join } from 'node:path'
+import { lstatSync, readFileSync, readdirSync } from 'node:fs'
 
 
 export const isFile = path => lstatSync(path, { throwIfNoEntry: false })?.isFile()
 export const isDirectory = path => lstatSync(path, { throwIfNoEntry: false })?.isDirectory()
 
 export const read = path => readFileSync(path)
+
+export const listFilesRecursively = dir => readdirSync(dir, { recursive: true })
+	.filter(f => isFile(join(dir, f)))