Skip to content

fix: trim timeuntil event names - #83

Merged
vycdev merged 1 commit into
vycdev:developfrom
vycdev2:fix/timeuntil-trim-whitespace
Aug 10, 2026
Merged

fix: trim timeuntil event names#83
vycdev merged 1 commit into
vycdev:developfrom
vycdev2:fix/timeuntil-trim-whitespace

Conversation

@vycdev2

@vycdev2 vycdev2 commented Aug 8, 2026

Copy link
Copy Markdown
Contributor

Summary

  • trim surrounding whitespace before matching timeuntil event names, so padded valid names are accepted
  • add regression coverage for spaces, tabs, and line endings around event names

Verification

  • dotnet build --no-restore — passed (1 existing NU1903 package advisory warning)
  • dotnet test Morpheus.Tests/Morpheus.Tests.csproj --no-restore --filter FullyQualifiedName~NormalizeTimeUntilEventName_TrimsSurroundingWhitespace --verbosity minimal — passed, 2/2
  • dotnet test Morpheus.Tests/Morpheus.Tests.csproj --no-restore --filter 'FullyQualifiedName!~NormalizeTimeUntilEventName_NormalizesCase' --verbosity minimal — passed, 299/299
  • dotnet test --no-restore --verbosity minimal — 299 passed, 2 failed because the existing Turkish-culture test cannot load tr-TR in globalization-invariant mode (tracked by open PR test: skip unavailable culture regression #81)
  • git diff --check upstream/develop...HEAD — passed

Risk

  • Low: normalization now removes only surrounding whitespace before the existing invariant case conversion; internal whitespace and supported aliases are unchanged.

This was generated by an AI agent (vycdev2). Please verify any changes before merging or applying.

@vycdev
vycdev merged commit 7a559b7 into vycdev:develop Aug 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants