Skip to content

Refactor integration tests for BI Report Server #2432

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
johlju merged 6 commits into from
Jan 22, 2026
Merged

Refactor integration tests for BI Report Server #2432

Show file tree
Hide file tree
Changes from 3 commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
21e6ad7
Refactor integration tests for SQL Server Reporting Services and Powe…
johlju Jan 22, 2026
e59f6ce
Update tests/Integration/Commands/Remove-SqlDscRSUrlReservation.Integ…
johlju Jan 22, 2026
9c8e5d7
Remove redundant secure connection enabling in integration tests for …
johlju Jan 22, 2026
ae2c150
Refactor integration tests for SQL Server Reporting Services and Powe…
johlju Jan 22, 2026
528b8b8
Remove unnecessary BeforeA block in Power BI Report Server integratio…
johlju Jan 22, 2026
7881f57
Refactor integration tests for Set-SqlDscRSUrlReservation to suppress…
johlju Jan 22, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 6 additions & 0 deletions CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,12 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0

## [Unreleased]

### Changed

- SqlServerDsc
- Refactor integration tests for SQL Server Reporting Services and Power BI Report
Server ([issue #2431](https://github.com/dsccommunity/SqlServerDsc/issues/2431)).

## [17.4.0] - 2026-01-19

### Added
Expand Down
3 changes: 0 additions & 3 deletions tests/Integration/Commands/Add-SqlDscRSUrlReservation.Integration.Tests.ps1
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -36,9 +36,6 @@ Describe 'Add-SqlDscRSUrlReservation' {
BeforeAll {
$script:configuration = Get-SqlDscRSConfiguration -InstanceName 'SSRS' -ErrorAction 'Stop'

# Store original URL reservations to restore later
$script:originalReservations = $script:configuration | Get-SqlDscRSUrlReservation -ErrorAction 'Stop'

# Use a unique port for testing to avoid conflicts
$script:testPort = 18080
$script:testUrl = "http://+:$script:testPort"
Expand Down
137 changes: 1 addition & 136 deletions tests/Integration/Commands/Disable-SqlDscRsSecureConnection.Integration.Tests.ps1
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -33,28 +33,7 @@ BeforeAll {

Describe 'Disable-SqlDscRsSecureConnection' {
Context 'When disabling secure connection for SQL Server Reporting Services' -Tag @('Integration_SQL2017_RS') {
BeforeAll {
$script:configuration = Get-SqlDscRSConfiguration -InstanceName 'SSRS'
$script:originalLevel = $script:configuration.SecureConnectionLevel
}

AfterAll {
# Restore the original secure connection level
$restoreConfig = Get-SqlDscRSConfiguration -InstanceName 'SSRS'
if ($script:originalLevel -eq 0)
{
$restoreConfig | Disable-SqlDscRsSecureConnection -Force
}
else
{
$restoreConfig | Enable-SqlDscRsSecureConnection -Force
}
}

It 'Should disable secure connection using pipeline' {
# First ensure secure connection is enabled
$script:configuration | Enable-SqlDscRsSecureConnection -Force

# Disable secure connection
$config = Get-SqlDscRSConfiguration -InstanceName 'SSRS'
$config | Disable-SqlDscRsSecureConnection -Force
Expand All @@ -63,44 +42,10 @@ Describe 'Disable-SqlDscRsSecureConnection' {
$verifyConfig = Get-SqlDscRSConfiguration -InstanceName 'SSRS'
$verifyConfig.SecureConnectionLevel | Should -Be 0
}

It 'Should return configuration when using PassThru' {
# First enable TLS
$config = Get-SqlDscRSConfiguration -InstanceName 'SSRS'
$config | Enable-SqlDscRsSecureConnection -Force

# Disable with PassThru
$config = Get-SqlDscRSConfiguration -InstanceName 'SSRS'
$result = $config | Disable-SqlDscRsSecureConnection -Force -PassThru

$result | Should -Not -BeNullOrEmpty
$result.InstanceName | Should -Be 'SSRS'
}
}

Context 'When disabling secure connection for SQL Server Reporting Services' -Tag @('Integration_SQL2019_RS') {
BeforeAll {
$script:configuration = Get-SqlDscRSConfiguration -InstanceName 'SSRS'
$script:originalLevel = $script:configuration.SecureConnectionLevel
}

AfterAll {
# Restore the original secure connection level
$restoreConfig = Get-SqlDscRSConfiguration -InstanceName 'SSRS'
if ($script:originalLevel -eq 0)
{
$restoreConfig | Disable-SqlDscRsSecureConnection -Force
}
else
{
$restoreConfig | Enable-SqlDscRsSecureConnection -Force
}
}

It 'Should disable secure connection using pipeline' {
# First ensure secure connection is enabled
$script:configuration | Enable-SqlDscRsSecureConnection -Force

# Disable secure connection
$config = Get-SqlDscRSConfiguration -InstanceName 'SSRS'
$config | Disable-SqlDscRsSecureConnection -Force
Expand All @@ -109,44 +54,10 @@ Describe 'Disable-SqlDscRsSecureConnection' {
$verifyConfig = Get-SqlDscRSConfiguration -InstanceName 'SSRS'
$verifyConfig.SecureConnectionLevel | Should -Be 0
}

It 'Should return configuration when using PassThru' {
# First enable TLS
$config = Get-SqlDscRSConfiguration -InstanceName 'SSRS'
$config | Enable-SqlDscRsSecureConnection -Force

# Disable with PassThru
$config = Get-SqlDscRSConfiguration -InstanceName 'SSRS'
$result = $config | Disable-SqlDscRsSecureConnection -Force -PassThru

$result | Should -Not -BeNullOrEmpty
$result.InstanceName | Should -Be 'SSRS'
}
}

Context 'When disabling secure connection for SQL Server Reporting Services' -Tag @('Integration_SQL2022_RS') {
BeforeAll {
$script:configuration = Get-SqlDscRSConfiguration -InstanceName 'SSRS'
$script:originalLevel = $script:configuration.SecureConnectionLevel
}

AfterAll {
# Restore the original secure connection level
$restoreConfig = Get-SqlDscRSConfiguration -InstanceName 'SSRS'
if ($script:originalLevel -eq 0)
{
$restoreConfig | Disable-SqlDscRsSecureConnection -Force
}
else
{
$restoreConfig | Enable-SqlDscRsSecureConnection -Force
}
}

It 'Should disable secure connection using pipeline' {
# First ensure secure connection is enabled
$script:configuration | Enable-SqlDscRsSecureConnection -Force

# Disable secure connection
$config = Get-SqlDscRSConfiguration -InstanceName 'SSRS'
$config | Disable-SqlDscRsSecureConnection -Force
Expand All @@ -155,44 +66,11 @@ Describe 'Disable-SqlDscRsSecureConnection' {
$verifyConfig = Get-SqlDscRSConfiguration -InstanceName 'SSRS'
$verifyConfig.SecureConnectionLevel | Should -Be 0
}

It 'Should return configuration when using PassThru' {
# First enable TLS
$config = Get-SqlDscRSConfiguration -InstanceName 'SSRS'
$config | Enable-SqlDscRsSecureConnection -Force

# Disable with PassThru
$config = Get-SqlDscRSConfiguration -InstanceName 'SSRS'
$result = $config | Disable-SqlDscRsSecureConnection -Force -PassThru

$result | Should -Not -BeNullOrEmpty
$result.InstanceName | Should -Be 'SSRS'
}
}

Context 'When disabling secure connection for Power BI Report Server' -Tag @('Integration_PowerBI') {
BeforeAll {
$script:configuration = Get-SqlDscRSConfiguration -InstanceName 'PBIRS'
$script:originalLevel = $script:configuration.SecureConnectionLevel
}

AfterAll {
# Restore the original secure connection level
$restoreConfig = Get-SqlDscRSConfiguration -InstanceName 'PBIRS'
if ($script:originalLevel -eq 0)
{
$restoreConfig | Disable-SqlDscRsSecureConnection -Force
}
else
{
$restoreConfig | Enable-SqlDscRsSecureConnection -Force
}
}

BeforeA
Comment thread
johlju marked this conversation as resolved.
Outdated
It 'Should disable secure connection for PBIRS using pipeline' {
# First ensure secure connection is enabled
$script:configuration | Enable-SqlDscRsSecureConnection -Force

# Disable secure connection
$config = Get-SqlDscRSConfiguration -InstanceName 'PBIRS'
$config | Disable-SqlDscRsSecureConnection -Force
Expand All @@ -201,18 +79,5 @@ Describe 'Disable-SqlDscRsSecureConnection' {
$verifyConfig = Get-SqlDscRSConfiguration -InstanceName 'PBIRS'
$verifyConfig.SecureConnectionLevel | Should -Be 0
}

It 'Should return configuration when using PassThru' {
# First enable TLS
$config = Get-SqlDscRSConfiguration -InstanceName 'PBIRS'
$config | Enable-SqlDscRsSecureConnection -Force

# Disable with PassThru
$config = Get-SqlDscRSConfiguration -InstanceName 'PBIRS'
$result = $config | Disable-SqlDscRsSecureConnection -Force -PassThru

$result | Should -Not -BeNullOrEmpty
$result.InstanceName | Should -Be 'PBIRS'
}
}
}
116 changes: 0 additions & 116 deletions tests/Integration/Commands/Enable-SqlDscRsSecureConnection.Integration.Tests.ps1
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -33,28 +33,7 @@ BeforeAll {

Describe 'Enable-SqlDscRsSecureConnection' {
Context 'When enabling secure connection for SQL Server Reporting Services' -Tag @('Integration_SQL2017_RS') {
BeforeAll {
$script:configuration = Get-SqlDscRSConfiguration -InstanceName 'SSRS'
$script:originalLevel = $script:configuration.SecureConnectionLevel
}

AfterAll {
# Restore the original secure connection level
$restoreConfig = Get-SqlDscRSConfiguration -InstanceName 'SSRS'
if ($script:originalLevel -eq 0)
{
$restoreConfig | Disable-SqlDscRsSecureConnection -Force
}
else
{
$restoreConfig | Enable-SqlDscRsSecureConnection -Force
}
}

It 'Should enable secure connection using pipeline' {
# First ensure secure connection is disabled
$script:configuration | Disable-SqlDscRsSecureConnection -Force

# Enable secure connection
$config = Get-SqlDscRSConfiguration -InstanceName 'SSRS'
$config | Enable-SqlDscRsSecureConnection -Force
Expand All @@ -63,39 +42,10 @@ Describe 'Enable-SqlDscRsSecureConnection' {
$verifyConfig = Get-SqlDscRSConfiguration -InstanceName 'SSRS'
$verifyConfig.SecureConnectionLevel | Should -BeGreaterOrEqual 1
}

It 'Should return configuration when using PassThru' {
$config = Get-SqlDscRSConfiguration -InstanceName 'SSRS'
$result = $config | Enable-SqlDscRsSecureConnection -Force -PassThru

$result | Should -Not -BeNullOrEmpty
$result.InstanceName | Should -Be 'SSRS'
}
}

Context 'When enabling secure connection for SQL Server Reporting Services' -Tag @('Integration_SQL2019_RS') {
BeforeAll {
$script:configuration = Get-SqlDscRSConfiguration -InstanceName 'SSRS'
$script:originalLevel = $script:configuration.SecureConnectionLevel
}

AfterAll {
# Restore the original secure connection level
$restoreConfig = Get-SqlDscRSConfiguration -InstanceName 'SSRS'
if ($script:originalLevel -eq 0)
{
$restoreConfig | Disable-SqlDscRsSecureConnection -Force
}
else
{
$restoreConfig | Enable-SqlDscRsSecureConnection -Force
}
}

It 'Should enable secure connection using pipeline' {
# First ensure secure connection is disabled
$script:configuration | Disable-SqlDscRsSecureConnection -Force

# Enable secure connection
$config = Get-SqlDscRSConfiguration -InstanceName 'SSRS'
$config | Enable-SqlDscRsSecureConnection -Force
Expand All @@ -104,39 +54,10 @@ Describe 'Enable-SqlDscRsSecureConnection' {
$verifyConfig = Get-SqlDscRSConfiguration -InstanceName 'SSRS'
$verifyConfig.SecureConnectionLevel | Should -BeGreaterOrEqual 1
}

It 'Should return configuration when using PassThru' {
$config = Get-SqlDscRSConfiguration -InstanceName 'SSRS'
$result = $config | Enable-SqlDscRsSecureConnection -Force -PassThru

$result | Should -Not -BeNullOrEmpty
$result.InstanceName | Should -Be 'SSRS'
}
}

Context 'When enabling secure connection for SQL Server Reporting Services' -Tag @('Integration_SQL2022_RS') {
BeforeAll {
$script:configuration = Get-SqlDscRSConfiguration -InstanceName 'SSRS'
$script:originalLevel = $script:configuration.SecureConnectionLevel
}

AfterAll {
# Restore the original secure connection level
$restoreConfig = Get-SqlDscRSConfiguration -InstanceName 'SSRS'
if ($script:originalLevel -eq 0)
{
$restoreConfig | Disable-SqlDscRsSecureConnection -Force
}
else
{
$restoreConfig | Enable-SqlDscRsSecureConnection -Force
}
}

It 'Should enable secure connection using pipeline' {
# First ensure secure connection is disabled
$script:configuration | Disable-SqlDscRsSecureConnection -Force

# Enable secure connection
$config = Get-SqlDscRSConfiguration -InstanceName 'SSRS'
$config | Enable-SqlDscRsSecureConnection -Force
Expand All @@ -145,39 +66,10 @@ Describe 'Enable-SqlDscRsSecureConnection' {
$verifyConfig = Get-SqlDscRSConfiguration -InstanceName 'SSRS'
$verifyConfig.SecureConnectionLevel | Should -BeGreaterOrEqual 1
}

It 'Should return configuration when using PassThru' {
$config = Get-SqlDscRSConfiguration -InstanceName 'SSRS'
$result = $config | Enable-SqlDscRsSecureConnection -Force -PassThru

$result | Should -Not -BeNullOrEmpty
$result.InstanceName | Should -Be 'SSRS'
}
}

Context 'When enabling secure connection for Power BI Report Server' -Tag @('Integration_PowerBI') {
BeforeAll {
$script:configuration = Get-SqlDscRSConfiguration -InstanceName 'PBIRS'
$script:originalLevel = $script:configuration.SecureConnectionLevel
}

AfterAll {
# Restore the original secure connection level
$restoreConfig = Get-SqlDscRSConfiguration -InstanceName 'PBIRS'
if ($script:originalLevel -eq 0)
{
$restoreConfig | Disable-SqlDscRsSecureConnection -Force
}
else
{
$restoreConfig | Enable-SqlDscRsSecureConnection -Force
}
}

It 'Should enable secure connection for PBIRS using pipeline' {
# First ensure secure connection is disabled
$script:configuration | Disable-SqlDscRsSecureConnection -Force

# Enable secure connection
$config = Get-SqlDscRSConfiguration -InstanceName 'PBIRS'
$config | Enable-SqlDscRsSecureConnection -Force
Expand All @@ -186,13 +78,5 @@ Describe 'Enable-SqlDscRsSecureConnection' {
$verifyConfig = Get-SqlDscRSConfiguration -InstanceName 'PBIRS'
$verifyConfig.SecureConnectionLevel | Should -BeGreaterOrEqual 1
}

It 'Should return configuration when using PassThru' {
$config = Get-SqlDscRSConfiguration -InstanceName 'PBIRS'
$result = $config | Enable-SqlDscRsSecureConnection -Force -PassThru

$result | Should -Not -BeNullOrEmpty
$result.InstanceName | Should -Be 'PBIRS'
}
}
}
Loading
Loading