Fix cross-platform unit test path compatibility issues

austoonz · web-flow · commit 55c81159a557 · 2025-07-18T16:42:59.000-07:00
Merge pull request awslabs#47 from austoonz/chore/fix-xplat-unit-testing - Add Assert-PathEquals helper function to normalize path separators for cross-platform compatibility in unit tests - Replace direct string path comparisons with Assert-PathEquals in Get-Handler.Tests.ps1 and Set-PSModulePath.Tests.ps1 - Remove Windows-specific read-only directory test that was causing cross-platform issues (not relevant for the Lambda runtime which is Linux only) - Ensure all unit tests pass on Windows, Linux, and macOS by handling path separator differences transparently The new Assert-PathEquals function normalizes both actual and expected paths to use forward slashes before comparison, allowing tests to pass regardless of the platform's native path separator behavior.
diff --git a/powershell-runtime/tests/helpers/AssertionHelpers.Tests.ps1 b/powershell-runtime/tests/helpers/AssertionHelpers.Tests.ps1
@@ -446,4 +446,154 @@ Describe "Assert-FileExists" {
             { Assert-FileExists -Path $unreadableFile -ShouldBeFile } | Should -Not -Throw
         }
     }
-}
+}
+
+Describe "Assert-PathEquals" {
+    Context "When paths are identical" {
+        It "Should pass for identical forward slash paths" {
+            { Assert-PathEquals -Actual "/var/task/handler.ps1" -Expected "/var/task/handler.ps1" } | Should -Not -Throw
+        }
+
+        It "Should pass for identical backslash paths" {
+            { Assert-PathEquals -Actual "C:\var\task\handler.ps1" -Expected "C:\var\task\handler.ps1" } | Should -Not -Throw
+        }
+
+        It "Should pass for identical relative paths" {
+            { Assert-PathEquals -Actual "lib/utilities.ps1" -Expected "lib/utilities.ps1" } | Should -Not -Throw
+        }
+
+        It "Should pass for identical single file names" {
+            { Assert-PathEquals -Actual "handler.ps1" -Expected "handler.ps1" } | Should -Not -Throw
+        }
+    }
+
+    Context "When paths differ only by separators" {
+        It "Should pass when actual uses backslashes and expected uses forward slashes" {
+            { Assert-PathEquals -Actual "C:\var\task\handler.ps1" -Expected "C:/var/task/handler.ps1" } | Should -Not -Throw
+        }
+
+        It "Should pass when actual uses forward slashes and expected uses backslashes" {
+            { Assert-PathEquals -Actual "/var/task/handler.ps1" -Expected "\var\task\handler.ps1" } | Should -Not -Throw
+        }
+
+        It "Should pass for mixed separators in both paths" {
+            { Assert-PathEquals -Actual "C:\var/task\handler.ps1" -Expected "C:/var\task/handler.ps1" } | Should -Not -Throw
+        }
+
+        It "Should pass for nested directory paths with different separators" {
+            { Assert-PathEquals -Actual "lib\modules\aws\tools\handler.ps1" -Expected "lib/modules/aws/tools/handler.ps1" } | Should -Not -Throw
+        }
+    }
+
+    Context "When paths are genuinely different" {
+        It "Should throw for different file names" {
+            { Assert-PathEquals -Actual "/var/task/handler.ps1" -Expected "/var/task/different.ps1" } | Should -Throw -ExpectedMessage "*Expected path '/var/task/different.ps1' but got '/var/task/handler.ps1'*"
+        }
+
+        It "Should throw for different directory paths" {
+            { Assert-PathEquals -Actual "/var/task/handler.ps1" -Expected "/var/runtime/handler.ps1" } | Should -Throw -ExpectedMessage "*Expected path '/var/runtime/handler.ps1' but got '/var/task/handler.ps1'*"
+        }
+
+        It "Should throw for different number of path segments" {
+            { Assert-PathEquals -Actual "/var/task/lib/handler.ps1" -Expected "/var/task/handler.ps1" } | Should -Throw -ExpectedMessage "*Expected path '/var/task/handler.ps1' but got '/var/task/lib/handler.ps1'*"
+        }
+
+        It "Should throw for completely different paths" {
+            { Assert-PathEquals -Actual "C:\Windows\System32\file.txt" -Expected "/usr/local/bin/script.sh" } | Should -Throw -ExpectedMessage "*Expected path '/usr/local/bin/script.sh' but got 'C:\Windows\System32\file.txt'*"
+        }
+    }
+
+    Context "When using custom Because parameter" {
+        It "Should include custom reason in error message" {
+            { Assert-PathEquals -Actual "/var/task/wrong.ps1" -Expected "/var/task/handler.ps1" -Because "the script file path should match the handler configuration" } | Should -Throw -ExpectedMessage "*because the script file path should match the handler configuration*"
+        }
+
+        It "Should include custom reason with normalized paths" {
+            { Assert-PathEquals -Actual "C:\var\task\wrong.ps1" -Expected "/var/task/handler.ps1" -Because "cross-platform paths should be normalized" } | Should -Throw -ExpectedMessage "*because cross-platform paths should be normalized*"
+        }
+    }
+
+    Context "When handling edge cases" {
+        It "Should handle PowerShell parameter validation for empty strings" {
+            # PowerShell parameter validation prevents empty strings for mandatory parameters
+            { Assert-PathEquals -Actual "" -Expected "" } | Should -Throw -ExpectedMessage "*Cannot bind argument to parameter*"
+        }
+
+        It "Should handle PowerShell parameter validation for empty actual path" {
+            { Assert-PathEquals -Actual "" -Expected "/var/task/handler.ps1" } | Should -Throw -ExpectedMessage "*Cannot bind argument to parameter*"
+        }
+
+        It "Should handle paths with trailing separators" {
+            { Assert-PathEquals -Actual "/var/task/" -Expected "/var/task" } | Should -Throw -ExpectedMessage "*Expected path '/var/task' but got '/var/task/'*"
+        }
+
+        It "Should handle paths with multiple consecutive separators" {
+            { Assert-PathEquals -Actual "/var//task/handler.ps1" -Expected "/var/task/handler.ps1" } | Should -Throw -ExpectedMessage "*Expected path '/var/task/handler.ps1' but got '/var//task/handler.ps1'*"
+        }
+
+        It "Should handle paths with dots" {
+            { Assert-PathEquals -Actual "/var/task/./handler.ps1" -Expected "/var/task/handler.ps1" } | Should -Throw -ExpectedMessage "*Expected path '/var/task/handler.ps1' but got '/var/task/./handler.ps1'*"
+        }
+
+        It "Should handle paths with parent directory references" {
+            { Assert-PathEquals -Actual "/var/task/../task/handler.ps1" -Expected "/var/task/handler.ps1" } | Should -Throw -ExpectedMessage "*Expected path '/var/task/handler.ps1' but got '/var/task/../task/handler.ps1'*"
+        }
+    }
+
+    Context "When testing real-world Lambda scenarios" {
+        It "Should pass for typical Lambda task paths with different separators" {
+            { Assert-PathEquals -Actual "/var\task\handler.ps1" -Expected "/var/task/handler.ps1" } | Should -Not -Throw
+        }
+
+        It "Should pass for module paths in subdirectories" {
+            { Assert-PathEquals -Actual "/var\task\lib\utilities.ps1" -Expected "/var/task/lib/utilities.ps1" } | Should -Not -Throw
+        }
+
+        It "Should pass for PowerShell module manifest paths" {
+            { Assert-PathEquals -Actual "/opt\powershell\modules\MyModule\MyModule.psd1" -Expected "/opt/powershell/modules/MyModule/MyModule.psd1" } | Should -Not -Throw
+        }
+
+        It "Should pass for Windows-style Lambda paths normalized to Linux" {
+            { Assert-PathEquals -Actual "C:\lambda\task\function.ps1" -Expected "C:/lambda/task/function.ps1" } | Should -Not -Throw
+        }
+
+        It "Should handle layer paths correctly" {
+            { Assert-PathEquals -Actual "/opt\powershell\modules\AWS.Tools.Common\AWS.Tools.Common.psd1" -Expected "/opt/powershell/modules/AWS.Tools.Common/AWS.Tools.Common.psd1" } | Should -Not -Throw
+        }
+    }
+
+    Context "When error messages show normalization details" {
+        It "Should show normalized paths in error message when normalization occurred" {
+            { Assert-PathEquals -Actual "C:\var\task\wrong.ps1" -Expected "/var/task/handler.ps1" } | Should -Throw -ExpectedMessage "*normalized: '/var/task/handler.ps1' vs 'C:/var/task/wrong.ps1'*"
+        }
+
+        It "Should not show normalization details when no normalization was needed" {
+            { Assert-PathEquals -Actual "/var/task/wrong.ps1" -Expected "/var/task/handler.ps1" } | Should -Throw -ExpectedMessage "*Expected path '/var/task/handler.ps1' but got '/var/task/wrong.ps1'*"
+        }
+
+        It "Should show normalization details only when actual path was normalized" {
+            { Assert-PathEquals -Actual "C:\var\task\handler.ps1" -Expected "C:/var/task/different.ps1" } | Should -Throw -ExpectedMessage "*normalized: 'C:/var/task/different.ps1' vs 'C:/var/task/handler.ps1'*"
+        }
+    }
+
+    Context "When testing verbose output" {
+        It "Should write verbose message on successful assertion" {
+            Mock Write-Verbose { }
+
+            Assert-PathEquals -Actual "/var/task/handler.ps1" -Expected "/var/task/handler.ps1" -Verbose
+
+            Should -Invoke Write-Verbose -Exactly 1 -ParameterFilter { $Message -like "*Path assertion passed*" }
+        }
+
+        It "Should write verbose message with both paths" {
+            Mock Write-Verbose { }
+
+            # This should pass since the paths are equivalent after normalization
+            Assert-PathEquals -Actual "C:\var\task\handler.ps1" -Expected "C:/var/task/handler.ps1" -Verbose
+
+            Should -Invoke Write-Verbose -Exactly 1 -ParameterFilter {
+                $Message -like "*'C:/var/task/handler.ps1' matches 'C:\var\task\handler.ps1'*"
+            }
+        }
+    }
+}
diff --git a/powershell-runtime/tests/helpers/AssertionHelpers.ps1 b/powershell-runtime/tests/helpers/AssertionHelpers.ps1
@@ -460,5 +460,61 @@ function Assert-FileExists {
     Write-Verbose "File existence assertion passed: '$Path'"
 }
 
+<#
+.SYNOPSIS
+    Verify path equality with cross-platform path separator handling.
+
+.DESCRIPTION
+    Asserts that two paths are equal by normalizing path separators to forward slashes,
+    allowing tests to pass on both Windows and Linux regardless of the path separator
+    used by System.IO.Path.Combine() in the runtime.
+
+.PARAMETER Actual
+    The actual path value from the test result.
+
+.PARAMETER Expected
+    The expected path value (should use forward slashes for consistency).
+
+.PARAMETER Because
+    Optional reason for the assertion failure.
+
+.EXAMPLE
+    Assert-PathEquals -Actual $result.scriptFilePath -Expected "/var/task/handler.ps1"
+    Verifies that the script file path matches, regardless of platform path separators.
+
+.EXAMPLE
+    Assert-PathEquals -Actual $result.scriptFilePath -Expected "/var/task/lib/utilities.ps1" -Because "subdirectory paths should be handled correctly"
+    Verifies path equality with a custom failure message.
+#>
+function Assert-PathEquals {
+    [CmdletBinding()]
+    param(
+        [Parameter(Mandatory)]
+        [string]$Actual,
+
+        [Parameter(Mandatory)]
+        [string]$Expected,
+
+        [string]$Because
+    )
+
+    # Normalize both paths to use forward slashes for comparison
+    $normalizedActual = $Actual -replace '\\', '/'
+    $normalizedExpected = $Expected -replace '\\', '/'
+
+    if ($normalizedActual -ne $normalizedExpected) {
+        $message = "Expected path '$Expected' but got '$Actual'"
+        if ($normalizedActual -ne $Actual) {
+            $message += " (normalized: '$normalizedExpected' vs '$normalizedActual')"
+        }
+        if ($Because) {
+            $message += " because $Because"
+        }
+        throw $message
+    }
+
+    Write-Verbose "Path assertion passed: '$Expected' matches '$Actual'"
+}
+
 # Functions are available when dot-sourced
-# To use: . ./AssertionHelpers.ps1
+# To use: . ./AssertionHelpers.ps1
diff --git a/powershell-runtime/tests/unit/Build/build-PwshRuntimeLayer.Tests.ps1 b/powershell-runtime/tests/unit/Build/build-PwshRuntimeLayer.Tests.ps1
@@ -310,33 +310,6 @@ Describe "build-PwshRuntimeLayer.ps1" {
                 $_.Exception.Message | Should -Not -BeNullOrEmpty
             }
         }
-
-        It "Should handle read-only layer path appropriately" {
-            # Create a directory and make it read-only (if supported on platform)
-            $readOnlyPath = Join-Path $TestDrive "readonly-layer"
-            New-Item -Path $readOnlyPath -ItemType Directory -Force
-
-            try {
-                # Try to make directory read-only (Windows only test)
-                if ($IsWindows) {
-                    Set-ItemProperty -Path $readOnlyPath -Name IsReadOnly -Value $true
-                    # Build script should handle this appropriately on Windows
-                    { & $script:BuildScript -LayerPath $readOnlyPath -SkipRuntimeSetup 6>$null } | Should -Throw
-                }
-                else {
-                    # On non-Windows platforms, test that build succeeds even with permission issues
-                    # The build script should handle permission issues gracefully
-                    { & $script:BuildScript -LayerPath $readOnlyPath -SkipRuntimeSetup 6>$null } | Should -Not -Throw
-                }
-            }
-            finally {
-                # Clean up read-only attribute
-                if ($IsWindows -and (Test-Path $readOnlyPath)) {
-                    Set-ItemProperty -Path $readOnlyPath -Name IsReadOnly -Value $false
-                    Remove-Item $readOnlyPath -Recurse -Force -ErrorAction SilentlyContinue
-                }
-            }
-        }
     }
 
     Context "When testing download logic without runtime setup" {
@@ -754,4 +727,4 @@ function Production-Function {
     }
 
 
-}
+}
diff --git a/powershell-runtime/tests/unit/Private/Get-Handler.Tests.ps1 b/powershell-runtime/tests/unit/Private/Get-Handler.Tests.ps1
@@ -53,7 +53,7 @@ Describe "Get-Handler" {
             $result | Should -Not -BeNullOrEmpty
             $result.handlerType | Should -Be 'Script'
             $result.scriptFileName | Should -Be "handler.ps1"
-            $result.scriptFilePath | Should -Be "/var/task/handler.ps1"
+            Assert-PathEquals -Actual $result.scriptFilePath -Expected "/var/task/handler.ps1"
             $result.PSObject.Properties['functionName'] | Should -BeNullOrEmpty
             $result.PSObject.Properties['moduleName'] | Should -BeNullOrEmpty
         }
@@ -68,7 +68,7 @@ Describe "Get-Handler" {
             # Assert
             $result.handlerType | Should -Be 'Script'
             $result.scriptFileName | Should -Be "my-complex-handler.ps1"
-            $result.scriptFilePath | Should -Be "/var/task/my-complex-handler.ps1"
+            Assert-PathEquals -Actual $result.scriptFilePath -Expected "/var/task/my-complex-handler.ps1"
         }
 
         It "Should use LAMBDA_TASK_ROOT environment variable for script path" {
@@ -80,7 +80,7 @@ Describe "Get-Handler" {
             $result = pwsh-runtime\Get-Handler
 
             # Assert
-            $result.scriptFilePath | Should -Be "/custom/task/root/test.ps1"
+            Assert-PathEquals -Actual $result.scriptFilePath -Expected "/custom/task/root/test.ps1"
         }
 
         It "Should handle script handler with subdirectory path" {
@@ -93,7 +93,7 @@ Describe "Get-Handler" {
             # Assert
             $result.handlerType | Should -Be 'Script'
             $result.scriptFileName | Should -Be "subfolder/handler.ps1"
-            $result.scriptFilePath | Should -Be "/var/task/subfolder/handler.ps1"
+            Assert-PathEquals -Actual $result.scriptFilePath -Expected "/var/task/subfolder/handler.ps1"
         }
     }
 
@@ -109,7 +109,7 @@ Describe "Get-Handler" {
             $result | Should -Not -BeNullOrEmpty
             $result.handlerType | Should -Be 'Function'
             $result.scriptFileName | Should -Be "handler.ps1"
-            $result.scriptFilePath | Should -Be "/var/task/handler.ps1"
+            Assert-PathEquals -Actual $result.scriptFilePath -Expected "/var/task/handler.ps1"
             $result.functionName | Should -Be "MyFunction"
             $result.PSObject.Properties['moduleName'] | Should -BeNullOrEmpty
         }
@@ -125,7 +125,7 @@ Describe "Get-Handler" {
             $result.handlerType | Should -Be 'Function'
             $result.scriptFileName | Should -Be "my-script-file.ps1"
             $result.functionName | Should -Be "My-Complex-Function-Name"
-            $result.scriptFilePath | Should -Be "/var/task/my-script-file.ps1"
+            Assert-PathEquals -Actual $result.scriptFilePath -Expected "/var/task/my-script-file.ps1"
         }
 
         It "Should use LAMBDA_TASK_ROOT for function handler script path" {
@@ -137,7 +137,7 @@ Describe "Get-Handler" {
             $result = pwsh-runtime\Get-Handler
 
             # Assert
-            $result.scriptFilePath | Should -Be "/custom/path/handler.ps1"
+            Assert-PathEquals -Actual $result.scriptFilePath -Expected "/custom/path/handler.ps1"
         }
 
         It "Should handle function handler with script in subdirectory" {
@@ -151,7 +151,7 @@ Describe "Get-Handler" {
             $result.handlerType | Should -Be 'Function'
             $result.scriptFileName | Should -Be "lib/utilities.ps1"
             $result.functionName | Should -Be "Get-Data"
-            $result.scriptFilePath | Should -Be "/var/task/lib/utilities.ps1"
+            Assert-PathEquals -Actual $result.scriptFilePath -Expected "/var/task/lib/utilities.ps1"
         }
     }
 
@@ -229,7 +229,7 @@ Describe "Get-Handler" {
 
             # Assert
             $result.scriptFileName | Should -Be "custom-handler.ps1"
-            $result.scriptFilePath | Should -Be "/var/task/custom-handler.ps1"
+            Assert-PathEquals -Actual $result.scriptFilePath -Expected "/var/task/custom-handler.ps1"
         }
 
         It "Should handle custom function handler parameter" {
@@ -347,4 +347,4 @@ Describe "Get-Handler" {
             { pwsh-runtime\Get-Handler } | Should -Throw
         }
     }
-}
+}
diff --git a/powershell-runtime/tests/unit/Private/Set-PSModulePath.Tests.ps1 b/powershell-runtime/tests/unit/Private/Set-PSModulePath.Tests.ps1

Original file line number	Diff line number	Diff line change
`@@ -53,7 +53,7 @@ Describe "Get-Handler" {`
`53`	`53`	`$result \| Should -Not -BeNullOrEmpty`
`54`	`54`	`$result.handlerType \| Should -Be 'Script'`
`55`	`55`	`$result.scriptFileName \| Should -Be "handler.ps1"`
`56`		`- $result.scriptFilePath \| Should -Be "/var/task/handler.ps1"`
	`56`	`+ Assert-PathEquals -Actual $result.scriptFilePath -Expected "/var/task/handler.ps1"`
`57`	`57`	`$result.PSObject.Properties['functionName'] \| Should -BeNullOrEmpty`
`58`	`58`	`$result.PSObject.Properties['moduleName'] \| Should -BeNullOrEmpty`
`59`	`59`	`}`
`@@ -68,7 +68,7 @@ Describe "Get-Handler" {`
`68`	`68`	`# Assert`
`69`	`69`	`$result.handlerType \| Should -Be 'Script'`
`70`	`70`	`$result.scriptFileName \| Should -Be "my-complex-handler.ps1"`
`71`		`- $result.scriptFilePath \| Should -Be "/var/task/my-complex-handler.ps1"`
	`71`	`+ Assert-PathEquals -Actual $result.scriptFilePath -Expected "/var/task/my-complex-handler.ps1"`
`72`	`72`	`}`
`73`	`73`
`74`	`74`	`It "Should use LAMBDA_TASK_ROOT environment variable for script path" {`
`@@ -80,7 +80,7 @@ Describe "Get-Handler" {`
`80`	`80`	`$result = pwsh-runtime\Get-Handler`
`81`	`81`
`82`	`82`	`# Assert`
`83`		`- $result.scriptFilePath \| Should -Be "/custom/task/root/test.ps1"`
	`83`	`+ Assert-PathEquals -Actual $result.scriptFilePath -Expected "/custom/task/root/test.ps1"`
`84`	`84`	`}`
`85`	`85`
`86`	`86`	`It "Should handle script handler with subdirectory path" {`
`@@ -93,7 +93,7 @@ Describe "Get-Handler" {`
`93`	`93`	`# Assert`
`94`	`94`	`$result.handlerType \| Should -Be 'Script'`
`95`	`95`	`$result.scriptFileName \| Should -Be "subfolder/handler.ps1"`
`96`		`- $result.scriptFilePath \| Should -Be "/var/task/subfolder/handler.ps1"`
	`96`	`+ Assert-PathEquals -Actual $result.scriptFilePath -Expected "/var/task/subfolder/handler.ps1"`
`97`	`97`	`}`
`98`	`98`	`}`
`99`	`99`
`@@ -109,7 +109,7 @@ Describe "Get-Handler" {`
`109`	`109`	`$result \| Should -Not -BeNullOrEmpty`
`110`	`110`	`$result.handlerType \| Should -Be 'Function'`
`111`	`111`	`$result.scriptFileName \| Should -Be "handler.ps1"`
`112`		`- $result.scriptFilePath \| Should -Be "/var/task/handler.ps1"`
	`112`	`+ Assert-PathEquals -Actual $result.scriptFilePath -Expected "/var/task/handler.ps1"`
`113`	`113`	`$result.functionName \| Should -Be "MyFunction"`
`114`	`114`	`$result.PSObject.Properties['moduleName'] \| Should -BeNullOrEmpty`
`115`	`115`	`}`
`@@ -125,7 +125,7 @@ Describe "Get-Handler" {`
`125`	`125`	`$result.handlerType \| Should -Be 'Function'`
`126`	`126`	`$result.scriptFileName \| Should -Be "my-script-file.ps1"`
`127`	`127`	`$result.functionName \| Should -Be "My-Complex-Function-Name"`
`128`		`- $result.scriptFilePath \| Should -Be "/var/task/my-script-file.ps1"`
	`128`	`+ Assert-PathEquals -Actual $result.scriptFilePath -Expected "/var/task/my-script-file.ps1"`
`129`	`129`	`}`
`130`	`130`
`131`	`131`	`It "Should use LAMBDA_TASK_ROOT for function handler script path" {`
`@@ -137,7 +137,7 @@ Describe "Get-Handler" {`
`137`	`137`	`$result = pwsh-runtime\Get-Handler`
`138`	`138`
`139`	`139`	`# Assert`
`140`		`- $result.scriptFilePath \| Should -Be "/custom/path/handler.ps1"`
	`140`	`+ Assert-PathEquals -Actual $result.scriptFilePath -Expected "/custom/path/handler.ps1"`
`141`	`141`	`}`
`142`	`142`
`143`	`143`	`It "Should handle function handler with script in subdirectory" {`
`@@ -151,7 +151,7 @@ Describe "Get-Handler" {`
`151`	`151`	`$result.handlerType \| Should -Be 'Function'`
`152`	`152`	`$result.scriptFileName \| Should -Be "lib/utilities.ps1"`
`153`	`153`	`$result.functionName \| Should -Be "Get-Data"`
`154`		`- $result.scriptFilePath \| Should -Be "/var/task/lib/utilities.ps1"`
	`154`	`+ Assert-PathEquals -Actual $result.scriptFilePath -Expected "/var/task/lib/utilities.ps1"`
`155`	`155`	`}`
`156`	`156`	`}`
`157`	`157`
`@@ -229,7 +229,7 @@ Describe "Get-Handler" {`
`229`	`229`
`230`	`230`	`# Assert`
`231`	`231`	`$result.scriptFileName \| Should -Be "custom-handler.ps1"`
`232`		`- $result.scriptFilePath \| Should -Be "/var/task/custom-handler.ps1"`
	`232`	`+ Assert-PathEquals -Actual $result.scriptFilePath -Expected "/var/task/custom-handler.ps1"`
`233`	`233`	`}`
`234`	`234`
`235`	`235`	`It "Should handle custom function handler parameter" {`
`@@ -347,4 +347,4 @@ Describe "Get-Handler" {`
`347`	`347`	`{ pwsh-runtime\Get-Handler } \| Should -Throw`
`348`	`348`	`}`
`349`	`349`	`}`
`350`		`-}`
	`350`	`+}`