Merge pull request #20 from bun913/feat/add-test-case

feat/add test case
bun913 · Aug 27, 2024 · 1141299 · 1141299
2 parents 0f5ba8e + 4927149
commit 1141299
Show file tree

Hide file tree

Showing 3 changed files with 52 additions and 3 deletions.
diff --git a/lib/TestrailReporter.js b/lib/TestrailReporter.js
@@ -10,6 +10,7 @@ const requiredOptions = ["domain", "username", "apikey", "projectId"]
 class TestRailReporter {
   constructor(emitter, reporterOptions, options) {
     this.results = []
+    this.testRailApi = new TestRailApi(request)
     emitter.on("beforeDone", (err, args) => {
       this.onComplete(err, args)
     })
@@ -43,7 +44,7 @@ class TestRailReporter {
     // If we're matching by full title, transform the names to use C123 titles
     //   so we don't have to modify follow up code
     if (this.env.useTitles.toLowerCase() === "true") {
-      const testRailApi = new TestRailApi(request)
+      const testRailApi = this.testRailApi
       const allCases = testRailApi.getCases()
 
       filteredExecutions.forEach((execution) => {
@@ -201,7 +202,7 @@ class TestRailReporter {
 
   pushToTestrail(summary) {
     if (this.results.length > 0) {
-      const testRailApi = new TestRailApi(request)
+      const testRailApi = this.testRailApi
       // TODO: not repeatable use responce variable
       let response
 

diff --git a/test/TestrailReporter.spec.js b/test/TestrailReporter.spec.js
@@ -201,6 +201,49 @@ describe("TestrailReporter", () => {
       })
     })
 
+    describe("User wants to establish connection by title not by case id", () => {
+      describe("When TESTRAIL_TITLE_MATCHING env value is `true`", () => {
+        it("Newman assertions with title `TestTitle` matched to TestRail test case with `TestTitle`", () => {
+          // arrange
+          setEnvVars(vi)
+          vi.stubEnv("TESTRAIL_TITLE_MATCHING", "true")
+          const sut = new TestrailReporter(makeSampleEmitter(vi.fn()), {}, {})
+          sut.testRailApi.getCases = vi
+            .fn()
+            .mockReturnValueOnce([{ id: 1, title: "TestTitle" }])
+          sut.env = getEnv()
+          const executions = makeNewmanResult({ assertionName: "TestTitle" })
+
+          // act
+          sut.jsonifyResults(executions)
+
+          // assert
+          expect(sut.results).lengthOf(1)
+          expect(sut.results[0].case_id).toBe("1")
+        })
+
+        it("Newman assertions with title `NoMatchedTitle` not matched to TestRail test case with `TestTitle`", () => {
+          // arrange
+          setEnvVars(vi)
+          vi.stubEnv("TESTRAIL_TITLE_MATCHING", "true")
+          const sut = new TestrailReporter(makeSampleEmitter(vi.fn()), {}, {})
+          sut.testRailApi.getCases = vi
+            .fn()
+            .mockReturnValueOnce([{ id: 1, title: "TestTitle" }])
+          sut.env = getEnv()
+          const executions = makeNewmanResult({
+            assertionName: "NoMatchedTitle",
+          })
+
+          // act
+          sut.jsonifyResults(executions)
+
+          // assert
+          expect(sut.results).lengthOf(0)
+        })
+      })
+    })
+
     describe("test rail v1 API", () => {
       it("works for TestRail v1 api", () => {
         // arrange

diff --git a/test/utils/newman.js b/test/utils/newman.js
@@ -1,9 +1,14 @@
 module.exports = function makeNewmanResult({
   caseNumbers = "C01",
+  assertionName = "",
   error = false,
   skipped = false,
   tldOnly = false,
 } = {}) {
+  let assertionFullName = assertionName
+  if (assertionName === "") {
+    assertionFullName = `${caseNumbers} Status code is 400`
+  }
   const json = {
     cursor: {},
     item: {
@@ -74,7 +79,7 @@ module.exports = function makeNewmanResult({
     id: "59395a21-ad0a-4baf-b452-beddb3d471a4",
     assertions: [
       {
-        assertion: `${caseNumbers} Status code is 400`,
+        assertion: assertionFullName,
         skipped,
       },
     ],