test: small improvements to the test cases

Author: thetutlage

src/env.ts

@@ -8,10 +8,11 @@
  */
 
 import { schema as envSchema } from '@poppinss/validator-lite'
+import type { ValidateFn } from '@poppinss/validator-lite/types'
+
+import { EnvParser } from './parser.js'
 import { EnvValidator } from './validator.js'
 import { EnvProcessor } from './processor.js'
-import { EnvParser } from './parser.js'
-import type { ValidateFn } from '@poppinss/validator-lite/types'
 
 /**
  * A wrapper over "process.env" with types information.

tests/env.spec.ts

@@ -62,21 +62,17 @@ test.group('Env', (group) => {
     cleanup,
     fs,
   }) => {
-    assert.plan(1)
-
     cleanup(() => {
       Env.removeIdentifier('file')
       delete process.env.PORT
     })
 
     Env.defineIdentifier('file', (_value: string) => {
-      assert.isTrue(true)
-
       return '3000'
     })
 
     Env.defineIdentifierIfMissing('file', (_value: string) => {
-      return '3000'
+      return '4000'
     })
 
     await fs.create('.env', 'PORT=file:romain')

tests/parser.spec.ts

@@ -95,21 +95,16 @@ test.group('Env Parser', () => {
     cleanup,
     expectTypeOf,
   }) => {
-    assert.plan(2)
-
     cleanup(() => {
       EnvParser.removeIdentifier('file')
     })
 
     EnvParser.defineIdentifier('file', (_value: string) => {
-      console.log('Never called')
-      assert.isTrue(true)
-
       return '3000'
     })
 
     EnvParser.defineIdentifierIfMissing('file', (_value: string) => {
-      return '3000'
+      return '4000'
     })
 
     const envString = ['ENV_USER=file:romain'].join('\n')