Skip to content

Commit 96b9ba8

Browse files
committed
test: remove normalize-path
1 parent 0b18b8c commit 96b9ba8

File tree

6 files changed

+11
-38
lines changed

6 files changed

+11
-38
lines changed

__tests__/cjs/import.test.ts

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
import os from 'os'
22
import path from 'path'
3-
import normalizePath from 'normalize-path'
43
import {
54
importFromString,
65
createImportFromString,
@@ -136,9 +135,8 @@ export default code
136135
it('should use relative filename in error stack trace', async () => {
137136
expect.assertions(1)
138137
const filename = 'foo.js'
139-
const relativeDirnamePath = path.relative(process.cwd(), __dirname)
140-
const relativeFilenamePath = path.join(relativeDirnamePath, filename)
141-
const relativeFilename = normalizePath(relativeFilenamePath)
138+
const relativeDirname = path.relative(process.cwd(), __dirname)
139+
const relativeFilename = path.join(relativeDirname, filename)
142140
try {
143141
await importFromStringFn('throw new Error("boom")', {
144142
filename,
@@ -155,8 +153,7 @@ export default code
155153

156154
it('should use absolute filename in error stack trace', async () => {
157155
expect.assertions(1)
158-
const filenamePath = path.join(os.homedir(), 'foo', 'bar', 'baz.js')
159-
const filename = normalizePath(filenamePath)
156+
const filename = path.join(os.homedir(), 'foo', 'bar', 'baz.js')
160157
try {
161158
await importFromStringFn('throw new Error("boom")', {
162159
filename,

__tests__/cjs/require.test.ts

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
import os from 'os'
22
import path from 'path'
3-
import normalizePath from 'normalize-path'
43
import { requireFromString, createRequireFromString } from '../../src/index'
54

65
it('should work with `module.exports`', () => {
@@ -92,9 +91,8 @@ it('should be able to override current global', () => {
9291
it('should use relative filename in error stack trace', () => {
9392
expect.assertions(1)
9493
const filename = 'foo.js'
95-
const relativeDirnamePath = path.relative(process.cwd(), __dirname)
96-
const relativeFilenamePath = path.join(relativeDirnamePath, filename)
97-
const relativeFilename = normalizePath(relativeFilenamePath)
94+
const relativeDirname = path.relative(process.cwd(), __dirname)
95+
const relativeFilename = path.join(relativeDirname, filename)
9896
try {
9997
requireFromString('throw new Error("boom")', {
10098
filename,
@@ -111,8 +109,7 @@ it('should use relative filename in error stack trace', () => {
111109

112110
it('should use absolute filename in error stack trace', () => {
113111
expect.assertions(1)
114-
const filenamePath = path.join(os.homedir(), 'foo', 'bar', 'baz.js')
115-
const filename = normalizePath(filenamePath)
112+
const filename = path.join(os.homedir(), 'foo', 'bar', 'baz.js')
116113
try {
117114
requireFromString('throw new Error("boom")', {
118115
filename,

__tests__/esm/import.test.ts

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
import os from 'os'
22
import path from 'path'
33
import { fileURLToPath, pathToFileURL } from 'url'
4-
import normalizePath from 'normalize-path'
54
import { importFromString, createImportFromString, importFromStringSync } from '../../src/index'
65

76
const __dirname = path.dirname(fileURLToPath(import.meta.url))
@@ -140,9 +139,8 @@ export default code
140139
it('should use relative filename in error stack trace', async () => {
141140
expect.assertions(1)
142141
const filename = 'foo.js'
143-
const relativeDirnamePath = path.relative(process.cwd(), __dirname)
144-
const relativeFilenamePath = path.join(relativeDirnamePath, filename)
145-
const relativeFilename = normalizePath(relativeFilenamePath)
142+
const relativeDirname = path.relative(process.cwd(), __dirname)
143+
const relativeFilename = path.join(relativeDirname, filename)
146144
try {
147145
await importFromString('throw new Error("boom")', {
148146
filename,

__tests__/esm/require.test.ts

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
import os from 'os'
22
import path from 'path'
3-
import normalizePath from 'normalize-path'
43
import { fileURLToPath } from 'url'
54
import { requireFromString } from '../../src/index'
65

@@ -86,9 +85,8 @@ it('should work with current global', () => {
8685
it('should use relative filename in error stack trace', () => {
8786
expect.assertions(1)
8887
const filename = 'foo.js'
89-
const relativeDirnamePath = path.relative(process.cwd(), __dirname)
90-
const relativeFilenamePath = path.join(relativeDirnamePath, filename)
91-
const relativeFilename = normalizePath(relativeFilenamePath)
88+
const relativeDirname = path.relative(process.cwd(), __dirname)
89+
const relativeFilename = path.join(relativeDirname, filename)
9290
try {
9391
requireFromString('throw new Error("boom")', {
9492
filename,
@@ -105,8 +103,7 @@ it('should use relative filename in error stack trace', () => {
105103

106104
it('should use absolute filename in error stack trace', () => {
107105
expect.assertions(1)
108-
const filenamePath = path.join(os.homedir(), 'foo', 'bar', 'baz.js')
109-
const filename = normalizePath(filenamePath)
106+
const filename = path.join(os.homedir(), 'foo', 'bar', 'baz.js')
110107
try {
111108
requireFromString('throw new Error("boom")', {
112109
filename,

package-lock.json

Lines changed: 0 additions & 14 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

package.json

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -52,10 +52,8 @@
5252
"devDependencies": {
5353
"@types/jest": "28.1.8",
5454
"@types/node": "18.7.16",
55-
"@types/normalize-path": "3.0.0",
5655
"cross-env": "7.0.3",
5756
"jest": "28.1.3",
58-
"normalize-path": "3.0.0",
5957
"rollup": "2.79.0",
6058
"rollup-plugin-dts": "4.2.2",
6159
"rollup-plugin-esbuild-transform": "1.5.0",

0 commit comments

Comments
 (0)