Refactor AppSource Gallery tests to use test library

src/System Application/App/AppSource Gallery/app.json

@@ -62,21 +62,19 @@
                          }
                      ],
     "internalsVisibleTo":  [
-                               {
-                                   "name":  "AppSource Product Gallery Test",
-                                   "id":  "c1830429-77f6-4469-837a-7bfd4705ff41",
-                                   "publisher":  "Microsoft"
-                               }
-                           ],
-    "screenshots":  [
-
-                    ],
-    "platform":  "24.0.0.0",
-    "idRanges":  [
-                     {
-                         "from":  2515,
-                         "to":  2517
-                     }
-                 ],
-    "target":  "OnPrem"
+        {
+            "id":  "ba17b564-d600-44d5-be0b-ca7ff7ac28fc",
+            "name":  "AppSource Gallery Test Library",
+            "publisher": "Microsoft"
+        }
+    ],
+    "screenshots": [],
+    "platform": "24.0.0.0",
+    "idRanges": [
+        {
+            "from": 2515,
+            "to": 2518
+        }
+    ],
+    "target": "OnPrem"
 }

src/System Application/App/AppSource Gallery/src/AppSourceJsonUtilities.Codeunit.al

@@ -5,7 +5,7 @@
 namespace System.Apps.AppSource;
 
 /// <summary>
-/// Library for managing AppSource product retrival and usage.
+/// Library for managing AppSource product retrieval and usage.
 /// </summary>
 codeunit 2516 "AppSource Json Utilities"
 {

src/System Application/App/AppSource Gallery/src/AppSourceProductManager.Codeunit.al

@@ -8,14 +8,13 @@ using System.Environment.Configuration;
 using System.Globalization;
 using System.Azure.Identity;
 using System.Utilities;
-using System.Environment;
 using System.Azure.KeyVault;
 using System.RestClient;
 
 /// <summary>
-/// Library for managing AppSource product retrival and usage.
+/// Library for managing AppSource product retrieval and usage.
 /// </summary>
-codeunit 2515 "AppSource Product Manager" implements "AppSource Product Manager Dependencies"
+codeunit 2515 "AppSource Product Manager"
 {
     Access = Internal;
     InherentEntitlements = X;
@@ -39,66 +38,6 @@ codeunit 2515 "AppSource Product Manager" implements "AppSource Product Manager
         UnsupportedLanguageNotificationLbl: Label 'Language %1 is not supported by AppSource. Defaulting to "en". Change the language in the user profile to use another language.', Comment = '%1=Language ID, such as en';
         UnsupportedMarketNotificationLbl: Label 'Market %1 is not supported by AppSource. Defaulting to "us". Change the region in the user profile to use another market.', Comment = '%1=Market ID, such as "us"';
 
-    #region Dependency Interface implementation
-    procedure GetCountryLetterCode(): Code[2]
-    var
-        AzureAdTenant: Codeunit "Azure AD Tenant";
-    begin
-        exit(AzureAdTenant.GetCountryLetterCode());
-    end;
-
-    procedure GetPreferredLanguage(): Text
-    var
-        AzureAdTenant: Codeunit "Azure AD Tenant";
-    begin
-        exit(AzureAdTenant.GetPreferredLanguage());
-    end;
-
-    procedure GetApplicationFamily(): Text
-    var
-        EnvironmentInformation: Codeunit "Environment Information";
-    begin
-        exit(EnvironmentInformation.GetApplicationFamily());
-    end;
-
-    procedure IsSaas(): boolean
-    var
-        EnvironmentInformation: Codeunit "Environment Information";
-    begin
-        exit(EnvironmentInformation.IsSaas());
-    end;
-
-    procedure GetFormatRegionOrDefault(FormatRegion: Text[80]): Text
-    var
-        Language: Codeunit Language;
-    begin
-        exit(Language.GetFormatRegionOrDefault(FormatRegion));
-    end;
-
-    procedure GetAsJSon(var RestClient: Codeunit "Rest Client"; RequestUri: Text): JsonToken
-    begin
-        exit(RestClient.GetAsJSon(RequestUri));
-    end;
-
-    procedure GetUserSettings(UserSecurityId: Guid; var TempUserSettingsRecord: record "User Settings" temporary)
-    var
-        UserSettings: Codeunit "User Settings";
-    begin
-        UserSettings.GetUserSettings(Database.UserSecurityID(), TempUserSettingsRecord);
-    end;
-
-    procedure ShouldSetCommonHeaders(): Boolean
-    begin
-        exit(true);
-    end;
-
-    #endregion
-    procedure SetDependencies(SpecificDependencies: Interface "AppSource Product Manager Dependencies")
-    begin
-        AppSourceProductManagerDependencies := SpecificDependencies;
-        IsDependenciesInterfaceSet := true;
-    end;
-
     #region Product helpers
     /// <summary>
     /// Opens Microsoft AppSource web page for the region is specified in the UserSessionSettings or 'en-us' by default.
@@ -494,9 +433,14 @@ codeunit 2515 "AppSource Product Manager" implements "AppSource Product Manager
 
     local procedure SetDefaultDependencyImplementation()
     var
-        AppSourceProductManager: Codeunit "AppSource Product Manager";
+        AppSrcProductDepsProvider: Codeunit "AppSrc Product Deps. Provider";
+    begin
+        SetDependencies(AppSrcProductDepsProvider);
+    end;
+
+    internal procedure SetDependencies(AppSourceProductManagerDependencyProvider: Interface "AppSource Product Manager Dependencies")
     begin
-        SetDependencies(AppSourceProductManager);
+        AppSourceProductManagerDependencies := AppSourceProductManagerDependencyProvider;
         IsDependenciesInterfaceSet := true;
     end;
 }

src/System Application/App/AppSource Gallery/src/AppSrcProductDepsProvider.Codeunit.al

@@ -0,0 +1,73 @@
+// ------------------------------------------------------------------------------------------------
+// Copyright (c) Microsoft Corporation. All rights reserved.
+// Licensed under the MIT License. See License.txt in the project root for license information.
+// ------------------------------------------------------------------------------------------------
+namespace System.Apps.AppSource;
+
+using System.Environment.Configuration;
+using System.Globalization;
+using System.Azure.Identity;
+using System.Environment;
+using System.RestClient;
+
+/// <summary>
+/// Provides the dependencies used by the AppSource Gallery module.
+/// </summary>
+codeunit 2518 "AppSrc Product Deps. Provider" implements "AppSource Product Manager Dependencies"
+{
+    Access = Internal;
+    InherentEntitlements = X;
+    InherentPermissions = X;
+
+    procedure GetCountryLetterCode(): Code[2]
+    var
+        AzureAdTenant: Codeunit "Azure AD Tenant";
+    begin
+        exit(AzureAdTenant.GetCountryLetterCode());
+    end;
+
+    procedure GetPreferredLanguage(): Text
+    var
+        AzureAdTenant: Codeunit "Azure AD Tenant";
+    begin
+        exit(AzureAdTenant.GetPreferredLanguage());
+    end;
+
+    procedure GetApplicationFamily(): Text
+    var
+        EnvironmentInformation: Codeunit "Environment Information";
+    begin
+        exit(EnvironmentInformation.GetApplicationFamily());
+    end;
+
+    procedure IsSaas(): boolean
+    var
+        EnvironmentInformation: Codeunit "Environment Information";
+    begin
+        exit(EnvironmentInformation.IsSaas());
+    end;
+
+    procedure GetFormatRegionOrDefault(FormatRegion: Text[80]): Text
+    var
+        Language: Codeunit Language;
+    begin
+        exit(Language.GetFormatRegionOrDefault(FormatRegion));
+    end;
+
+    procedure GetAsJSon(var RestClient: Codeunit "Rest Client"; RequestUri: Text): JsonToken
+    begin
+        exit(RestClient.GetAsJSon(RequestUri));
+    end;
+
+    procedure GetUserSettings(UserSecurityId: Guid; var TempUserSettingsRecord: record "User Settings" temporary)
+    var
+        UserSettings: Codeunit "User Settings";
+    begin
+        UserSettings.GetUserSettings(Database.UserSecurityID(), TempUserSettingsRecord);
+    end;
+
+    procedure ShouldSetCommonHeaders(): Boolean
+    begin
+        exit(true);
+    end;
+}

src/System Application/Test Library/AppSource Gallery/app.json

@@ -0,0 +1,48 @@
+{
+    "id":  "ba17b564-d600-44d5-be0b-ca7ff7ac28fc",
+    "name":  "AppSource Gallery Test Library",
+    "publisher":  "Microsoft",
+    "brief":  "Test objects for the AppSource Gallery module.",
+    "description":  "Test objects for the AppSource Gallery module.",
+    "version":  "25.0.0.0",
+    "privacyStatement":  "https://go.microsoft.com/fwlink/?LinkId=724009",
+    "EULA":  "https://go.microsoft.com/fwlink/?linkid=2009120",
+    "help":  "",
+    "url":  "https://go.microsoft.com/fwlink/?LinkId=724011",
+    "logo":  "",
+    "dependencies":  [
+                         {
+                            "id": "79952567-63d7-4586-8b47-ba13a11a8a18",
+                            "name": "AppSource Product Gallery",
+                            "publisher": "Microsoft",
+                            "version": "25.0.0.0"
+                         },
+                         {
+                            "id": "812b339d-a9db-4a6e-84e4-fe35cbef0c44",
+                            "name": "Rest Client",
+                            "publisher": "Microsoft",
+                            "version": "25.0.0.0"
+                         },
+                         {
+                            "id":  "7b9b59f5-a68d-4271-b11a-0d3b9c0938dd",
+                            "name":  "User Settings",
+                            "publisher":  "Microsoft",
+                            "version": "25.0.0.0"
+                         }
+                     ],
+    "screenshots":  [
+
+                    ],
+    "idRanges":  [
+                     {
+                         "from":  132910,
+                         "to":  132910
+                     },
+                     {
+                        "from":  132913,
+                        "to":  132913
+                    }
+                 ],
+    "platform":  "24.0.0.0",
+    "target":  "OnPrem"
+}

src/System Application/Test Library/AppSource Gallery/src/AppSourceMockDepsProvider.Codeunit.al

@@ -0,0 +1,118 @@
+// ------------------------------------------------------------------------------------------------
+// Copyright (c) Microsoft Corporation. All rights reserved.
+// Licensed under the MIT License. See License.txt in the project root for license information.
+// ------------------------------------------------------------------------------------------------
+namespace System.TestLibraries.Apps.AppSource;
+
+using System.Environment.Configuration;
+using System.RestClient;
+using System.Apps.AppSource;
+
+/// <summary>
+/// Library for providing mock dependencies for the AppSource product manager.
+/// </summary>
+codeunit 132913 "AppSource Mock Deps. Provider" implements "AppSource Product Manager Dependencies"
+{
+    InherentEntitlements = X;
+    InherentPermissions = X;
+
+    var
+        FormatRegionStore: Text[80];
+        CountryLetterCode: Code[2];
+        PreferredLanguage: Text;
+        LanguageID: Variant;
+        IsInSaas: Boolean;
+        JsonRequests: JsonArray;
+        ApplicationFamily: Text;
+
+    // Dependency to Azure AD Tenant
+    procedure GetCountryLetterCode(): Code[2]
+    begin
+        exit(CountryLetterCode)
+    end;
+
+    procedure SetCountryLetterCode(InputCountryLetterCode: Code[2])
+    begin
+        CountryLetterCode := InputCountryLetterCode;
+    end;
+
+    procedure GetPreferredLanguage(): Text
+    begin
+        exit(PreferredLanguage);
+    end;
+
+    procedure SetPreferredLanguage(InputPreferredLanguage: Text)
+    begin
+        PreferredLanguage := InputPreferredLanguage;
+    end;
+
+    // Dependency to Environment Information 
+    procedure GetApplicationFamily(): Text
+    begin
+        exit(ApplicationFamily);
+    end;
+
+    procedure SetApplicationFamily(InputApplicationFamily: Text)
+    begin
+        ApplicationFamily := InputApplicationFamily;
+    end;
+
+    procedure IsSaas(): Boolean
+    begin
+        exit(IsInSaas);
+    end;
+
+    procedure SetIsSaas(InputIsSaas: Boolean)
+    begin
+        IsInSaas := InputIsSaas;
+    end;
+
+    // Dependency to Language 
+    procedure GetFormatRegionOrDefault(InputFormatRegion: Text[80]): Text
+    begin
+        if (InputFormatRegion <> '') then
+            exit(InputFormatRegion);
+        exit(FormatRegionStore);
+    end;
+
+    procedure SetFormatRegionStore(InputFormatRegion: Text[80])
+    begin
+        FormatRegionStore := InputFormatRegion;
+    end;
+
+    // Rest client override
+    procedure GetAsJSon(var RestClient: Codeunit "Rest Client"; RequestUri: Text): JsonToken
+    var
+        Json: JsonToken;
+    begin
+        JsonRequests.Get(0, Json);
+        JsonRequests.RemoveAt(0);
+        exit(Json);
+    end;
+
+    procedure SetJSon(JsonText: Text)
+    var
+        Json: JsonToken;
+    begin
+        Json.ReadFrom(JsonText);
+        JsonRequests.Add(Json);
+    end;
+
+    procedure ShouldSetCommonHeaders(): Boolean
+    begin
+        exit(false);
+    end;
+
+    // Dependency to User Settings
+    procedure GetUserSettings(UserSecurityID: Guid; var TempUserSettingsRecord: Record "User Settings" temporary)
+    begin
+        TempUserSettingsRecord.Init();
+        TempUserSettingsRecord."User Security ID" := UserSecurityID;
+        TempUserSettingsRecord."Language ID" := LanguageID;
+    end;
+
+    procedure SetUserSettings(InputLanguageId: Variant)
+    begin
+        LanguageID := InputLanguageId;
+    end;
+}