Add site metadata fields to diff handlers and tests

- ✨ Introduced new metadata fields: dataModelVersion, websiteUrl, siteVisibility, creator, and createdOn in ExportMetadataDiffHandler, GenerateHtmlReportHandler, and ResyncMetadataDiffHandler.
- 🧪 Updated tests in MetadataDiffSiteTreeItem to validate new fields and their handling of optional values.

-Priyanshu

Author: priyanshu92

src/client/test/Integration/power-pages/actions-hub/handlers/metadata-diff/ExportMetadataDiffHandler.test.ts

@@ -9,6 +9,7 @@ import sinon from "sinon";
 import { exportMetadataDiff } from "../../../../../../power-pages/actions-hub/handlers/metadata-diff/ExportMetadataDiffHandler";
 import { MetadataDiffSiteTreeItem } from "../../../../../../power-pages/actions-hub/tree-items/metadata-diff/MetadataDiffSiteTreeItem";
 import { FileComparisonStatus, IFileComparisonResult, ISiteComparisonResults } from "../../../../../../power-pages/actions-hub/models/IFileComparisonResult";
+import { SiteVisibility } from "../../../../../../power-pages/actions-hub/models/SiteVisibility";
 import * as TelemetryHelper from "../../../../../../power-pages/actions-hub/TelemetryHelper";
 
 describe("ExportMetadataDiffHandler", () => {
@@ -43,7 +44,12 @@ describe("ExportMetadataDiffHandler", () => {
             localSiteName,
             environmentName,
             websiteId,
-            environmentId
+            environmentId,
+            dataModelVersion: 2,
+            websiteUrl: "https://test-site.powerappsportals.com",
+            siteVisibility: SiteVisibility.Public,
+            creator: "test-creator@contoso.com",
+            createdOn: "2024-01-15T10:30:00Z"
         };
     }
 

src/client/test/Integration/power-pages/actions-hub/handlers/metadata-diff/GenerateHtmlReportHandler.test.ts

@@ -9,6 +9,7 @@ import sinon from "sinon";
 import { generateHtmlReport } from "../../../../../../power-pages/actions-hub/handlers/metadata-diff/GenerateHtmlReportHandler";
 import { MetadataDiffSiteTreeItem } from "../../../../../../power-pages/actions-hub/tree-items/metadata-diff/MetadataDiffSiteTreeItem";
 import { FileComparisonStatus, IFileComparisonResult, ISiteComparisonResults } from "../../../../../../power-pages/actions-hub/models/IFileComparisonResult";
+import { SiteVisibility } from "../../../../../../power-pages/actions-hub/models/SiteVisibility";
 import * as TelemetryHelper from "../../../../../../power-pages/actions-hub/TelemetryHelper";
 
 describe("GenerateHtmlReportHandler", () => {
@@ -38,7 +39,12 @@ describe("GenerateHtmlReportHandler", () => {
             localSiteName,
             environmentName,
             websiteId: "test-website-id",
-            environmentId: "test-environment-id"
+            environmentId: "test-environment-id",
+            dataModelVersion: 2,
+            websiteUrl: "https://test-site.powerappsportals.com",
+            siteVisibility: SiteVisibility.Public,
+            creator: "test-creator@contoso.com",
+            createdOn: "2024-01-15T10:30:00Z"
         };
     }
 

src/client/test/Integration/power-pages/actions-hub/handlers/metadata-diff/ResyncMetadataDiffHandler.test.ts

@@ -11,6 +11,7 @@ import { Constants } from "../../../../../../power-pages/actions-hub/Constants";
 import { PacTerminal } from "../../../../../../lib/PacTerminal";
 import { MetadataDiffSiteTreeItem } from "../../../../../../power-pages/actions-hub/tree-items/metadata-diff/MetadataDiffSiteTreeItem";
 import { ISiteComparisonResults, FileComparisonStatus } from "../../../../../../power-pages/actions-hub/models/IFileComparisonResult";
+import { SiteVisibility } from "../../../../../../power-pages/actions-hub/models/SiteVisibility";
 import MetadataDiffContext from "../../../../../../power-pages/actions-hub/MetadataDiffContext";
 import * as TelemetryHelper from "../../../../../../power-pages/actions-hub/TelemetryHelper";
 import * as WorkspaceInfoFinderUtil from "../../../../../../../common/utilities/WorkspaceInfoFinderUtil";
@@ -67,6 +68,11 @@ describe("ResyncMetadataDiffHandler", () => {
                 }
             ],
             isImported: false,
+            dataModelVersion: 2,
+            websiteUrl: "https://test-site.powerappsportals.com",
+            siteVisibility: SiteVisibility.Public,
+            creator: "test-creator@contoso.com",
+            createdOn: "2024-01-15T10:30:00Z",
             ...overrides
         };
     }

src/client/test/Integration/power-pages/actions-hub/tree-items/metadata-diff/MetadataDiffSiteTreeItem.test.ts

@@ -12,6 +12,7 @@ import { ActionsHubTreeItem } from "../../../../../../power-pages/actions-hub/tr
 import { IFileComparisonResult, FileComparisonStatus, ISiteComparisonResults } from "../../../../../../power-pages/actions-hub/models/IFileComparisonResult";
 import { Constants } from "../../../../../../power-pages/actions-hub/Constants";
 import MetadataDiffContext, { MetadataDiffViewMode, MetadataDiffSortMode } from "../../../../../../power-pages/actions-hub/MetadataDiffContext";
+import { SiteVisibility } from "../../../../../../power-pages/actions-hub/models/SiteVisibility";
 
 /**
  * Helper function to create ISiteComparisonResults for testing
@@ -24,7 +25,12 @@ function createSiteResults(
     websiteId = "test-website-id",
     environmentId = "test-environment-id",
     isImported = false,
-    exportedAt?: string
+    exportedAt?: string,
+    dataModelVersion: 1 | 2 = 2,
+    websiteUrl = "https://test-site.powerappsportals.com",
+    siteVisibility = SiteVisibility.Public,
+    creator = "test-creator@contoso.com",
+    createdOn = "2024-01-15T10:30:00Z"
 ): ISiteComparisonResults {
     return {
         comparisonResults,
@@ -34,7 +40,12 @@ function createSiteResults(
         websiteId,
         environmentId,
         isImported,
-        exportedAt
+        exportedAt,
+        dataModelVersion,
+        websiteUrl,
+        siteVisibility,
+        creator,
+        createdOn
     };
 }
 
@@ -179,6 +190,56 @@ describe("MetadataDiffSiteTreeItem", () => {
         });
     });
 
+    describe("site details", () => {
+        it("should return the dataModelVersion", () => {
+            const treeItem = new MetadataDiffSiteTreeItem(createSiteResults([], "Test Site", "Local Test Site", "Test Environment", "web-id", "env-id", false, undefined, 2));
+
+            expect(treeItem.dataModelVersion).to.equal(2);
+        });
+
+        it("should return the websiteUrl", () => {
+            const treeItem = new MetadataDiffSiteTreeItem(createSiteResults([], "Test Site", "Local Test Site", "Test Environment", "web-id", "env-id", false, undefined, 2, "https://my-site.powerappsportals.com"));
+
+            expect(treeItem.websiteUrl).to.equal("https://my-site.powerappsportals.com");
+        });
+
+        it("should return the siteVisibility", () => {
+            const treeItem = new MetadataDiffSiteTreeItem(createSiteResults([], "Test Site", "Local Test Site", "Test Environment", "web-id", "env-id", false, undefined, 2, "https://my-site.powerappsportals.com", SiteVisibility.Private));
+
+            expect(treeItem.siteVisibility).to.equal(SiteVisibility.Private);
+        });
+
+        it("should return the creator", () => {
+            const treeItem = new MetadataDiffSiteTreeItem(createSiteResults([], "Test Site", "Local Test Site", "Test Environment", "web-id", "env-id", false, undefined, 2, "https://my-site.powerappsportals.com", SiteVisibility.Public, "admin@contoso.com"));
+
+            expect(treeItem.creator).to.equal("admin@contoso.com");
+        });
+
+        it("should return the createdOn", () => {
+            const treeItem = new MetadataDiffSiteTreeItem(createSiteResults([], "Test Site", "Local Test Site", "Test Environment", "web-id", "env-id", false, undefined, 2, "https://my-site.powerappsportals.com", SiteVisibility.Public, "admin@contoso.com", "2024-06-01T12:00:00Z"));
+
+            expect(treeItem.createdOn).to.equal("2024-06-01T12:00:00Z");
+        });
+
+        it("should handle undefined optional fields", () => {
+            const results: ISiteComparisonResults = {
+                comparisonResults: [],
+                siteName: "Test Site",
+                localSiteName: "Local Test Site",
+                environmentName: "Test Environment",
+                websiteId: "web-id",
+                environmentId: "env-id"
+            };
+            const treeItem = new MetadataDiffSiteTreeItem(results);
+
+            expect(treeItem.dataModelVersion).to.be.undefined;
+            expect(treeItem.websiteUrl).to.be.undefined;
+            expect(treeItem.siteVisibility).to.be.undefined;
+            expect(treeItem.creator).to.be.undefined;
+            expect(treeItem.createdOn).to.be.undefined;
+        });
+    });
+
     describe("getChildren - list view mode", () => {
         beforeEach(() => {
             MetadataDiffContext.setViewMode(MetadataDiffViewMode.List);