formatting

Author: Maarten-Dekker

packages/vite-plugin-tempest/src/config.test.ts

@@ -1,71 +1,76 @@
+import fs from 'node:fs'
+import path from 'node:path'
+import { fileURLToPath } from 'node:url'
 import { afterEach, beforeEach, describe, expect, test, vi } from 'vitest'
 import { loadTempestConfiguration } from './config'
 import { mockTempestConfiguration } from './test-utils'
-import fs from 'node:fs'
-import path from 'node:path'
 import * as utils from './utils'
 
-test('the configuration can be loaded', async () => {
-	mockTempestConfiguration()
+const __dirname = path.dirname(fileURLToPath(import.meta.url))
 
-	const config = await loadTempestConfiguration()
+describe('loading configuration via command or environment variable', () => {
+	afterEach(() => {
+		vi.restoreAllMocks()
+		delete process.env.TEMPEST_PLUGIN_CONFIGURATION_OVERRIDE
+	})
 
-	expect(config).toHaveProperty('build_directory')
-	expect(config).toHaveProperty('bridge_file_name')
-	expect(config).toHaveProperty('manifest')
-	expect(config).toHaveProperty('entrypoints')
-})
+	test('the configuration can be loaded', async () => {
+		mockTempestConfiguration()
 
-test('the configuration can be overriden', async () => {
-	process.env.TEMPEST_PLUGIN_CONFIGURATION_OVERRIDE = JSON.stringify({
-		build_directory: 'build',
-		bridge_file_name: 'vite-tempest',
-		manifest: 'manifest.json',
-		entrypoints: [],
+		const config = await loadTempestConfiguration()
+		expect(config.build_directory).toBe('build')
 	})
 
-	const config = await loadTempestConfiguration()
+	test('the configuration can be overriden with an environment variable', async () => {
+		const execSpy = vi.spyOn(utils, 'exec')
+		process.env.TEMPEST_PLUGIN_CONFIGURATION_OVERRIDE = JSON.stringify({
+			build_directory: 'build/from-env',
+			bridge_file_name: 'vite-tempest',
+			manifest: 'manifest.json',
+			entrypoints: [],
+		})
 
-	expect(config).toHaveProperty('build_directory')
-	expect(config).toHaveProperty('bridge_file_name')
-	expect(config).toHaveProperty('manifest')
-	expect(config).toHaveProperty('entrypoints')
+		const config = await loadTempestConfiguration()
+
+		expect(config.build_directory).toBe('build/from-env')
+		expect(execSpy).not.toHaveBeenCalled()
+	})
 })
 
 describe('loading configuration from a file', () => {
-	const tempConfigPath = path.join(__dirname, 'tempest-vite.test.json');
+	const tempConfigPath = path.join(__dirname, 'tempest-vite.test.json')
 	const mockConfig = {
 		build_directory: 'build/from-file',
 		bridge_file_name: 'test-bridge',
 		manifest: 'test-manifest.json',
 		entrypoints: ['resources/js/test.js'],
-	};
+	}
 
 	beforeEach(() => {
-		fs.writeFileSync(tempConfigPath, JSON.stringify(mockConfig));
-	});
+		fs.writeFileSync(tempConfigPath, JSON.stringify(mockConfig))
+	})
 
 	afterEach(() => {
-		fs.rmSync(tempConfigPath);
-		vi.restoreAllMocks();
-	});
+		fs.rmSync(tempConfigPath)
+		vi.restoreAllMocks()
+	})
 
 	test('it loads configuration from the specified file path', async () => {
 		const config = await loadTempestConfiguration({
 			config_file_path: tempConfigPath,
-		});
+		})
 
-		expect(config.build_directory).toBe('build/from-file');
-		expect(config.entrypoints).toEqual(['resources/js/test.js']);
-	});
+		expect(config.build_directory).toBe('build/from-file')
+		expect(config.entrypoints).toEqual(['resources/js/test.js'])
+	})
 
 	test('it does not execute the php command when a file path is provided', async () => {
-		const execSpy = vi.spyOn(utils, 'exec');
+		const execSpy = vi.spyOn(utils, 'exec')
 
 		await loadTempestConfiguration({
 			config_file_path: tempConfigPath,
-		});
+		})
 
-		expect(execSpy).not.toHaveBeenCalled();
-	});
-});
+		expect(execSpy).not.toHaveBeenCalled()
+	})
+})

packages/vite-plugin-tempest/src/config.ts

@@ -1,22 +1,22 @@
+import { readFileSync } from 'node:fs'
+import { resolve } from 'node:path'
 import type { TempestViteConfiguration } from './types'
 import { exec, php } from './utils'
-import { readFileSync } from 'node:fs';
-import { resolve } from 'node:path';
 
 const TEMPEST_BIN = 'tempest'
 const VITE_CONFIG_COMMAND = 'vite:config'
 
 export async function loadTempestConfiguration(
-	options: { config_file_path?: string } = {}
+	options: { config_file_path?: string } = {},
 ): Promise<TempestViteConfiguration> {
 	if (options.config_file_path) {
 		try {
-			const filePath = resolve(process.cwd(), options.config_file_path);
-			const fileContent = readFileSync(filePath, 'utf-8');
-			return JSON.parse(fileContent);
+			const filePath = resolve(process.cwd(), options.config_file_path)
+			const fileContent = readFileSync(filePath, 'utf-8')
+			return JSON.parse(fileContent)
 		} catch (e) {
-			console.error(`[vite-plugin-tempest] Error: Failed to read or parse the file at [${options.config_file_path}].`);
-			throw e;
+			console.error(`[vite-plugin-tempest] Error: Failed to read or parse the file at [${options.config_file_path}].`)
+			throw e
 		}
 	}
 

packages/vite-plugin-tempest/src/plugin.ts

@@ -16,7 +16,7 @@ const TEMPEST_ORIGIN_PLACEHOLDER = 'http://__tempest_placeholder__.test'
 let exitHandlersBound = false
 
 export interface TempestPluginOptions {
-	config_file_path?: string;
+	config_file_path?: string
 }
 
 export default function tempest(options: TempestPluginOptions = {}): Plugin {