fix(setup): Validate minimum number of characters for DigitalTwins na…

…me (Azure#12842)
AzureSDKAutomation · Jun 17, 2020 · 3c6a978 · 3c6a978
1 parent 9caad5f
commit 3c6a978
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git a/...digitaltwins/Azure.DigitalTwins.Core/tests/prerequisites/prerequisite readme.md b/...digitaltwins/Azure.DigitalTwins.Core/tests/prerequisites/prerequisite readme.md
@@ -2,6 +2,10 @@
 
 ## Install
 
+### Install the latest Powershell 7
+
+- Make sure you run the script using the latest stable version of [powershell 7](https://github.com/PowerShell/PowerShell/releases)
+
 ### Install the latest Azure CLI package
 
 - If already installed, check latest version:

diff --git a/sdk/digitaltwins/Azure.DigitalTwins.Core/tests/prerequisites/setup.ps1 b/sdk/digitaltwins/Azure.DigitalTwins.Core/tests/prerequisites/setup.ps1
@@ -8,14 +8,15 @@ param(
     [Parameter(Mandatory)]
     [string] $SubscriptionId,
 
-    [Parameter()]
+    [Parameter(Mandatory)]
+    [ValidateLength(6, 64)]
     [string] $DigitalTwinName,
 
     [Parameter()]
     [string] $AppRegistrationName
 )
 
-Function Connect-AzureSubscription()
+Function Connect-AzureSubscription
 {
     # Ensure the user is logged in
     try
@@ -52,11 +53,6 @@ Connect-AzureSubscription
 
 $Region = $Region.Replace(' ', '')
 
-if (-not $DigitalTwinName)
-{
-    $DigitalTwinName = $ResourceGroup
-}
-
 if (-not $AppRegistrationName)
 {
     $AppRegistrationName = $ResourceGroup