[Bug Fix] Remove mandatory layout (#654)

* fixing layout error

* Adding UT

* Keeping this PR short.
I've removed the following feature: Remove --package-store requirement in bootstrap subcommand.

---------

Co-authored-by: Nirjan Chapagain <nchapagain@example.com>

Author: nchapagain001

src/VirtualClient/VirtualClient.Main/BootstrapCommand.cs

@@ -136,9 +136,9 @@ protected void Validate()
                 //
                 // e.g.
                 // --key-vault="https://any.vault.azure.net?cid=8cdebecc...&tid=42005d4d...&crti=ANY&crts=any.corp.azure.com"
-                if (string.IsNullOrWhiteSpace(this.AccessToken) 
+                if (string.IsNullOrWhiteSpace(this.AccessToken)
                     && string.IsNullOrWhiteSpace(this.TokenFilePath)
-                    && string.IsNullOrWhiteSpace(this.TenantId) 
+                    && string.IsNullOrWhiteSpace(this.TenantId)
                     && (this.KeyVaultStore as DependencyKeyVaultStore)?.Credentials == null)
                 {
                     throw new ArgumentException(
@@ -153,4 +153,4 @@ protected void Validate()
             }
         }
     }
-}
+}

src/VirtualClient/VirtualClient.Main/CommandLineParser.cs

@@ -642,4 +642,4 @@ private static string[] PreprocessArguments(params string[] args)
             return preprocessedArgs;
         }
     }
-}
+}

src/VirtualClient/VirtualClient.Main/ExecuteProfileCommand.cs

@@ -669,7 +669,7 @@ protected void SetHostMetadataTelemetryProperties(IEnumerable<string> profiles,
                 new Dictionary<string, object>
                 {
                     { "exitWait", this.ExitWait },
-                    { "layout", this.Layout.ToString() },
+                    { "layout", this.Layout?.ToString() },
                     { "logToFile", this.LogToFile },
                     { "iterations", this.Iterations?.ProfileIterations },
                     { "profiles", string.Join(",", profiles.Select(p => Path.GetFileName(p))) },

src/VirtualClient/VirtualClient.UnitTests/ExecuteProfileCommandTests.cs

@@ -538,5 +538,18 @@ private static Tuple<string, string> GetAccessTokenPair()
 
             return new Tuple<string, string>(decodedOriginalBytes, decodedObscuredBytes);
         }
+
+        [Test]
+        public void SetHostMetadataTelemetryPropertiesDoesNotThrowWhenLayoutIsNull()
+        {
+            // Layout is not mandatory for setting host metadata properties. 
+            this.command.Layout = null;
+
+            string profile = "TEST-WORKLOAD-PROFILE.json";
+            List<string> profiles = new List<string> { this.mockFixture.GetProfilesPath(profile) };
+
+            // Act & Assert - Should not throw expection
+            Assert.DoesNotThrow(() => this.command.SetHostMetadataTelemetryProperties(profiles, this.mockFixture.Dependencies));
+        }
     }
 }