Skip to content

Commit

Permalink
fix(module): set import.meta.url as parameter
Browse files Browse the repository at this point in the history
  • Loading branch information
jlenon7 committed Jul 6, 2022
1 parent 4c4df95 commit daa84d0
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 7 deletions.
4 changes: 2 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -547,8 +547,8 @@ import { Module } from '@secjs/utils'
const setInGlobalTrue = true
const setInGlobalFalse = false

const dirname = Module.createDirname(setInGlobalFalse)
const filename = Module.createFilename(setInGlobalTrue)
const dirname = Module.createDirname(import.meta.url, setInGlobalFalse)
const filename = Module.createFilename(import.meta.url, setInGlobalTrue)

console.log(__dirname) // Error! __dirname is not defined in global
console.log(__filename) // '/Users/...'
Expand Down
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "@secjs/utils",
"version": "1.9.6",
"version": "1.9.7",
"description": "Utils functions and classes for Node.js",
"license": "MIT",
"author": "João Lenon <lenon@athenna.io>",
Expand Down
8 changes: 4 additions & 4 deletions tests/Unit/ModuleTest.js
Original file line number Diff line number Diff line change
Expand Up @@ -70,14 +70,14 @@ test.group('ExecTest', group => {
})

test('should be able to create __filename property inside node global', async ({ assert }) => {
Module.createFilename(true)
Module.createFilename(import.meta.url, true)

assert.isTrue(__filename.includes('Module.js'))
assert.isTrue(__filename.includes('ModuleTest.js'))
})

test('should be able to create __dirname property inside node global', async ({ assert }) => {
Module.createDirname(true)
Module.createDirname(import.meta.url, true)

assert.isTrue(__dirname.includes('Helpers'))
assert.isTrue(__dirname.includes('Unit'))
})
})

0 comments on commit daa84d0

Please sign in to comment.