@sap-ux/ui5-test-writer 0.7.96 → 0.7.97

package/dist/types.d.ts

@@ -67,12 +67,26 @@ export type ObjectPageNavigationParents = {
     parentOPName?: string;
     parentOPTableSection?: string;
 };
+export type BodySubSectionFeatureData = {
+    id: string;
+    isTable: boolean;
+    custom: boolean;
+    order: number;
+};
+export type BodySectionFeatureData = {
+    id: string;
+    isTable: boolean;
+    custom: boolean;
+    order: number;
+    subSections: BodySubSectionFeatureData[];
+};
 export type ObjectPageFeatures = {
     name?: string;
     navigationParents?: ObjectPageNavigationParents;
     headerTitle?: string;
     headerDescription?: string;
     headerSections?: HeaderSectionFeatureData[];
+    bodySections?: BodySectionFeatureData[];
 };
 export type ListReportFeatures = {
     name?: string;

package/dist/utils/modelUtils.d.ts

@@ -19,6 +19,7 @@ export interface SectionItem extends AggregationItem {
     title?: string;
     custom?: boolean;
     name?: string;
+    order?: number;
     schema: {
         keys: {
             name: string;
@@ -27,6 +28,9 @@ export interface SectionItem extends AggregationItem {
         dataType?: string;
     };
 }
+export interface BodySectionItem extends SectionItem {
+    isTable?: boolean;
+}
 export interface HeaderSectionItem extends SectionItem {
     properties: {
         stashed: {

package/dist/utils/objectPageUtils.js

@@ -25,6 +25,8 @@ async function getObjectPageFeatures(objectPages, listReportPageKey, log) {
         pageFeatureData.navigationParents = getObjectPageNavigationParents(objectPage.name, objectPages, listReportPageKey);
         // extract header sections (facets)
         pageFeatureData.headerSections = extractObjectPageHeaderSectionsData(objectPage);
+        // extract body sections
+        pageFeatureData.bodySections = extractObjectPageBodySectionsData(objectPage);
         objectPageFeatures.push(pageFeatureData);
     }
     return objectPageFeatures;
@@ -103,6 +105,53 @@ function extractObjectPageHeaderSectionsData(objectPage) {
     }
     return headerSections;
 }
+/**
+ * Extracts body sections data from an object page model.
+ *
+ * @param objectPage - object page from the application model
+ * @returns body sections data including sub-sections
+ */
+function extractObjectPageBodySectionsData(objectPage) {
+    const bodySections = [];
+    if (objectPage.model) {
+        const sectionsAggregation = (0, modelUtils_1.getAggregations)(objectPage.model.root)['sections'];
+        const sections = (0, modelUtils_1.getAggregations)(sectionsAggregation);
+        Object.entries(sections).forEach(([sectionKey, section]) => {
+            const sectionId = getSectionIdentifier(section) ?? sectionKey;
+            const subSections = extractBodySubSectionsData(section, sectionId);
+            bodySections.push({
+                id: sectionId,
+                isTable: !!section.isTable,
+                custom: !!section.custom,
+                order: section?.order ?? -1, // put a negative order number to signal that order was not in spec
+                subSections
+            });
+        });
+    }
+    return bodySections;
+}
+/**
+ * Extracts sub-sections data from a body section.
+ *
+ * @param section - body section entry from the application model
+ * @param parentSectionId - identifier of the parent section (used as fallback key prefix)
+ * @returns array of sub-section feature data
+ */
+function extractBodySubSectionsData(section, parentSectionId) {
+    const subSections = [];
+    const subSectionsAggregation = (0, modelUtils_1.getAggregations)(section)['subSections'];
+    const subSectionItems = (0, modelUtils_1.getAggregations)(subSectionsAggregation);
+    Object.entries(subSectionItems).forEach(([subSectionKey, subSection]) => {
+        const subSectionId = getSectionIdentifier(subSection) ?? `${parentSectionId}_${subSectionKey}`;
+        subSections.push({
+            id: subSectionId,
+            isTable: !!subSection.isTable,
+            custom: !!subSection.custom,
+            order: subSection?.order ?? -1 // put a negative order number to signal that order was not in spec
+        });
+    });
+    return subSections;
+}
 /**
  * Gets the identifier of a section for OPA5 tests.
  *
@@ -113,14 +162,14 @@ function getSectionIdentifier(section) {
     return getSectionIdentifierFromKey(section) ?? getSectionIdentifierFromTitle(section);
 }
 /**
- * Gets the identifier of a section from the 'ID' entry in the schema keys for OPA5 tests.
+ * Gets the identifier of a section from the 'ID' or 'Key' entry in the schema keys for OPA5 tests.
  * If no such entry is found, undefined is returned.
  *
  * @param section - section entry from ux specification
- * @returns identifier of the section for OPA5 tests; can be undefined if no 'ID' entry is found
+ * @returns identifier of the section for OPA5 tests; can be undefined if no 'ID' or 'Key' entry is found
  */
 function getSectionIdentifierFromKey(section) {
-    const keyEntry = section?.schema?.keys?.find((key) => key.name === 'ID');
+    const keyEntry = section?.schema?.keys?.find((key) => key.name === 'ID' || key.name === 'Key');
     return keyEntry ? keyEntry.value.replace('#', '::') : undefined;
 }
 /**

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@sap-ux/ui5-test-writer",
   "description": "SAP UI5 tests writer",
-  "version": "0.7.96",
+  "version": "0.7.97",
   "repository": {
     "type": "git",
     "url": "https://github.com/SAP/open-ux-tools.git",
@@ -27,9 +27,9 @@
     "@sap/ux-specification": "1.144.0",
     "@sap-ux/edmx-parser": "0.10.0",
     "@sap-ux/annotation-converter": "0.10.21",
+    "@sap-ux/ui5-application-writer": "1.8.3",
     "@sap-ux/logger": "0.8.4",
-    "@sap-ux/project-access": "1.35.17",
-    "@sap-ux/ui5-application-writer": "1.8.3"
+    "@sap-ux/project-access": "1.35.17"
   },
   "devDependencies": {
     "@types/ejs": "3.1.5",

package/templates/v4/integration/ObjectPageJourney.js

@@ -59,6 +59,22 @@ sap.ui.define([
         });
 <% } -%>
 
+<% if (bodySections?.length > 0) { -%>
+        opaTest("Check body sections of the Object Page", function (Given, When, Then) {
+            Then.onThe<%- name%>.iCheckNumberOfSections(<%- bodySections.length %>);
+<% bodySections.forEach(function(section) { -%>
+            When.onThe<%- name%>.iGoToSection({ section: "<%- section.id %>" });
+            Then.onThe<%- name%>.iCheckSection({ section: "<%- section.id %>" });
+<% if (section?.subSections?.length > 0) { -%>
+<% section.subSections.forEach(function(subSection) { -%>
+            When.onThe<%- name%>.iGoToSection({ section: "<%- section.id %>", subSection: "<%- subSection.id %>" });
+            Then.onThe<%- name%>.iCheckSubSection({ section: "<%- subSection.id %>" });
+<% }) -%>
+<% } -%>
+<% }) -%>
+       });
+<% } -%>
+
         opaTest("Teardown", function (Given, When, Then) { 
             // Cleanup
             Given.iTearDownMyApp();