Submit Defect + reusing field methods for requirements

Author: GTFalcao

components/tricentis_qtest/actions/create-requirement/create-requirement.mjs

@@ -1,5 +1,5 @@
 import {
-  getRequirementFieldProps as additionalProps, getProperties,
+  getFieldProps as additionalProps, getProperties,
 } from "../../common/utils.mjs";
 import tricentisQtest from "../../tricentis_qtest.app.mjs";
 
@@ -40,11 +40,14 @@ export default {
   },
   additionalProps,
   methods: {
+    getDataFields() {
+      return this.tricentisQtest.getRequirementFields(this.projectId);
+    },
     getProperties,
   },
   async run({ $ }) {
     const { // eslint-disable-next-line no-unused-vars
-      tricentisQtest, projectId, parentId, useFields, name, getProperties, ...fields
+      tricentisQtest, projectId, parentId, useFields, name, getProperties, getDataFields, ...fields
     } = this;
     const response = await tricentisQtest.createRequirement({
       $,

components/tricentis_qtest/actions/submit-defect/submit-defect.mjs

@@ -0,0 +1,43 @@
+import {
+  getFieldProps as additionalProps, getProperties,
+} from "../../common/utils.mjs";
+import tricentisQtest from "../../tricentis_qtest.app.mjs";
+
+export default {
+  key: "tricentis_qtest-submit-defect",
+  name: "Submit Defect",
+  description: "Submit a new defect. [See the documentation](https://documentation.tricentis.com/qtest/od/en/content/apis/apis/defect_apis.htm#SubmitaDefect)",
+  version: "0.0.{{ts}}",
+  type: "action",
+  props: {
+    tricentisQtest,
+    projectId: {
+      propDefinition: [
+        tricentisQtest,
+        "projectId",
+      ],
+      reloadProps: true,
+    },
+  },
+  additionalProps,
+  methods: {
+    getDataFields() {
+      return this.tricentisQtest.getDefectFields(this.projectId);
+    },
+    getProperties,
+  },
+  async run({ $ }) {
+    const { // eslint-disable-next-line no-unused-vars
+      tricentisQtest, projectId, getProperties, getDataFields, ...fields
+    } = this;
+    const response = await tricentisQtest.createDefect({
+      $,
+      projectId,
+      data: {
+        properties: getProperties(fields),
+      },
+    });
+    $.export("$summary", `Successfully submitted defect (ID: ${response.id})`);
+    return response;
+  },
+};

components/tricentis_qtest/actions/update-requirement/update-requirement.mjs

@@ -1,5 +1,5 @@
 import {
-  getRequirementFieldProps as additionalProps, getProperties,
+  getFieldProps as additionalProps, getProperties,
 } from "../../common/utils.mjs";
 import tricentisQtest from "../../tricentis_qtest.app.mjs";
 
@@ -40,12 +40,22 @@ export default {
   },
   additionalProps,
   methods: {
+    getDataFields() {
+      return this.tricentisQtest.getRequirementFields(this.projectId);
+    },
     getProperties,
   },
   async run({ $ }) {
-    const { // eslint-disable-next-line no-unused-vars
-      tricentisQtest, projectId, requirementId, useFields, name, getProperties, ...fields
-    } = this;
+    const { /* eslint-disable no-unused-vars */
+      tricentisQtest,
+      projectId,
+      requirementId,
+      useFields,
+      name,
+      getProperties,
+      getDataFields,
+      ...fields
+    } = this; /* eslint-enable no-unused-vars */
     const response = await tricentisQtest.updateRequirement({
       $,
       projectId,

components/tricentis_qtest/common/utils.mjs

@@ -1,7 +1,7 @@
-export async function getRequirementFieldProps() {
-  if (!this.useFields) return {};
+export async function getFieldProps() {
+  if (this.useFields === false) return {};
 
-  const fields = await this.tricentisQtest.getRequirementFields(this.projectId);
+  const fields = await this.getDataFields();
 
   function getFieldType(type) {
     switch (type) {
@@ -15,22 +15,28 @@ export async function getRequirementFieldProps() {
     }
   }
 
+  console.log(fields);
+
   const result = {};
+  const isUpdate = this.requirementId || this.defectId;
 
   fields?.forEach(({
-    id, label, attribute_type: type, allowed_values: options,
+    id, label, attribute_type: fieldType, allowed_values: options, required,
   }) => {
+    const type = getFieldType(fieldType);
     result[`field_${id}`] = {
       label,
-      type: getFieldType(type),
+      type,
       description: `Field ID: ${id}`,
-      optional: true,
+      optional: isUpdate || !required,
       ...(options && {
         options: options.map(({
           label, value,
         }) => ({
           label,
-          value,
+          value: (type === "string" && typeof value !== "string")
+            ? value.toString()
+            : value,
         })),
       }),
     };