Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

jest.config.js created with hardcoded paths #90

Closed
veksen opened this issue May 8, 2019 · 2 comments
Closed

jest.config.js created with hardcoded paths #90

veksen opened this issue May 8, 2019 · 2 comments

Comments

@veksen
Copy link

veksen commented May 8, 2019

Current Behavior

When running yarn test, a jest.config.js is created, but the paths are hardcoded to the computer that runs it.

module.exports = {
    "transform": {
        ".(ts|tsx)": "/Users/veksen/Projects/mylib/node_modules/ts-jest/dist/index.js"
    },
    "transformIgnorePatterns": [
        "[/\\\\]node_modules[/\\\\].+\\.(js|jsx)$"
    ],
    "moduleFileExtensions": [
        "ts",
        "tsx",
        "js",
        "jsx",
        "json",
        "node"
    ],
    "collectCoverageFrom": [
        "src/**/*.{ts,tsx}"
    ],
    "testMatch": [
        "<rootDir>/test/**/*.(spec|test).{ts,tsx}"
    ],
    "testURL": "http://localhost",
    "rootDir": "/Users/veksen/Projects/mylib",
    "watchPlugins": [
        "/Users/veksen/Projects/mylib/node_modules/jest-watch-typeahead/filename.js",
        "/Users/veksen/Projects/mylib/node_modules/jest-watch-typeahead/testname.js"
    ]
}

Expected behavior

Probably either use <rootDir>, or should jest.config.js be gitignored? It doesn't seem to make sense to commit it in its current state (aka, the user computer hardcoded)

Your environment

Software Version(s)
TSDX 0.5.7
TypeScript 3.4.5
Browser N/A
npm/Yarn 6.4.1/1.15.2
Operating System MacOS
@veksen veksen changed the title jest.config.js jest.config.js created with hardcoded paths May 8, 2019
@jaredpalmer
Copy link
Owner

I think i just need to get rid of my require.resolve

@jaredpalmer
Copy link
Owner

Upgrade and remove, your should remove jest.config.js

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants