Robert

We potentially output files with environment variables in formats for a few different shells, but we base that decision solely on which OS we're running on. This leads to problems...

(Builds upon the https://github.com/github/vscode-codeql/pull/3452 PR because otherwise it'll just conflict. The changes aren't related except that the functions being moved are next to each other.) I argue that the `convertGithubNwoToDatabaseUrl`...

The `findDirWithFile` function is pretty general and I'd argue belongs in `files.ts` more than it does in `database-fetcher.ts`. This refactoring is not a huge win in itself but it helps...

This PR is trying to add tests that would have caught the bug fixed in https://github.com/github/vscode-codeql/pull/3439 To do this we need to initialize a `MethodModelingViewProvider` and send it a "viewLoaded"...