@sap-ux/ui5-test-writer 1.2.3 → 1.2.4

package/dist/utils/objectPageUtils.js

@@ -135,21 +135,22 @@ function extractObjectPageBodySectionsData(objectPage, convertedMetadata, schema
             const sectionId = getSectionIdentifier(section) ?? sectionKey;
             const subSections = extractBodySubSectionsData(section, sectionId);
             const navigationProperty = getNavigationPropertyFromKey(sectionKey);
+            const isTable = isTableSection(section);
             const sectionData = {
                 id: sectionId,
                 navigationProperty,
-                isTable: !!section.isTable,
+                isTable,
                 custom: !!section.custom,
                 order: section?.order ?? -1,
-                fields: section.custom || section.isTable ? [] : extractFormFields(section),
-                tableColumns: section.custom || !section.isTable ? {} : extractTableColumnsFromNode(section),
+                fields: section.custom || isTable ? [] : extractFormFields(section),
+                tableColumns: section.custom || !isTable ? {} : extractTableColumnsFromNode(section),
                 subSections,
                 actions: !section.custom && convertedMetadata && schemaNamespace
                     ? extractSectionActions(section, convertedMetadata, schemaNamespace)
                     : []
             };
             // For table sections, resolve Create/Delete visibility from target entity set
-            if (section.isTable && navigationProperty && metadata && convertedMetadata) {
+            if (isTable && navigationProperty && metadata && convertedMetadata) {
                 const targetEntitySet = resolveNavigationTargetEntitySet(convertedMetadata, objectPage.entitySet, navigationProperty);
                 if (targetEntitySet) {
                     const buttonVisibility = safeCheckButtonVisibility(metadata, targetEntitySet, log);
@@ -192,7 +193,7 @@ function extractHeaderActions(objectPage, convertedMetadata, schemaNamespace) {
  */
 function extractSectionActions(section, convertedMetadata, schemaNamespace) {
     let actionsAgg;
-    if (section.isTable) {
+    if (isTableSection(section)) {
         const tableAgg = getAggregations(section)['table'];
         const toolBarAgg = getAggregations(tableAgg)['toolBar'];
         actionsAgg = getAggregations(toolBarAgg)['actions'];
@@ -222,14 +223,15 @@ function extractBodySubSectionsData(section, parentSectionId) {
     const subSectionItems = getAggregations(subSectionsAggregation);
     Object.entries(subSectionItems).forEach(([subSectionKey, subSection]) => {
         const subSectionId = getSectionIdentifier(subSection) ?? `${parentSectionId}_${subSectionKey}`;
+        const isTable = isTableSection(subSection);
         subSections.push({
             id: subSectionId,
             navigationProperty: getNavigationPropertyFromKey(subSectionKey),
-            isTable: !!subSection.isTable,
+            isTable,
             custom: !!subSection.custom,
             order: subSection?.order ?? -1, // put a negative order number to signal that order was not in spec
-            fields: subSection.custom || subSection.isTable ? [] : extractFormFields(subSection),
-            tableColumns: subSection.custom || !subSection.isTable ? {} : extractTableColumnsFromNode(subSection)
+            fields: subSection.custom || isTable ? [] : extractFormFields(subSection),
+            tableColumns: subSection.custom || !isTable ? {} : extractTableColumnsFromNode(subSection)
         });
     });
     return subSections;
@@ -392,6 +394,18 @@ function getFieldGroupQualifier(formAggregation) {
 function isSectionMicroChart(section) {
     return section?.schema?.dataType === 'ChartDefinition';
 }
+/**
+ * Detects whether a body section represents a table.
+ * The spec model exposes the section-level `isTable` flag inconsistently — for OP body sections
+ * driven by `_<NavProp>/@UI.LineItem` facets the flag is not set, but the section carries a
+ * `table` aggregation. Presence of that aggregation is the authoritative signal.
+ *
+ * @param section - body section or sub-section entry from ux specification
+ * @returns true if the section is a table section
+ */
+function isTableSection(section) {
+    return !!section.isTable || !!getAggregations(section).table;
+}
 /**
  * Checks if the section contains a form based on it's aggregations.
  *

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@sap-ux/ui5-test-writer",
   "description": "SAP UI5 tests writer",
-  "version": "1.2.3",
+  "version": "1.2.4",
   "repository": {
     "type": "git",
     "url": "https://github.com/SAP/open-ux-tools.git",
@@ -28,11 +28,11 @@
     "@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": "2.0.4",
+    "@sap-ux/ui5-application-writer": "2.0.5",
     "@sap-ux/logger": "1.0.1",
-    "@sap-ux/fiori-generator-shared": "1.0.11",
-    "@sap-ux/project-access": "2.1.2",
-    "@sap-ux/preview-middleware": "1.0.27"
+    "@sap-ux/fiori-generator-shared": "1.0.12",
+    "@sap-ux/project-access": "2.1.3",
+    "@sap-ux/preview-middleware": "1.0.28"
   },
   "devDependencies": {
     "@jest/globals": "30.3.0",