Repository navigation
test(v3.2.65): New-CloudSchedule.ps1 ユニットテスト追加 — Build-CreatePrompt / New-LoopPreset 23件 #227
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change | ||||||
|---|---|---|---|---|---|---|---|---|
| @@ -0,0 +1,180 @@ | ||||||||
| # ============================================================ | ||||||||
| # NewCloudSchedule.Tests.ps1 - New-CloudSchedule.ps1 unit tests | ||||||||
| # Pester 5.x / Issue #226 | ||||||||
| # | ||||||||
| # Strategy: dot-source the script with exit->return patching so the | ||||||||
| # Pester process is not terminated by the -NonInteractive exit 0. | ||||||||
| # A fake claude.cmd stub is added to PATH to satisfy the CLI check. | ||||||||
| # ============================================================ | ||||||||
|
|
||||||||
| BeforeAll { | ||||||||
| $script:RepoRoot = Split-Path -Parent (Split-Path -Parent $PSScriptRoot) | ||||||||
| $script:ScriptPath = Join-Path $script:RepoRoot 'scripts\main\New-CloudSchedule.ps1' | ||||||||
|
|
||||||||
| # --- Fake claude stub (satisfies the CLI-detection block) --- | ||||||||
| $script:TempDir = Join-Path ([System.IO.Path]::GetTempPath()) 'pester-cloudschedule-tests' | ||||||||
|
There was a problem hiding this comment. 一時ディレクトリが固定名
Suggested change
|
||||||||
| $null = New-Item -ItemType Directory -Path $script:TempDir -Force | ||||||||
| '@echo off' | Set-Content (Join-Path $script:TempDir 'claude.cmd') -Encoding ASCII | ||||||||
| $script:SavedPath = $env:PATH | ||||||||
| $env:PATH = "$script:TempDir;$env:PATH" | ||||||||
|
|
||||||||
| # --- Patch: replace 'exit <n>' with 'return' to prevent killing Pester --- | ||||||||
| $raw = Get-Content $script:ScriptPath -Raw -Encoding UTF8 | ||||||||
| $patched = $raw -replace '(?m)^\s*exit\s+\d+\s*$', 'return' | ||||||||
| $script:TempScript = Join-Path $script:TempDir 'NewCloudSchedule.tmp.ps1' | ||||||||
| $patched | Set-Content $script:TempScript -Encoding UTF8 | ||||||||
|
Comment on lines
+15
to
+25
There was a problem hiding this comment. 一時ディレクトリ名が固定で並列実行時に競合します
💡 修正案- $script:TempDir = Join-Path ([System.IO.Path]::GetTempPath()) 'pester-cloudschedule-tests'
+ $script:TempDir = Join-Path ([System.IO.Path]::GetTempPath()) ("pester-cloudschedule-tests-{0}" -f [guid]::NewGuid().ToString('N'))🤖 Prompt for AI Agents |
||||||||
|
|
||||||||
| # Dot-source in NonInteractive mode: defines all functions, sets script: vars, then returns | ||||||||
| . $script:TempScript -NonInteractive | ||||||||
|
Comment on lines
+21
to
+28
There was a problem hiding this comment.
|
||||||||
| } | ||||||||
|
|
||||||||
| AfterAll { | ||||||||
| $env:PATH = $script:SavedPath | ||||||||
| if (Test-Path $script:TempDir) { Remove-Item $script:TempDir -Recurse -Force -ErrorAction SilentlyContinue } | ||||||||
| } | ||||||||
|
|
||||||||
| # ============================================================ | ||||||||
| # New-LoopPreset | ||||||||
| # ============================================================ | ||||||||
| Describe 'New-LoopPreset' { | ||||||||
|
|
||||||||
| It 'returns exactly 4 presets' { | ||||||||
| $result = New-LoopPreset -Url 'https://github.com/test/repo' | ||||||||
| $result.Count | Should -Be 4 | ||||||||
| } | ||||||||
|
|
||||||||
| It 'preset 0 is named ClaudeOS Monitor' { | ||||||||
| $result = New-LoopPreset -Url 'https://github.com/test/repo' | ||||||||
| $result[0].Name | Should -Be 'ClaudeOS Monitor' | ||||||||
| } | ||||||||
|
|
||||||||
| It 'Monitor cron is hourly on Mon-Sat (0 * * * 1-6)' { | ||||||||
| $result = New-LoopPreset -Url 'https://github.com/test/repo' | ||||||||
| $result[0].Cron | Should -Be '0 * * * 1-6' | ||||||||
| } | ||||||||
|
|
||||||||
| It 'preset 1 is named ClaudeOS Development' { | ||||||||
| $result = New-LoopPreset -Url 'https://github.com/test/repo' | ||||||||
| $result[1].Name | Should -Be 'ClaudeOS Development' | ||||||||
| } | ||||||||
|
|
||||||||
| It 'Development cron is every 2 hours on Mon-Sat (0 */2 * * 1-6)' { | ||||||||
| $result = New-LoopPreset -Url 'https://github.com/test/repo' | ||||||||
| $result[1].Cron | Should -Be '0 */2 * * 1-6' | ||||||||
| } | ||||||||
|
|
||||||||
| It 'preset 2 is named ClaudeOS Verify with hourly cron' { | ||||||||
| $result = New-LoopPreset -Url 'https://github.com/test/repo' | ||||||||
| $result[2].Name | Should -Be 'ClaudeOS Verify' | ||||||||
| $result[2].Cron | Should -Be '0 * * * 1-6' | ||||||||
| } | ||||||||
|
|
||||||||
| It 'preset 3 is named ClaudeOS Improvement with hourly cron' { | ||||||||
| $result = New-LoopPreset -Url 'https://github.com/test/repo' | ||||||||
| $result[3].Name | Should -Be 'ClaudeOS Improvement' | ||||||||
| $result[3].Cron | Should -Be '0 * * * 1-6' | ||||||||
| } | ||||||||
|
|
||||||||
| It 'injects repository URL into every Content field' { | ||||||||
| $url = 'https://github.com/TestOrg/my-special-repo' | ||||||||
| $urlPattern = [regex]::Escape($url) | ||||||||
| $result = New-LoopPreset -Url $url | ||||||||
| foreach ($preset in $result) { | ||||||||
| $preset.Content | Should -Match $urlPattern | ||||||||
| } | ||||||||
| } | ||||||||
|
|
||||||||
| It 'each preset has Label, Name, Cron, Content properties' { | ||||||||
| $result = New-LoopPreset -Url 'https://github.com/test/repo' | ||||||||
| foreach ($preset in $result) { | ||||||||
| $preset.PSObject.Properties.Name | Should -Contain 'Label' | ||||||||
| $preset.PSObject.Properties.Name | Should -Contain 'Name' | ||||||||
| $preset.PSObject.Properties.Name | Should -Contain 'Cron' | ||||||||
| $preset.PSObject.Properties.Name | Should -Contain 'Content' | ||||||||
| } | ||||||||
| } | ||||||||
|
|
||||||||
| It 'all Content values are non-empty' { | ||||||||
| $result = New-LoopPreset -Url 'https://github.com/test/repo' | ||||||||
| foreach ($preset in $result) { | ||||||||
| $preset.Content.Trim() | Should -Not -BeNullOrEmpty | ||||||||
| } | ||||||||
| } | ||||||||
|
|
||||||||
| It 'each preset Label is non-empty' { | ||||||||
| $result = New-LoopPreset -Url 'https://github.com/test/repo' | ||||||||
| foreach ($preset in $result) { | ||||||||
| $preset.Label | Should -Not -BeNullOrEmpty | ||||||||
| } | ||||||||
| } | ||||||||
|
|
||||||||
| It 'accepts different repository URLs without error' { | ||||||||
| { New-LoopPreset -Url 'https://github.com/another-org/another-repo' } | Should -Not -Throw | ||||||||
| } | ||||||||
| } | ||||||||
|
|
||||||||
| # ============================================================ | ||||||||
| # Build-CreatePrompt | ||||||||
| # ============================================================ | ||||||||
| Describe 'Build-CreatePrompt' { | ||||||||
|
|
||||||||
| It 'contains RemoteTrigger creation instruction' { | ||||||||
| $result = Build-CreatePrompt -Name 'TestTrigger' -Cron '0 * * * 1-6' -PromptContent 'hello' | ||||||||
| $result | Should -Match 'RemoteTrigger' | ||||||||
| } | ||||||||
|
|
||||||||
| It 'embeds Name in prompt output' { | ||||||||
| $result = Build-CreatePrompt -Name 'ClaudeOS Monitor' -Cron '0 * * * 1-6' -PromptContent 'content' | ||||||||
| $result | Should -Match '"ClaudeOS Monitor"' | ||||||||
| } | ||||||||
|
|
||||||||
| It 'embeds Cron expression in prompt output' { | ||||||||
| $cronPattern = [regex]::Escape('"0 */2 * * 1-6"') | ||||||||
| $result = Build-CreatePrompt -Name 'T' -Cron '0 */2 * * 1-6' -PromptContent 'content' | ||||||||
| $result | Should -Match $cronPattern | ||||||||
| } | ||||||||
|
|
||||||||
| It 'sets enabled: true' { | ||||||||
| $result = Build-CreatePrompt -Name 'T' -Cron '0 * * * 1-6' -PromptContent 'x' | ||||||||
| $result | Should -Match 'enabled: true' | ||||||||
| } | ||||||||
|
|
||||||||
| It 'sets model to claude-sonnet-4-6' { | ||||||||
| $result = Build-CreatePrompt -Name 'T' -Cron '0 * * * 1-6' -PromptContent 'x' | ||||||||
| $result | Should -Match 'claude-sonnet-4-6' | ||||||||
| } | ||||||||
|
|
||||||||
| It 'escapes double-quotes in PromptContent' { | ||||||||
| $result = Build-CreatePrompt -Name 'T' -Cron '0 * * * 1-6' -PromptContent 'say "hello" now' | ||||||||
| $result | Should -Match '\\"hello\\"' | ||||||||
| } | ||||||||
|
|
||||||||
| It 'escapes newlines in PromptContent as literal \n' { | ||||||||
| $result = Build-CreatePrompt -Name 'T' -Cron '0 * * * 1-6' -PromptContent "line1`nline2" | ||||||||
| # Content field should contain literal \n (2 chars), not a real newline | ||||||||
| $result | Should -Match 'line1\\nline2' | ||||||||
| } | ||||||||
|
|
||||||||
| It 'generates a UUID (RFC 4122 format)' { | ||||||||
| $result = Build-CreatePrompt -Name 'T' -Cron '0 * * * 1-6' -PromptContent 'x' | ||||||||
| $result | Should -Match '[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}' | ||||||||
| } | ||||||||
|
|
||||||||
| It 'generates a different UUID on each call' { | ||||||||
| $r1 = Build-CreatePrompt -Name 'T' -Cron '0 * * * 1-6' -PromptContent 'x' | ||||||||
| $r2 = Build-CreatePrompt -Name 'T' -Cron '0 * * * 1-6' -PromptContent 'x' | ||||||||
| $uuid1 = [regex]::Match($r1, '[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}').Value | ||||||||
| $uuid2 = [regex]::Match($r2, '[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}').Value | ||||||||
| $uuid1 | Should -Not -Be $uuid2 | ||||||||
| } | ||||||||
|
|
||||||||
| It 'includes CREATED_ID output instruction' { | ||||||||
| $result = Build-CreatePrompt -Name 'T' -Cron '0 * * * 1-6' -PromptContent 'x' | ||||||||
| $result | Should -Match 'CREATED_ID=' | ||||||||
| } | ||||||||
|
|
||||||||
| It 'includes git_repository source URL' { | ||||||||
| $result = Build-CreatePrompt -Name 'T' -Cron '0 * * * 1-6' -PromptContent 'x' | ||||||||
| $result | Should -Match 'git_repository' | ||||||||
| } | ||||||||
| } | ||||||||
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
README 内のテスト件数が不整合です
Line 31 は 703 件に更新されていますが、Line 96 / Line 231 / Line 507 / Line 538 などに旧値が残っており、README 単体で真値が判別しづらい状態です。
💡 修正案（同PR内で揃える）
As per coding guidelines
**/*.md: maintain README as external-facing truth with clear organization, extensive use of tables and icons, and include architecture diagrams.🤖 Prompt for AI Agents