Skip to content

feat(logging): Enable conditional logging based on --logger flag using dynamic Pino config#89

Open
lyutvs wants to merge 4 commits intomainfrom
fix/test_arguments
Open

feat(logging): Enable conditional logging based on --logger flag using dynamic Pino config#89
lyutvs wants to merge 4 commits intomainfrom
fix/test_arguments

Conversation

@lyutvs
Copy link
Collaborator

@lyutvs lyutvs commented Apr 17, 2025

This pull request introduces changes to enhance the configurability of the HubBackend and TestAutomation modules, particularly by adding support for enabling or disabling logging through a new logger option. It also refactors how options are handled and passed between components to improve flexibility and maintainability.

Enhancements to HubBackend:

  • Added a logger option to the open method of HubBackend, allowing conditional activation of logging and global exception filters. [1] [2]
  • Updated the import statements in src/HubBackend.ts to include TestAutomation, enabling integration with the new options.

Updates to TestAutomation:

  • Introduced a logger property in the TestAutomation.IOptions interface to enable or disable logging during test automation.
  • Updated the getOptions function in test/TestAutomation.ts to parse and handle the new logger option from the command line. [1] [2]

Refactoring in test/index.ts:

  • Modified the open method in the IBackend interface and its implementation to accept the TestAutomation.IOptions object, ensuring consistent option handling across components. [1] [2]

@lyutvs lyutvs added the enhancement New feature or request label Apr 17, 2025
@lyutvs lyutvs self-assigned this Apr 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant