robojerk/codeql-action

Separate paths with / in resolveUriToFile

Browse source
This commit is contained in:
Henry Mercer 2022-06-17 13:35:06 -07:00
parent c15604920a
commit 30681e79db
3 changed files with 11 additions and 9 deletions
Download patch file Download diff file Expand all files Collapse all files

9
lib/fingerprints.test.js generated
View file

@ -135,10 +135,11 @@ function testResolveUriToFile(uri, index, artifactsURIs) {
// so we need to give it real files to look at. We will use this file as an example.
// For this to work we require the current working directory to be a parent, but this
// should generally always be the case so this is fine.
const cwd = process.cwd();
const filepath = __filename;
t.true(filepath.startsWith(cwd + path.sep));
const relativeFilepath = filepath.substring(cwd.length + 1);
const filepath = __filename.split(path.sep).join("/");
const relativeFilepath = path
.relative(process.cwd(), __filename)
.split(path.sep)
.join("/");
// Absolute paths are unmodified
t.is(testResolveUriToFile(filepath, undefined, []), filepath);
t.is(testResolveUriToFile(`file://${filepath}`, undefined, []), filepath);

2
lib/fingerprints.test.js.map
View file

File diff suppressed because one or more lines are too long

9
src/fingerprints.test.ts
View file

@ -139,10 +139,11 @@ test("resolveUriToFile", (t) => {
// so we need to give it real files to look at. We will use this file as an example.
// For this to work we require the current working directory to be a parent, but this
// should generally always be the case so this is fine.
const cwd = process.cwd();
const filepath = __filename;
t.true(filepath.startsWith(cwd + path.sep));
const relativeFilepath = filepath.substring(cwd.length + 1);
const filepath = __filename.split(path.sep).join("/");
const relativeFilepath = path
.relative(process.cwd(), __filename)
.split(path.sep)
.join("/");
// Absolute paths are unmodified
t.is(testResolveUriToFile(filepath, undefined, []), filepath);