Skip to content

Commit

Permalink
chore: run format
Browse files Browse the repository at this point in the history
  • Loading branch information
@bun913
bun913 committed Aug 24, 2024
1 parent d5ab070 commit ca802b9
Show file tree
Hide file tree
Showing 3 changed files with 69 additions and 60 deletions.
101 changes: 55 additions & 46 deletions test/lib/testRailApi.spec.js
View file
Original file line number Diff line number Diff line change
@@ -1,63 +1,65 @@
import { afterEach, describe, expect, it, vi } from "vitest";
import { makeFakeRequest, makeFakeTestRailApi } from "../utils/request";
import { setEnvVars } from "../utils/env";
import { afterEach, describe, expect, it, vi } from "vitest"
import { setEnvVars } from "../utils/env"
import { makeFakeRequest, makeFakeTestRailApi } from "../utils/request"

describe("TestRailApi", () => {
afterEach(() => {
vi.resetAllMocks();
vi.unstubAllEnvs();
});
vi.resetAllMocks()
vi.unstubAllEnvs()
})

describe("getProjectInfo", () => {
it("sends reqeust to DOMAIN/api/v2/get_project/testProjectId when suiteId is not set", () => {
// arrange
setEnvVars(vi, { projectId: "123" });
setEnvVars(vi, { projectId: "123" })
const fakeRequest = makeFakeRequest(vi)
const sut = makeFakeTestRailApi(vi, fakeRequest, { projectId: "123" });
const sut = makeFakeTestRailApi(vi, fakeRequest, { projectId: "123" })

// act
sut.getProjectInfo();
sut.getProjectInfo()

// assert
expect(sut.request).toHaveBeenCalledWith(
"GET",
"https://example.com/index.php?/api/v2/get_project/testProjectId",
expect.anything()
expect.anything(),
)
});
})

it("sends request to DOMAIN/api_v2_get_suite/:suiteId when suiteId is set", () => {
// arrange
setEnvVars(vi, { TESTRAIL_SUITEID: "456", TESTRAIL_PROJECTID: "123" });
setEnvVars(vi, { TESTRAIL_SUITEID: "456", TESTRAIL_PROJECTID: "123" })
const fakeRequest = makeFakeRequest(vi)
const sut = makeFakeTestRailApi(vi, fakeRequest);
const sut = makeFakeTestRailApi(vi, fakeRequest)

// act
sut.getProjectInfo();
sut.getProjectInfo()

// assert
expect(sut.request).toHaveBeenCalledWith(
"GET",
"https://example.com/index.php?/api/v2/get_suite/456",
expect.anything()
expect.anything(),
)
})
});
})

describe("getRuns", () => {
it("sends request to DOMAIN/api/v2/get_runs/:projectId when suiteId is not set", () => {
// arrange
const fakeRequest = makeFakeRequest(vi)
const sut = makeFakeTestRailApi(vi, fakeRequest, { TESTRAIL_PROJECTID: "123" });
const sut = makeFakeTestRailApi(vi, fakeRequest, {
TESTRAIL_PROJECTID: "123",
})

// act
sut.getRuns();
sut.getRuns()

// assert
expect(sut.request).toHaveBeenCalledWith(
"GET",
"https://example.com/index.php?/api/v2/get_runs/123",
expect.anything()
expect.anything(),
)
})
})
Expand All @@ -66,16 +68,16 @@ describe("TestRailApi", () => {
it("sends request to DOMAIN/api/v2/get_run/:runId", () => {
// arrange
const fakeRequest = makeFakeRequest(vi)
const sut = makeFakeTestRailApi(vi, fakeRequest);
const sut = makeFakeTestRailApi(vi, fakeRequest)

// act
sut.getRun("456");
sut.getRun("456")

// assert
expect(sut.request).toHaveBeenCalledWith(
"GET",
"https://example.com/index.php?/api/v2/get_run/456",
expect.anything()
expect.anything(),
)
})
})
Expand All @@ -84,65 +86,72 @@ describe("TestRailApi", () => {
it("sends request to DOMAIN/api/v2/get_cases/:projectId when suiteId is not set", () => {
// arrange
const fakeRequest = makeFakeRequest(vi, {
"_links": {
"next": null
_links: {
next: null,
},
cases: [
{ id: 1, title: "case1" },
{ id: 2, title: "case2" },
]
],
})
const sut = makeFakeTestRailApi(vi, fakeRequest, {
TESTRAIL_PROJECTID: "123",
})
const sut = makeFakeTestRailApi(vi, fakeRequest, { TESTRAIL_PROJECTID: "123" });

// act
sut.getCases();
sut.getCases()

// assert
expect(sut.request).toHaveBeenCalledWith(
"GET",
"https://example.com/index.php?/api/v2/get_cases/123",
expect.anything()
expect.anything(),
)
})

it("sends request to DOMAIN/api/v2/get_cases/:projectId&suite_id=:suiteId when suiteId is set", () => {
// arrange
const fakeRequest = makeFakeRequest(vi, {
"_links": {
"next": null
_links: {
next: null,
},
cases: [
{ id: 1, title: "case1" },
{ id: 2, title: "case2" },
]
],
})
const sut = makeFakeTestRailApi(vi, fakeRequest, {
TESTRAIL_PROJECTID: "123",
TESTRAIL_SUITEID: "456",
})
const sut = makeFakeTestRailApi(vi, fakeRequest, { TESTRAIL_PROJECTID: "123", TESTRAIL_SUITEID: "456" });

// act
sut.getCases();
sut.getCases()

// assert
expect(sut.request).toHaveBeenCalledWith(
"GET",
"https://example.com/index.php?/api/v2/get_cases/123&suite_id=456",
expect.anything()
expect.anything(),
)
})

describe("addRun", () => {
it("sends request to DOMAIN/api/v2/add_run/:projectId", () => {
// arrange
const fakeRequest = makeFakeRequest(vi)
const sut = makeFakeTestRailApi(vi, fakeRequest, { TESTRAIL_PROJECTID: "123" });
const sut = makeFakeTestRailApi(vi, fakeRequest, {
TESTRAIL_PROJECTID: "123",
})

// act
sut.addRun("title", []);
sut.addRun("title", [])

// assert
expect(sut.request).toHaveBeenCalledWith(
"POST",
"https://example.com/index.php?/api/v2/add_run/123",
expect.anything()
expect.anything(),
)
})
})
Expand All @@ -151,16 +160,18 @@ describe("TestRailApi", () => {
it("sends request to DOMAIN/api/v2/add_plan_entry/:planId", () => {
// arrange
const fakeRequest = makeFakeRequest(vi)
const sut = makeFakeTestRailApi(vi, fakeRequest, { TESTRAIL_PLANID: "123" });
const sut = makeFakeTestRailApi(vi, fakeRequest, {
TESTRAIL_PLANID: "123",
})

// act
sut.addPlanEntry("title", []);
sut.addPlanEntry("title", [])

// assert
expect(sut.request).toHaveBeenCalledWith(
"POST",
"https://example.com/index.php?/api/v2/add_plan_entry/123",
expect.anything()
expect.anything(),
)
})
})
Expand All @@ -169,22 +180,20 @@ describe("TestRailApi", () => {
it("sends request to DOMAIN/api/v2/close_run/:runId", () => {
// arrange
const fakeRequest = makeFakeRequest(vi)
const sut = makeFakeTestRailApi(vi, fakeRequest);
const sut = makeFakeTestRailApi(vi, fakeRequest)

// act
sut.closeRun("456");
sut.closeRun("456")

// assert
expect(sut.request).toHaveBeenCalledWith(
"POST",
"https://example.com/index.php?/api/v2/close_run/456",
expect.anything()
expect.anything(),
)
})
})
})

// TODO: refactor pushToTestrail and write tests using fakes


});
})
20 changes: 10 additions & 10 deletions test/utils/request.js
View file
Original file line number Diff line number Diff line change
@@ -1,18 +1,18 @@
import { setEnvVars } from "./env";
import TestRailApi from "../../lib/testRailApi";
import TestRailApi from "../../lib/testRailApi"
import { setEnvVars } from "./env"

export const makeFakeRequest = (vi, fakeResponse={"fake": true}) => {
export const makeFakeRequest = (vi, fakeResponse = { fake: true }) => {
const mockRequest = vi.fn((method, url, options) => {
return {
statusCode: 200,
getBody: () => JSON.stringify(fakeResponse),
};
});
return mockRequest;
};
}
})
return mockRequest
}

export const makeFakeTestRailApi = (vi,fakeRequest, envOverrids = {}) => {
export const makeFakeTestRailApi = (vi, fakeRequest, envOverrids = {}) => {
// fake TestRailApi
setEnvVars(vi, envOverrids);
setEnvVars(vi, envOverrids)
return new TestRailApi(fakeRequest)
};
}
8 changes: 4 additions & 4 deletions vitest.config.js
View file
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
import { defineConfig} from 'vitest/config'
import { defineConfig } from "vitest/config"

export default defineConfig({
test: {
globals: true
}
test: {
globals: true,
},
})

0 comments on commit ca802b9

Please sign in to comment.