Skip to content

Conversation

@PrzemyslawKlys
Copy link
Member

Summary

  • stream parsed events from log files instead of buffering in memory
  • lazily consume ParseLog results in Get-IISParsedLog
  • add test coverage and example for streaming behavior

Testing

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

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

@codecov-commenter
Copy link

codecov-commenter commented Aug 23, 2025

Codecov Report

❌ Patch coverage is 46.15385% with 7 lines in your changes missing coverage. Please review.
✅ Project coverage is 85.82%. Comparing base (6ac3ed8) to head (740304e).
⚠️ Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
IISParser/ParserEngine.cs 46.15% 7 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master      #12      +/-   ##
==========================================
- Coverage   86.50%   85.82%   -0.69%     
==========================================
  Files           3        3              
  Lines         126      127       +1     
  Branches       18       20       +2     
==========================================
  Hits          109      109              
- Misses         13       14       +1     
  Partials        4        4              

☔ 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 2059bdb into master Aug 23, 2025
8 checks passed
@PrzemyslawKlys PrzemyslawKlys deleted the refactor-parsing-methods-for-lazy-consumption branch August 23, 2025 07:15
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.

3 participants