Skip to content

Commit 2e0854f

Browse files
committed
Fix tests
1 parent b2d6299 commit 2e0854f

File tree

4 files changed

+7
-4
lines changed

4 files changed

+7
-4
lines changed

src/analyze-action-env.test.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -47,6 +47,7 @@ test("analyze action with RAM & threads from environment variables", async (t) =
4747
const requiredInputStub = sinon.stub(actionsUtil, "getRequiredInput");
4848
requiredInputStub.withArgs("token").returns("fake-token");
4949
requiredInputStub.withArgs("upload-database").returns("false");
50+
requiredInputStub.withArgs("output").returns("out");
5051
const optionalInputStub = sinon.stub(actionsUtil, "getOptionalInput");
5152
optionalInputStub.withArgs("cleanup-level").returns("none");
5253
optionalInputStub.withArgs("expect-error").returns("false");

src/analyze-action-input.test.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -45,6 +45,7 @@ test("analyze action with RAM & threads from action inputs", async (t) => {
4545
const requiredInputStub = sinon.stub(actionsUtil, "getRequiredInput");
4646
requiredInputStub.withArgs("token").returns("fake-token");
4747
requiredInputStub.withArgs("upload-database").returns("false");
48+
requiredInputStub.withArgs("output").returns("out");
4849
const optionalInputStub = sinon.stub(actionsUtil, "getOptionalInput");
4950
optionalInputStub.withArgs("cleanup-level").returns("none");
5051
optionalInputStub.withArgs("expect-error").returns("false");

src/upload-lib.ts

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@ import * as jsonschema from "jsonschema";
99
import * as semver from "semver";
1010

1111
import * as actionsUtil from "./actions-util";
12-
import { getOptionalInput, getRequiredInput } from "./actions-util";
1312
import * as api from "./api-client";
1413
import { getGitHubVersion, wrapApiConfigurationError } from "./api-client";
1514
import { CodeQL, getCodeQL } from "./codeql";
@@ -259,8 +258,8 @@ async function combineSarifFilesUsingCLI(
259258
);
260259

261260
const apiDetails = {
262-
auth: getRequiredInput("token"),
263-
externalRepoAuth: getOptionalInput("external-repository-token"),
261+
auth: actionsUtil.getRequiredInput("token"),
262+
externalRepoAuth: actionsUtil.getOptionalInput("external-repository-token"),
264263
url: getRequiredEnvParam("GITHUB_SERVER_URL"),
265264
apiURL: getRequiredEnvParam("GITHUB_API_URL"),
266265
};

src/util.test.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -299,13 +299,15 @@ const shortTime = 10;
299299

300300
test("withTimeout on long task", async (t) => {
301301
let longTaskTimedOut = false;
302+
let timer : NodeJS.Timeout | undefined = undefined;
302303
const longTask = new Promise((resolve) => {
303-
setTimeout(() => {
304+
timer = setTimeout(() => {
304305
resolve(42);
305306
}, longTime);
306307
});
307308
const result = await util.withTimeout(shortTime, longTask, () => {
308309
longTaskTimedOut = true;
310+
clearTimeout(timer);
309311
});
310312
t.deepEqual(longTaskTimedOut, true);
311313
t.deepEqual(result, undefined);

0 commit comments

Comments
 (0)