Skip to content

Normalize expanded field names#16

Merged
PrzemyslawKlys merged 3 commits into
masterfrom
transform-field-keys-to-powershell-identifiers
Aug 23, 2025
Merged

Normalize expanded field names#16
PrzemyslawKlys merged 3 commits into
masterfrom
transform-field-keys-to-powershell-identifiers

Conversation

@PrzemyslawKlys
Copy link
Copy Markdown
Member

Summary

  • sanitize expanded field keys into PowerShell-friendly identifiers
  • document field name transformation and provide usage example
  • test expanded property names via Pester

Testing

  • dotnet build
  • pwsh -NoLogo -Command "Import-Module Pester; Import-Module ./Module/IISParser.psd1; Invoke-Pester Module/Tests/Get-IISParsedLog.Tests.ps1 -Output Detailed"
  • dotnet test

https://chatgpt.com/codex/tasks/task_e_68a97a322948832e92ec7c4da655ac08

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Aug 23, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.80%. Comparing base (3e113bf) to head (5a84c70).
⚠️ Report is 19 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master      #16   +/-   ##
=======================================
  Coverage   82.80%   82.80%           
=======================================
  Files           4        4           
  Lines         157      157           
  Branches       32       32           
=======================================
  Hits          130      130           
  Misses         20       20           
  Partials        7        7           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@PrzemyslawKlys PrzemyslawKlys merged commit 4ae247e into master Aug 23, 2025
8 checks passed
@PrzemyslawKlys PrzemyslawKlys deleted the transform-field-keys-to-powershell-identifiers branch August 23, 2025 14:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants