Skip to content

Commit 3ba73f1

Browse files
committed
Addressed Review -- Renamed methods, reverted version
1 parent bad9bc4 commit 3ba73f1

File tree

5 files changed

+14
-16
lines changed

5 files changed

+14
-16
lines changed

src/ResourceManager/IotCentral/Commands.IotCentral/Az.IotCentral.psd1

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@
1212
# RootModule = ''
1313

1414
# Version number of this module.
15-
ModuleVersion = '0.1.1'
15+
ModuleVersion = '0.1.0'
1616

1717
# Supported PSEditions
1818
CompatiblePSEditions = 'Core', 'Desktop'

src/ResourceManager/IotCentral/Commands.IotCentral/AzureRM.IotCentral.psd1

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@
1212
# RootModule = ''
1313

1414
# Version number of this module.
15-
ModuleVersion = '1.0.1'
15+
ModuleVersion = '1.0.0'
1616

1717
# Supported PSEditions
1818
# CompatiblePSEditions = @()

src/ResourceManager/IotCentral/Commands.IotCentral/ChangeLog.md

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,3 @@
2121
* Preview Release
2222
* Added Tabular Output Format
2323
* Added Subdomain Checking in New-AzureRmIotCentralApp
24-
25-
## 2018.11.06 - Version 1.0.0
26-
* Preview Release
27-
* Initial Module Prerelease

src/ResourceManager/IotCentral/Commands.IotCentral/Common/IotCentralBaseCmdlet.cs

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,8 @@ public abstract class IotCentralBaseCmdlet : AzureRMCmdlet
3232
protected const string ResourceIdParameterSet = "ResourceIdParameterSet";
3333
protected const string InputObjectParameterSet = "InputObjectParameterSet";
3434

35+
protected const string resourceType = "IoTApps";
36+
3537
[Parameter(
3638
Mandatory = true,
3739
Position = 0,

src/ResourceManager/IotCentral/Commands.IotCentral/IotCentral/NewAzureRmIotCentralApp.cs

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -83,8 +83,8 @@ public override void ExecuteCmdlet()
8383
{
8484
if (ShouldProcess(Name, ResourceProperties.Resources.NewIotCentralApp))
8585
{
86-
this.CheckNameAvailability();
87-
this.CheckSubdomainAvailability();
86+
this.EnsureNameAvailabilityOrThrow();
87+
this.EnsureSubdomainAvailabilityOrThrow();
8888

8989
var iotCentralApp = new App()
9090
{
@@ -103,23 +103,23 @@ public override void ExecuteCmdlet()
103103

104104
}
105105

106-
private void CheckNameAvailability()
106+
private void EnsureNameAvailabilityOrThrow()
107107
{
108-
var checkNameInputs = new OperationInputs(this.Name, "IoTApps");
108+
var checkNameInputs = new OperationInputs(this.Name, resourceType);
109109
var nameAvailabilityInfo = this.IotCentralClient.Apps.CheckNameAvailability(checkNameInputs);
110-
this.CheckAvailability(nameAvailabilityInfo);
110+
this.EnsureAvailabilityOrThrow(nameAvailabilityInfo);
111111
}
112112

113-
private void CheckSubdomainAvailability()
113+
private void EnsureSubdomainAvailabilityOrThrow()
114114
{
115-
var checkSubdomainInputs = new OperationInputs(this.Subdomain, "IoTApps");
115+
var checkSubdomainInputs = new OperationInputs(this.Subdomain, resourceType);
116116
var subdomainAvailabilityInfo = this.IotCentralClient.Apps.CheckSubdomainAvailability(checkSubdomainInputs);
117-
this.CheckAvailability(subdomainAvailabilityInfo);
117+
this.EnsureAvailabilityOrThrow(subdomainAvailabilityInfo);
118118
}
119119

120-
private void CheckAvailability(AppAvailabilityInfo availabilityInfo)
120+
private void EnsureAvailabilityOrThrow(AppAvailabilityInfo availabilityInfo)
121121
{
122-
if (!(availabilityInfo.NameAvailable == true))
122+
if (availabilityInfo.NameAvailable != true)
123123
{
124124
throw new PSArgumentException(availabilityInfo.Message);
125125
}

0 commit comments

Comments
 (0)