remove special characters from file (#66)

* include title in frontmatter

* remove special characters from file name

---------

Co-authored-by: bryan cook <3217452+bryancook@users.noreply.github.com>

Author: bryanbcook

src/public/Get-BloggerPost.ps1

@@ -135,6 +135,7 @@ Function Get-BloggerPost {
           $title = $result.title
           $frontMatter = [ordered]@{
             postId = $result.id
+            title = $result.title
           }
           if ($result.PSObject.Properties.Name -contains "labels") {
             Write-Verbose "Using post labels: $($result.labels)"
@@ -144,7 +145,8 @@ Function Get-BloggerPost {
             $frontMatter['tags'] = @()
           }
           Write-Verbose "Saving frontmatter: $($frontMatter | ConvertTo-Json -Depth 10)"
-          $file = "$title.md"
+          $sanitizedTitle = $title -replace '[\\\/:\*\?"<>\|]', '_' -replace '\s+', ' ' -replace '^\s+|\s+$',''
+          $file = "$sanitizedTitle.md"
 
           $filePath = Join-Path -Path $OutDirectory -ChildPath $file
           ConvertTo-MarkdownFromHtml -Content $result.content -OutFile $filePath

src/tests/Get-BloggerPost.Tests.ps1

@@ -213,6 +213,15 @@ Describe "Get-BloggerPost" {
       $frontMatter.postId | Should -Be "123"
     }
 
+    It "Should write title to frontmatter" {
+      # act
+      Get-BloggerPost -PostId $postId -Format Markdown -OutDirectory "TestDrive:\"
+
+      # assert
+      $frontMatter = Get-MarkdownFrontMatter -File $outFile
+      $frontMatter.title | Should -Be "Test Post"
+    }
+
     It "Should write labels to frontmatter" {
       # act
       Get-BloggerPost -PostId $postId -Format Markdown -OutDirectory "TestDrive:\"
@@ -250,6 +259,54 @@ Describe "Get-BloggerPost" {
       $frontMatter['tags'] -eq $null | Should -BeFalse
       $frontMatter['tags'] | Should -Be @()
     }
+
+    Context "Illegal characters in title" {
+      It "Should sanitize filename when title contains special character (<character>)" -TestCases @(
+        @{ character = '?'; }
+        @{ character = '/'; }
+        @{ character = ':'; }
+        @{ character = '*'; }
+        @{ character = '"'; }
+        @{ character = '<'; }
+        @{ character = '>'; }
+        @{ character = '|'; }
+      ) {
+        $title = "Test$($character)Post"
+        
+        # arrange
+        InModuleScope PSBlogger -Parameters @{ testTitle = $title } {
+          param($testTitle)
+          
+          $script:currentTitle = $testTitle
+
+          # Mock the session to return a test blog ID
+          $BloggerSession.BlogId = "test-blog-id"
+
+          $postId = "123"
+
+          # mock post retrieval
+          Mock Invoke-GApi -MockWith { 
+            [pscustomobject]@{ 
+              id = "123"
+              title = $script:currentTitle
+              published = [datetime]"2023-10-01T17:30:00-04:00"
+              content = "<h1>Hello World</h1><p>This is a post.</p>" 
+            }
+          }
+        }
+
+        $expectedFileName = "Test_Post.md"
+        $outFile = Get-TestFilePath $expectedFileName
+
+        # act
+        Get-BloggerPost -PostId $postId -Format Markdown -OutDirectory "TestDrive:\"
+
+        # assert
+        Test-Path $outFile | Should -BeTrue
+        $frontMatter = Get-MarkdownFrontMatter -File $outFile
+        $frontMatter.title | Should -Be $title
+      }
+    }
   }
 
   Context "As Json" {