Merge pull request #3 from yowainwright/init-parseModule-tests

feat: new tests + some chomp 🔥

Author: JayaKrishnaNamburu

chompfile.toml

@@ -3,7 +3,14 @@ version = 0.1
 [[task]]
 name = 'build'
 run = 'tsup src/loader.ts --format esm,cjs --dts'
-deps = 'npm:install'
+dep = 'src/loader.ts'
+target = 'dist/loader.js'
+
+[[task]]
+name = 'npm:install'
+run = 'npm install'
+dep = 'package.json'
+target = 'package-lock.json'
 
 [[task]]
 name = 'clean'
@@ -21,14 +28,10 @@ run = 'commitlint --edit $1'
 name = 'default'
 run = 'npm install --no-save react-dom@17.0.2 @teleporthq/teleport-project-generator-react@0.25.4 @teleporthq/teleport-project-generator-preact@0.20.0 chalk express morgan react-router@5.2.1 react-router-dom@5.2.1 && node tests/e2e/test.js'
 
-[[task]]
-name = 'npm:install'
-run = 'npm install'
-deps = ['package.json']
-
 [[task]]
 name = 'lint'
 run = 'eslint src --ext .ts'
+deps = ['src/#.ts', 'tests/loader.test.ts']
 
 [[task]]
 name = 'lint-fix'
@@ -37,6 +40,9 @@ run = 'chomp lint --fix'
 [[task]]
 name = 'pre-commit'
 run = 'lint-staged'
+deps = ['src/#.ts', 'tests/loader.test.ts']
+
+[[task]]
 
 [[task]]
 name = 'prepare'
@@ -77,3 +83,4 @@ run = 'vitest run'
 [[task]]
 name = 'typecheck'
 run = 'tsc --noEmit'
+deps = ['src/#.ts']

src/loader.ts

@@ -13,7 +13,7 @@ const cacheMap = new Map();
  * @description a convenience function to ensure a file exists
  * @param path
  */
-function ensureFileSync(path: string) {
+export function ensureFileSync(path: string) {
   try {
     accessSync(path);
   } catch (err) {

tests/loader.test.ts

@@ -1,3 +1,5 @@
+import * as global from "node:fs";
+import { join } from "node:path";
 import { vi, test, expect } from 'vitest'
 import { ImportMap } from "@jspm/import-map";
 
@@ -9,6 +11,8 @@ import {
   getLastPart,
   getPackageNameVersionFromUrl,
   getVersion,
+  parseModule,
+  ensureFileSync,
 } from "../src/loader"
 
 vi.mock('url', () => ({
@@ -74,3 +78,12 @@ test('getPackageNameVersionFromUrl', () => {
     version: '1.10.0',
   });
 });
+
+
+test('should return the specifier if the module path is a node or file protocol', async () => {
+  const specifier = 'my-module';
+  const modulePath1 = 'node:fs';
+  const modulePath2 = 'file:///path/to/my/module.js';
+  expect(await parseModule(specifier, modulePath1)).toEqual(specifier);
+  expect(await parseModule(specifier, modulePath2)).toEqual(specifier);
+});