chore: Console output overrides lambda console out

[hjgraca]

Please provide the issue number

Issue number: #932

Summary

Changes

This pull request refactors the ConsoleWrapper class to improve its handling of console output in AWS Lambda environments and enhance testability. It introduces new methods for determining and overriding console output, removes deprecated logic, and updates the test suite to ensure comprehensive coverage of the changes.

Refactoring of ConsoleWrapper:

• Removed the _outputResetPerformed flag and replaced EnsureConsoleOutputOnce with EnsureConsoleOutput, which uses the new ShouldOverrideConsole method to decide when to override console output. [1] [2] [3] [4]
• Added ShouldOverrideConsole and HasLambdaReInterceptedConsole methods to improve the detection of whether console output should be overridden in Lambda environments.
• Enhanced error handling in OverrideLambdaLogger to prevent crashes if console override fails.

Test Suite Enhancements:

• Refactored ConsoleWrapperTests to include setup and teardown logic for isolating test state, ensuring reliable test execution.
• Added new test cases to validate behavior in Lambda and non-Lambda environments, including scenarios where Lambda re-intercepts console output or console override fails. [1] [2]
• Removed outdated tests for _outputResetPerformed and replaced them with updated tests for the new logic.

Code Cleanup:

• Fixed a typo in variable naming (errordOutput to errorOutput) and improved code readability.
• Deprecated the ClearOutputResetFlag method but retained it for backward compatibility, ensuring it does not throw errors. [1] [2]

User experience

Please share what the user experience looks like before and after this change

Checklist

Please leave checklist items unchecked if they do not apply to your change.

• Meets tenets criteria
• I have performed a self-review of this change
• Changes have been tested
• Changes are documented
• PR title follows conventional commit semantics

Is this a breaking change?

RFC issue number:

Checklist:

• Migration process documented
• Implement warnings (if it can live side by side)

Acknowledgment

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.

[codecov]

Codecov Report

Attention: Patch coverage is 92.30769% with 3 lines in your changes missing coverage. Please review.

Project coverage is 77.35%. Comparing base (c2711e2) to head (49b66e8).
Report is 4 commits behind head on develop.

Files with missing lines	Patch %	Lines
...WS.Lambda.Powertools.Common/Core/ConsoleWrapper.cs	92.30%	0 Missing and 3 partials ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##           develop     #936      +/-   ##
===========================================
+ Coverage    77.31%   77.35%   +0.04%     
===========================================
  Files          271      271              
  Lines        10839    10864      +25     
  Branches      1277     1280       +3     
===========================================
+ Hits          8380     8404      +24     
+ Misses        2043     2042       -1     
- Partials       416      418       +2     

☔ 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.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud