Skip to content

Commit 92c6b95

Browse files
committed
Fix review comment
1 parent 51cab16 commit 92c6b95

File tree

2 files changed

+29
-35
lines changed

2 files changed

+29
-35
lines changed

source/Public/ConvertTo-SqlDscEditionName.ps1

Lines changed: 28 additions & 35 deletions
Original file line numberDiff line numberDiff line change
@@ -35,49 +35,42 @@ function ConvertTo-SqlDscEditionName
3535
$Id
3636
)
3737

38-
# Maps the EditionId integer value to each respective edition and edition name.
39-
$EditionIdMap = @{
40-
# Reporting Services v16
41-
ID_2176971986 = @{
42-
Edition = 'Developer'
43-
EditionName = 'SQL Server Developer'
44-
}
45-
46-
# Power BI Report Server v15
47-
ID_2017617798 = @{
48-
Edition = 'Developer'
49-
EditionName = 'Power BI Report Server - Developer'
50-
}
51-
ID_1369084056 = @{
52-
Edition = 'Evaluation'
53-
EditionName = 'Power BI Report Server - Evaluation'
54-
}
55-
}
56-
5738
Write-Debug -Message ($script:localizedData.ConvertTo_EditionName_ConvertingEditionId -f $Id)
5839

59-
$mappingID = 'ID_' + $Id
40+
$resultObject = [PSCustomObject] @{
41+
EditionId = $Id
42+
Edition = ''
43+
EditionName = ''
44+
}
6045

61-
if ($EditionIdMap.ContainsKey($mappingID))
46+
switch ($Id)
6247
{
63-
$editionInfo = $EditionIdMap[$mappingID]
48+
2176971986
49+
{
50+
$resultObject.Edition = 'Developer'
51+
$resultObject.EditionName = 'SQL Server Developer'
52+
}
6453

65-
$resultObject = [PSCustomObject]@{
66-
EditionId = $Id
67-
Edition = $editionInfo.Edition
68-
EditionName = $editionInfo.EditionName
54+
2017617798
55+
{
56+
$resultObject.Edition = 'Developer'
57+
$resultObject.EditionName = 'Power BI Report Server - Developer'
6958
}
7059

71-
return $resultObject
72-
}
73-
else
74-
{
75-
Write-Debug -Message ($script:localizedData.ConvertTo_EditionName_UnknownEditionId -f $Id)
60+
1369084056
61+
{
62+
$resultObject.Edition = 'Evaluation'
63+
$resultObject.EditionName = 'Power BI Report Server - Evaluation'
64+
}
65+
66+
default
67+
{
68+
Write-Debug -Message ($script:localizedData.ConvertTo_EditionName_UnknownEditionId -f $Id)
7669

77-
return [PSCustomObject]@{
78-
EditionId = $Id
79-
Edition = 'Unknown'
80-
EditionName = 'Unknown'
70+
$resultObject.Edition = 'Unknown'
71+
$resultObject.EditionName = 'Unknown'
8172
}
8273
}
74+
75+
return $resultObject
8376
}

tests/Unit/Public/ConvertTo-SqlDscEditionName.Tests.ps1

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -50,6 +50,7 @@ Describe 'ConvertTo-SqlDscEditionName' {
5050
Context 'When converting a known EditionId' {
5151
BeforeAll {
5252
$testEditionId = 2176971986
53+
5354
$mockExpectedResult = @{
5455
EditionId = 2176971986
5556
Edition = 'Developer'

0 commit comments

Comments
 (0)