test: fix process.argv reset during cli tests (#60)

This commit is contained in:
Andrea Lamparelli 2023-07-11 22:43:22 +02:00 committed by GitHub
parent 49a7350406
commit 91782505ce
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
6 changed files with 15 additions and 18 deletions

View file

@ -44,11 +44,11 @@ describe("cli args parser", () => {
});
beforeEach(() => {
// create a fresh new instance every time
parser = new CLIArgsParser();
// reset process.env variables
resetProcessArgs();
// create a fresh new instance every time
parser = new CLIArgsParser();
});
test("valid execution [default, short]", () => {

View file

@ -63,11 +63,12 @@ describe("github pull request config parser", () => {
});
beforeEach(() => {
mockGitHubClient("http://localhost/api/v3");
// reset process.env variables
resetProcessArgs();
// mock octokit
mockGitHubClient("http://localhost/api/v3");
// create a fresh new instance every time
argsParser = new CLIArgsParser();
configParser = new PullRequestConfigsParser();

View file

@ -45,6 +45,10 @@ afterAll(() => {
});
beforeEach(() => {
// reset process.env variables
resetProcessArgs();
// mock octokit
mockGitHubClient();
// create CLI arguments parser
@ -54,11 +58,6 @@ beforeEach(() => {
runner = new Runner(parser);
});
afterEach(() => {
// reset process.env variables
resetProcessArgs();
});
describe("cli runner", () => {
test("with dry run", async () => {

View file

@ -60,6 +60,9 @@ afterAll(() => {
});
beforeEach(() => {
// reset process.env variables
resetProcessArgs();
// create CLI arguments parser
parser = new CLIArgsParser();
@ -67,13 +70,7 @@ beforeEach(() => {
runner = new Runner(parser);
});
afterEach(() => {
// reset process.env variables
resetProcessArgs();
});
describe("cli runner", () => {
test("with dry run", async () => {
addProcessArgs([
"-d",