Feature request: Spec files next to sources (co-located tests)

[capaj]

I like having spec files right next to the source files.
Currently this is not achievable. Can we get a support for this style of filesystem organisation?

[jaredpalmer]

I think it’s possible, but would need to investigate and play with the default TS settings.

[mormahr]

So it's not sufficient to change jest's testMatch config? I'm new to TypeScript and used a babel / flow / webpack setup and this was the default. Does TypeScript handle this different?

[techniq]

In another project I use Typescript and babel with this config in package.json

  "jest": {
    "rootDir": "src",
    "moduleFileExtensions": [
      "ts",
      "tsx",
      "js",
      "jsx"
    ],
    "transform": {
      "^.+\\.(js|jsx|ts|tsx)$": "babel-jest"
    }
  }

I just tried this in my tsdx project, but it doesn't seem to work.

[techniq]

@mormahr the following does seem to locate the tests...

  "jest": {
    "testMatch": [
      "<rootDir>/src/**/?(*.)(spec|test).(ts|js)?(x)"
    ]
  },

but it's failing to find any tests to run

 FAIL  src/app/State.test.tsx
  ● Test suite failed to run

    Your test suite must contain at least one test.

      at node_modules/@jest/core/build/TestScheduler.js:243:24
      at asyncGeneratorStep (node_modules/@jest/core/build/TestScheduler.js:131:24)
      at _next (node_modules/@jest/core/build/TestScheduler.js:151:9)
      at node_modules/@jest/core/build/TestScheduler.js:156:7
      at node_modules/@jest/core/build/TestScheduler.js:148:12
      at onResult (node_modules/@jest/core/build/TestScheduler.js:272:25)

Test Suites: 1 failed, 1 total
Tests:       0 total
Snapshots:   0 total
Time:        1.348s
Ran all test suites.
npm ERR! Test failed.  See above for more details.

although after looking closer, this might be due to some Typescript errors (I was using Typescript 3.1 and not 3.4.3 like tsdx is currently). I'll investigate more later.

[techniq]

It did appear to be my failing Typescript types from changes in 3.1 to 3.4.3. Adding testMatch mentioned above was enough to get passing tests within src

[natemoo-re]

Would folks be open to updating the default testMatch at createJestConfig.ts:15 to support this out of the box?

The version below would support both use cases and running tsdx test from the root of a monorepo (cc #122.)

<rootDir>/**/*.(spec|test).{ts,tsx}

This may have some larger implications, but I think it's worth investigating to keep things as zero-config as possible.

[mormahr]

Yes also: the __tests__/ convention (without needing .test. in the actual test/spec file).

Essentially: Why not use the default jest config of (/__tests__/.*|(\\.|/)(test|spec))\\.[jt]sx?$?

[wachunga]

I prefer test files next to the source as well, and would prefer not to have to override the testWatch config in my package.json.

[malcolm-kee]

Is this possible now with v0.8?

[arvinsim]

Yes, I prefer the test files be next to the source. It's just so much better than having to switch folders view.

[kylemh]

https://github.com/jaredpalmer/tsdx/blob/master/src/createJestConfig.ts#L12

This is currently possible, and this issue should be closed.

[capaj]

thanks @kylemh, closing. Will reopen if it doesn't work for me.

[agilgur5]

This was fixed by #159 . Also related to #464 / #472 and #342 / #344