Adding the client code for supporting experiments

doc/admx/DesktopAppInstaller.admx

@@ -43,6 +43,16 @@
         <decimal value="0" />
       </disabledValue>
     </policy>
+    <policy name="EnableExperiments" class="Machine" displayName="$(string.EnableExperiments)" explainText="$(string.EnableExperimentsExplanation)" key="Software\Policies\Microsoft\Windows\AppInstaller" valueName="EnableExperiments">
+      <parentCategory ref="AppInstaller" />
+      <supportedOn ref="windows:SUPPORTED_Windows_10_0_RS5" />
+      <enabledValue>
+        <decimal value="1" />
+      </enabledValue>
+      <disabledValue>
+        <decimal value="0" />
+      </disabledValue>
+    </policy>
     <policy name="EnableLocalManifestFiles" class="Machine" displayName="$(string.EnableLocalManifestFiles)" explainText="$(string.EnableLocalManifestFilesExplanation)" key="Software\Policies\Microsoft\Windows\AppInstaller" valueName="EnableLocalManifestFiles">
       <parentCategory ref="AppInstaller" />
       <supportedOn ref="windows:SUPPORTED_Windows_10_0_RS5" />

doc/admx/en-US/DesktopAppInstaller.adml

@@ -25,6 +25,12 @@ If you disable this setting, users will not be able to change settings for the W
 If you enable or do not configure this setting, users will be able to enable experimental features for the Windows Package Manager.
 
 If you disable this setting, users will not be able to enable experimental features for the Windows Package Manager.</string>
+      <string id="EnableExperiments">Enable Windows Package Manager Experiments</string>
+      <string id="EnableExperimentsExplanation">This policy controls whether users can enable experiments in the Windows Package Manager.
+
+If you enable or do not configure this setting, users will be able to enable experiments for the Windows Package Manager.
+
+If you disable this setting, users will not be able to enable experiments for the Windows Package Manager.</string>
       <string id="EnableLocalManifestFiles">Enable Windows Package Manager Local Manifest Files</string>
       <string id="EnableLocalManifestFilesExplanation">This policy controls whether users can install packages with local manifest files.
 

src/AppInstallerCLIPackage/Shared/Strings/en-us/winget.resw

@@ -897,6 +897,9 @@ They can be configured through the settings file 'winget settings'.</value>
   <data name="PolicyEnableExperimentalFeatures" xml:space="preserve">
     <value>Enable Windows App Installer Experimental Features</value>
   </data>
+  <data name="PolicyEnableExperiments" xml:space="preserve">
+    <value>Enable Windows App Installer Experiments</value>
+  </data>
   <data name="PolicyEnableMSStoreSource" xml:space="preserve">
     <value>Enable Windows App Installer Microsoft Store Source</value>
   </data>

src/AppInstallerCLITests/AppInstallerCLITests.vcxproj

@@ -279,6 +279,7 @@
     <ClCompile Include="Downloader.cpp" />
     <ClCompile Include="DownloadFlow.cpp" />
     <ClCompile Include="Errors.cpp" />
+    <ClCompile Include="Experiment.cpp" />
     <ClCompile Include="ExperimentalFeature.cpp" />
     <ClCompile Include="ExportFlow.cpp" />
     <ClCompile Include="FileCache.cpp" />

src/AppInstallerCLITests/AppInstallerCLITests.vcxproj.filters

@@ -371,6 +371,9 @@
     <ClCompile Include="Fonts.cpp">
       <Filter>Source Files\Common</Filter>
     </ClCompile>
+    <ClCompile Include="Experiment.cpp">
+      <Filter>Source Files\Repository</Filter>
+    </ClCompile>
   </ItemGroup>
   <ItemGroup>
     <None Include="PropertySheet.props" />

src/AppInstallerCLITests/Experiment.cpp

@@ -0,0 +1,76 @@
+
+// Copyright (c) Microsoft Corporation.
+// Licensed under the MIT License.
+#include "pch.h"
+#include "TestCommon.h"
+#include "TestSettings.h"
+#include <winget/Experiment.h>
+#include <AppInstallerStrings.h>
+
+using namespace TestCommon;
+using namespace AppInstaller::Settings;
+
+#define SET_POLICY_STATE(_policy_, _state_) \
+    GroupPolicyTestOverride policies; \
+    policies.SetState(_policy_, _state_);
+
+#define SET_USER_SETTINGS(_value_) \
+    TestUserSettings settings; \
+    settings.Set<Setting::Experiments>({ \
+        {"TestExperiment", _value_} \
+    });
+
+#define ASSERT_EXPERIMENT(_isEnabled_, _toggleSource_) \
+    auto testExperimentState = Experiment::GetState(Experiment::Key::TestExperiment); \
+    REQUIRE(_isEnabled_ == testExperimentState.IsEnabled()); \
+    REQUIRE(_toggleSource_ == testExperimentState.ToggleSource());
+
+TEST_CASE("Experiment_GroupPolicyControl", "[experiment]")
+{
+    SECTION("Not configured")
+    {
+        SET_POLICY_STATE(TogglePolicy::Policy::Experiments, PolicyState::NotConfigured);
+        ASSERT_EXPERIMENT(true, ExperimentToggleSource::Default);
+    }
+
+    SECTION("Enabled")
+    {
+        SET_POLICY_STATE(TogglePolicy::Policy::Experiments, PolicyState::Enabled);
+        ASSERT_EXPERIMENT(true, ExperimentToggleSource::Default);
+    }
+
+    SECTION("Disabled")
+    {
+        SET_POLICY_STATE(TogglePolicy::Policy::Experiments, PolicyState::Disabled);
+        ASSERT_EXPERIMENT(false, ExperimentToggleSource::Policy);
+    }
+}
+
+TEST_CASE("Experiment_GroupPolicyDisabled_ReturnFalse", "[experiment]")
+{
+    // If the policy is disabled, then also the user settings should be ignored.
+    SET_POLICY_STATE(TogglePolicy::Policy::Experiments, PolicyState::Disabled);
+    SET_USER_SETTINGS(true);
+    ASSERT_EXPERIMENT(false, ExperimentToggleSource::Policy);
+}
+
+TEST_CASE("Experiment_UserSettingsControl", "[experiment]")
+{
+    SECTION("Experiments not configured in user settings")
+    {
+        // Default value are used
+        ASSERT_EXPERIMENT(true, ExperimentToggleSource::Default);
+    }
+
+    SECTION("Experiments enabled in user settings")
+    {
+        SET_USER_SETTINGS(true);
+        ASSERT_EXPERIMENT(true, ExperimentToggleSource::UserSetting);
+    }
+
+    SECTION("Experiments disabled in user settings")
+    {
+        SET_USER_SETTINGS(false);
+        ASSERT_EXPERIMENT(false, ExperimentToggleSource::UserSetting);
+    }
+}

src/AppInstallerCLITests/GroupPolicy.cpp

@@ -402,6 +402,7 @@ TEST_CASE("GroupPolicy_AllEnabled", "[groupPolicy]")
     SetRegistryValue(policiesKey.get(), EnableWindowsPackageManagerCommandLineInterfaces, 1);
     SetRegistryValue(policiesKey.get(), ConfigurationPolicyValueName, 1);
     SetRegistryValue(policiesKey.get(), ProxyCommandLineOptionsPolicyValueName, 1);
+    SetRegistryValue(policiesKey.get(), EnableExperimentsPolicyValueName , 1);
 
     GroupPolicy groupPolicy{ policiesKey.get() };
     for (const auto& policy : TogglePolicy::GetAllPolicies())

src/AppInstallerCLITests/TestSettings.h

@@ -23,6 +23,7 @@ namespace TestCommon
     const std::wstring EnableWindowsPackageManagerCommandLineInterfaces = L"EnableWindowsPackageManagerCommandLineInterfaces";
     const std::wstring ConfigurationPolicyValueName = L"EnableWindowsPackageManagerConfiguration";
     const std::wstring ProxyCommandLineOptionsPolicyValueName = L"EnableWindowsPackageManagerProxyCommandLineOptions";
+    const std::wstring EnableExperimentsPolicyValueName = L"EnableExperiments";
 
     const std::wstring SourceUpdateIntervalPolicyValueName = L"SourceAutoUpdateInterval";
     const std::wstring SourceUpdateIntervalPolicyOldValueName = L"SourceAutoUpdateIntervalInMinutes";
@@ -90,4 +91,4 @@ namespace TestCommon
     };
 
 #define REQUIRE_POLICY_EXCEPTION(_expr_, _policy_)     REQUIRE_THROWS_MATCHES(_expr_, AppInstaller::Settings::GroupPolicyException, TestCommon::GroupPolicyExceptionMatcher(_policy_))
-}
+}