fix: remove lint warnings, update tests and rename file (#1691)

* fix silent failing test

* remove lint warnings, update tests and rename file

* Update packages/cli/tests/build.test.js

* Update packages/cli/babel/index.js

Co-authored-by: Ryan Christian <33403762+rschristian@users.noreply.github.com>

Author: zubhav

packages/cli/babel/index.js

@@ -21,7 +21,7 @@ var defaultBrowserListDev = [
 	'ios_saf >= 10.3',
 	'edge >= 16',
 	'opera >= 48',
-	'samsung >= 8.2'
+	'samsung >= 8.2',
 ];
 
 // preact-cli babel configs

packages/cli/sw/contants.js renamed to packages/cli/sw/constants.js

@@ -1,7 +1,7 @@
 import { registerRoute, setCatchHandler } from 'workbox-routing';
 import { precacheAndRoute, getCacheKeyForURL } from 'workbox-precaching';
 import { isNav } from './utils';
-import { NETWORK_HANDLER, PRECACHING_OPTIONS } from './contants';
+import { NETWORK_HANDLER, PRECACHING_OPTIONS } from './constants';
 
 export function getFiles() {
 	return self.__WB_MANIFEST;

packages/cli/sw/index.js

@@ -182,8 +182,10 @@ describe('preact build', () => {
 			`${dir}/build/${bundleFile}`,
 			'utf8'
 		);
+
 		// "Hello World!" should replace 'process.env.PREACT_APP_MY_VARIABLE'
-		expect(transpiledChunk.includes('console.log("Hello World!")')).toBe(true);
+		expect(transpiledChunk.includes('"Hello World!"')).toBe(true);
+		expect(transpiledChunk.includes('PREACT_APP_MY_VARIABLE')).toBe(false);
 	});
 
 	it('should respect `publicPath` value', async () => {

packages/cli/tests/build.test.js

@@ -12,7 +12,7 @@ module.exports.waitUntilExpression = async function (page, expression) {
 			`Waiting for ${expression}`
 		);
 
-		return result; // && result.value;
+		return result;
 	};
 
 	await waitUntil(evaluate, `Waiting for ${expression} timed out!`);

packages/cli/tests/lib/chrome.js

@@ -10,7 +10,7 @@ async function tmpDir() {
 	let str = Math.random()
 		.toString(36)
 		.replace(/[^a-z]+/g, '')
-		.substr(0, 12);
+		.substring(0, 12);
 	await mkdir(output, { recursive: true });
 	return resolve(output, str);
 }

packages/cli/tests/lib/output.js

@@ -1 +1 @@
-console.log(process.env.PREACT_APP_MY_VARIABLE);
+export const MY_VARIABLE = process.env.PREACT_APP_MY_VARIABLE;