From c3aa057bfc34f72debdb7f89f8e7e237cedbefc2 Mon Sep 17 00:00:00 2001 From: "Rodweil, Theodor" Date: Mon, 31 Jul 2023 00:18:00 +0200 Subject: [PATCH] fix(manifest): update test Well, this reminds us to refactor the testing suite... This currently is pretty dangerous, since the scopes are mixed up and there is no proper isolation between test blocks. --- PSConfluencePublisher/Manifest.Tests.ps1 | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/PSConfluencePublisher/Manifest.Tests.ps1 b/PSConfluencePublisher/Manifest.Tests.ps1 index ab54fc9..afbdecb 100755 --- a/PSConfluencePublisher/Manifest.Tests.ps1 +++ b/PSConfluencePublisher/Manifest.Tests.ps1 @@ -59,9 +59,9 @@ Describe 'Set-Manifest' ` Mock Set-Content { Should -Invoke -CommandName 'Set-Content' -Exactly -Times 1 - $args[1] | Should -Be 'foobar.x' + $Path | Should -Be 'foobar.x' - $args[3] | Should -Be ($mockManifest | ConvertTo-JSON) + $Value | Should -Be ($mockManifest | ConvertTo-JSON) } #mocking Get-Content, therefore file name can be bogus