refactor: ♻️ move around tests and text

Author: HofmannZ

src/helpers/get-public-env.spec.ts

@@ -22,16 +22,6 @@ describe('getPublicEnv()', () => {
     delete process.env.NEXT_PUBLIC_BAZ;
   });
 
-  it('should show an info message after reading the environment', () => {
-    getPublicEnv();
-
-    expect(infoSpy).toHaveBeenCalledWith(
-      `${chalk.cyan(
-        `info`
-      )}  - [next-runtime-env] - Read environment variables prefixed with 'NEXT_PUBLIC_' from process.env.`
-    );
-  });
-
   it('should return an allow-listed value', () => {
     process.env.NEXT_PUBLIC_FOO = 'foo';
 
@@ -76,4 +66,14 @@ describe('getPublicEnv()', () => {
       NEXT_PUBLIC_BAZ: 'baz',
     });
   });
+
+  it('should show an info message after reading the environment', () => {
+    getPublicEnv();
+
+    expect(infoSpy).toHaveBeenCalledWith(
+      `${chalk.cyan(
+        `info`
+      )}  - [next-runtime-env] - Read environment variables prefixed with 'NEXT_PUBLIC_' from process.env.`
+    );
+  });
 });

src/helpers/write-browser-env.spec.ts

@@ -10,7 +10,7 @@ const path = `${base}/public`;
 const file = `${path}/__ENV.js`;
 const message = `${chalk.cyan(
   `info`
-)}  - [next-runtime-env] - Wrote browser runtime environment variables to ${file}`;
+)}  - [next-runtime-env] - Wrote browser runtime environment variables to '${file}'.`;
 
 beforeAll(() => {
   infoSpy.mockImplementation();

src/helpers/write-browser-env.ts

@@ -14,5 +14,5 @@ export function writeBrowserEnv(env: NodeJS.ProcessEnv) {
 
   fs.writeFileSync(path, content);
 
-  log.info(`Wrote browser runtime environment variables to ${path}`);
+  log.info(`Wrote browser runtime environment variables to '${path}'.`);
 }

src/utils/log.ts

@@ -1,20 +1,21 @@
-/* eslint-disable no-console */
-
 import chalk from 'chalk';
 
 const libraryName = '[next-runtime-env]';
-const librarySeperator = '-';
-const libraryPrefix = `${libraryName} ${librarySeperator}`;
+const librarySeparator = '-';
+const libraryPrefix = `${libraryName} ${librarySeparator}`;
 
 const prefixes = {
+  // Double space before the dash aligns messages in the terminal and improves readability.
   warn: `${chalk.yellow(`warn`)}  - ${libraryPrefix}`,
-  info: `${chalk.cyan(`info`)}  - ${libraryPrefix}`, // Double space before the dash aligns messages in the terminal and improves readability.
+  info: `${chalk.cyan(`info`)}  - ${libraryPrefix}`,
 };
 
 export function warn(message: string) {
+  // eslint-disable-next-line no-console
   console.warn(`${prefixes.warn} ${message}`);
 }
 
 export function info(message: string) {
+  // eslint-disable-next-line no-console
   console.info(`${prefixes.info} ${message}`);
 }