better skip

Author: aduh95

tests/Up.test.ts

@@ -34,7 +34,9 @@ describe(`UpCommand`, () => {
     });
   });
 
-  it(`should update the ".corepack.env" file from the current project`, async () => {
+  it(`should update the ".corepack.env" file from the current project`, async t => {
+    // Skip that test on Node.js 18.x as it lacks support for .env files.
+    if (process.version.startsWith(`v18.`)) t.skip();
     await Promise.all([
       `COREPACK_DEV_ENGINES_YARN=2.1.0\n`,
       `\nCOREPACK_DEV_ENGINES_YARN=2.1.0\n`,
@@ -68,7 +70,10 @@ describe(`UpCommand`, () => {
     })));
   });
 
-  it(`should update the ".other.env" file from the current project when configured to use that`, async () => {
+  it(`should update the ".other.env" file from the current project when configured to use that`, async t => {
+    // Skip that test on Node.js 18.x as it lacks support for .env files.
+    if (process.version.startsWith(`v18.`)) t.skip();
+
     await Promise.all([
       `COREPACK_DEV_ENGINES_YARN=2.1.0\n`,
       `\nCOREPACK_DEV_ENGINES_YARN=2.1.0\n`,

tests/Use.test.ts

@@ -33,7 +33,10 @@ describe(`UseCommand`, () => {
     });
   });
 
-  it(`should update .corepack.env if present`, async () => {
+  it(`should update .corepack.env if present`, async t => {
+    // Skip that test on Node.js 18.x as it lacks support for .env files.
+    if (process.version.startsWith(`v18.`)) t.skip();
+
     await Promise.all([
       `COREPACK_DEV_ENGINES_YARN=1.1.0\n`,
       `\nCOREPACK_DEV_ENGINES_YARN=1.1.0\n`,
@@ -66,7 +69,10 @@ describe(`UseCommand`, () => {
     })));
   });
 
-  it(`should update .other.env if present`, async () => {
+  it(`should update .other.env if present`, async t => {
+    // Skip that test on Node.js 18.x as it lacks support for .env files.
+    if (process.version.startsWith(`v18.`)) t.skip();
+
     await Promise.all([
       `COREPACK_DEV_ENGINES_YARN=1.1.0\n`,
       `\nCOREPACK_DEV_ENGINES_YARN=1.1.0\n`,

tests/main.test.ts

@@ -315,7 +315,7 @@ it(`should use hash from env even when "devEngines" defines a different one`, as
 });
 
 it(`should use hash from env even when ".corepack.env" defines a different one`, async t => {
-  // Skip rest of the test on Node.js 18.x as it lacks support for .env files.
+  // Skip that test on Node.js 18.x as it lacks support for .env files.
   if (process.version.startsWith(`v18.`)) t.skip();
 
   await xfs.mktempPromise(async cwd => {
@@ -340,7 +340,7 @@ it(`should use hash from env even when ".corepack.env" defines a different one`,
 
 describe(`should accept range in devEngines only if a specific version is provided`, () => {
   it(`either in .corepack.env`, async t => {
-    // Skip rest of the test on Node.js 18.x as it lacks support for .env files.
+    // Skip that test on Node.js 18.x as it lacks support for .env files.
     if (process.version.startsWith(`v18.`)) t.skip();
 
     await xfs.mktempPromise(async cwd => {
@@ -368,7 +368,7 @@ describe(`should accept range in devEngines only if a specific version is provid
     });
   });
   it(`either in a different env file specified in env`, async t => {
-    // Skip rest of the test on Node.js 18.x as it lacks support for .env files.
+    // Skip that test on Node.js 18.x as it lacks support for .env files.
     if (process.version.startsWith(`v18.`)) t.skip();
 
     await xfs.mktempPromise(async cwd => {
@@ -455,7 +455,7 @@ describe(`should accept range in devEngines only if a specific version is provid
 });
 
 it(`Should use version from correct source`, async t => {
-  // Skip rest of the test on Node.js 18.x as it lacks support for .env files.
+  // Skip that test on Node.js 18.x as it lacks support for .env files.
   if (process.version.startsWith(`v18.`)) t.skip();
 
   await xfs.mktempPromise(async cwd => {
@@ -497,7 +497,7 @@ it(`Should use version from correct source`, async t => {
 
 describe(`should reject if range in devEngines does not match version provided`,  () => {
   it(`in .corepack.env`, async t => {
-    // Skip rest of the test on Node.js 18.x as it lacks support for .env files.
+    // Skip that test on Node.js 18.x as it lacks support for .env files.
     if (process.version.startsWith(`v18.`)) t.skip();
 
     await xfs.mktempPromise(async cwd => {