mockaton 13.3.0 → 13.3.1

package/package.json

@@ -2,7 +2,7 @@
 	"name": "mockaton",
 	"description": "HTTP Mock Server",
 	"type": "module",
-	"version": "13.3.0",
+	"version": "13.3.1",
 	"exports": {
 		".": {
 			"import": "./index.js",

package/src/client/app-store.js

@@ -108,7 +108,6 @@ export const store = {
 	},
 
 	setProxyFallback(value) {
-		store.skipNextRender = true
 		store._request(() => api.setProxyFallback(value), () => {
 			store.proxyFallback = value
 		})

package/src/client/app.css

@@ -432,7 +432,7 @@ main {
 		}
 
 		&.canProxy {
-			margin-left: 100px;
+			margin-left: 124px;
 		}
 	}
 
@@ -480,8 +480,15 @@ main {
 
 			.FolderName {
 				margin-left: 125px;
+				&.canProxy {
+					margin-left: 155px;
+				}
+
 				&.groupedByMethod {
 					margin-left: 79px;
+					&.canProxy {
+						margin-left: 109px;
+					}
 				}
 			}
 

package/src/client/app.js

@@ -120,7 +120,12 @@ function FolderGroups(groups) {
 			},
 			r('summary', null,
 				r('span', { className: CSS.FolderChevron }, ChevronDownIcon()),
-				r('span', { className: classNames(CSS.FolderName, store.groupByMethod && CSS.groupedByMethod) },
+				r('span', {
+						className: classNames(
+							CSS.FolderName,
+							store.groupByMethod && CSS.groupedByMethod,
+							store.canProxy && CSS.canProxy)
+					},
 					folder + '…')),
 			children.map(Row))
 	})

package/src/server/MockBroker.js

@@ -36,10 +36,10 @@ export class MockBroker {
 
 	unregister(file) {
 		this.mocks = this.mocks.filter(f => f !== file)
-		const isEmpty = !this.mocks.length
-		if (!isEmpty && this.file === file)
+		const brokerIsEmpty = !this.mocks.length
+		if (!brokerIsEmpty && this.file === file)
 			this.selectDefaultFile()
-		return isEmpty
+		return brokerIsEmpty
 	}
 
 	hasMock = file => this.mocks.includes(file)

package/src/server/Mockaton.test.js

@@ -7,7 +7,7 @@ import { mkdtempSync } from 'node:fs'
 import { randomUUID } from 'node:crypto'
 import { equal, deepEqual, match } from 'node:assert/strict'
 import { describe, test, before, beforeEach, after } from 'node:test'
-import { unlink, mkdir, readFile, rename, readdir, writeFile } from 'node:fs/promises'
+import { unlink, mkdir, readFile, rename, readdir, writeFile, rm } from 'node:fs/promises'
 
 import { mimeFor } from './utils/mime.js'
 import { parseFilename } from '../client/Filename.js'
@@ -26,8 +26,15 @@ const proc = spawn(join(import.meta.dirname, 'cli.js'), [
 	'--no-open'
 ])
 
-proc.stdout.on('data', data => stdout.push(data.toString()))
-proc.stderr.on('data', data => stderr.push(data.toString()))
+const DEBUG = false
+proc.stdout.on('data', data => {
+	stdout.push(data.toString())
+	DEBUG && process.stdout.write(stdout.at(-1))
+})
+proc.stderr.on('data', data => {
+	stderr.push(data.toString())
+	DEBUG && process.stderr.write(stdout.at(-1))
+})
 
 const serverAddr = await new Promise((resolve, reject) => {
 	proc.stdout.once('data', () => {
@@ -41,12 +48,14 @@ after(() => proc.kill('SIGUSR2'))
 
 
 const rmFromMocksDir = f => unlink(join(mocksDir, f))
-const listFromMocksDir = d => readdir(join(mocksDir, d))
 const readFromMocksDir = f => readFile(join(mocksDir, f), 'utf8')
 const writeInMocksDir = (f, data) => writeFile(join(mocksDir, f), data)
-const makeDirInMocks = dir => mkdir(join(mocksDir, dir), { recursive: true })
 const renameInMocksDir = (src, target) => rename(join(mocksDir, src), join(mocksDir, target))
 
+const listFromMocksDir = d => readdir(join(mocksDir, d))
+const rmDirFromMocks = d => rm(join(mocksDir, d), { recursive: true })
+const makeDirInMocks = dir => mkdir(join(mocksDir, dir), { recursive: true })
+
 
 const api = new Commander(serverAddr)
 
@@ -1120,6 +1129,16 @@ describe('Registering Mocks', () => {
 			equal(s.brokersByMethod.GET['/reg1/runtime0'].file, 'reg1/runtime0.GET.200.txt')
 		})
 	})
+
+	test('deleting a folder unregisters mocks in it', async () => {
+		const fx = new Fixture('api/bulk-delete/bar.GET.200.json')
+		await fx.write()
+		await sleep(0)
+		const nextVerPromise = resolveOnNextSyncVersion()
+		await rmDirFromMocks('api/bulk-delete')
+		await nextVerPromise
+		equal(await fx.fetchBroker(), undefined)
+	})
 })
 
 

package/src/server/mockBrokersCollection.js

@@ -47,7 +47,6 @@ export function registerMock(file, isFromWatcher = false) {
 	collection[method] ??= {}
 
 	let broker = collection[method][urlMask]
-
 	if (!broker)
 		broker = collection[method][urlMask] = new MockBroker(file)
 	else
@@ -61,15 +60,26 @@ export function registerMock(file, isFromWatcher = false) {
 
 export function unregisterMock(file) {
 	const broker = brokerByFilename(file)
-	const methodHasNoMoreMocks = broker?.unregister(file) // TODO or it was a directory of many mocks
-	if (methodHasNoMoreMocks) {
-		const { method, urlMask } = parseFilename(file)
-		delete collection[method][urlMask]
-		if (!Object.keys(collection[method]).length)
-			delete collection[method]
+	if (broker) {
+		const brokerIsEmpty = broker.unregister(file)
+		if (brokerIsEmpty) {
+			const { method, urlMask } = parseFilename(file)
+			delete collection[method][urlMask]
+			if (!Object.keys(collection[method]).length)
+				delete collection[method]
+		}
 	}
+	else for (const f of filesInDir(file)) // maybe it was a dir
+		unregisterMock(f)
 }
 
+function filesInDir(dir) {
+	const files = []
+	forEachBroker(b => {
+		files.push(...(b.mocks.filter(m => m.startsWith(dir + '/'))))
+	})
+	return files
+}
 
 /** @returns {MockBroker | undefined} */
 export function brokerByFilename(file) {