"nodeTdd.glob" won't work recursive [WINDOWS] about node-tdd

[prashaantt]

Thanks for digging this up @kbrandwijk! I don't have ready access to a Windows machine to test this out. Would you mind submitting a PR from a PC that you've tested this on?

[prashaantt]

@orgi I've seen the glob behave slightly differently on Windows. It's part of VS Code and I have no control over it. Also see #5 (comment).

[kbrandwijk]

Unfortunately, I can confirm that glob patterns are still not working on Windows. Maybe it's an idea to use https://code.visualstudio.com/docs/extensionAPI/vscode-api#RelativePattern instead of resolve? I'm not sure if it will fix things...

Update: upon further research, I'm positive that changing:

node-tdd/src/TestRunner.ts

Line 20 in 212e7fd

const globPath = resolve(workspace.rootPath, NodeTDD.getConfig<string>(config.GLOB));

into:

const globPath = new RelativePattern(workspace.workspaceFolders[0], NodeTDD.getConfig<string>(config.GLOB))

should do the trick, because path.resolve() will never result in more than one folder on Windows. Also, since the glob pattern uses /, path.posix.resolve() would be needed, but would still give you a single folder.

[kbrandwijk]

@prashaantt Please see the referenced PR. I have tested this on Windows only with a deep tree and the default glob setting, and it works like a charm.