Skip to content

Update tests to use new logger module #75

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 1 commit into
base: feature/logger
Choose a base branch
from

Conversation

TinaMor
Copy link
Contributor

@TinaMor TinaMor commented May 21, 2025

PR Description

What does this PR achieve or fix?

- _Related isuue/discussion(s): Link to the issue or discussion that this PR addresses._
- _Background information: From the reviewer's perspective, what context do they need to understand this PR?_
_Additional information: What does this PR achieve/fix._

TODOs Not Included in This PR

What issues will/should be addressed in follow-up PRs? Link any existing related issues.

Testing Information

How to test the changes

Relevant Links

Any links from your research that help explain the changes

Checklist

As part of our commitment to engineering excellence, before submitting this PR, please ensure:

  • You have tested this code in both Desktop and Server environments, as well as AMD64 and ARM64 environments (functional testing).
  • You have added unit tests for new code.
  • You have added or updated documentation in the cmdlet docs, command-reference.md, and the modules help files.
  • You have reviewed the PR/code best practices defined in the CONTRIBUTING.md.

In addition, after this PR has been reviewed, please ensure:

  • If you make changes while addressing review comments, you test the final version again in both AMD64 and ARM64 environments.
  • You validate that your changes have not introduced any regressions.

    - Import logger module and mock PS EventLog commands
@TinaMor TinaMor force-pushed the tinamor/persist-logs-update-tests branch from 8da4f87 to e0fc6dc Compare May 21, 2025 14:17
@TinaMor TinaMor changed the title Tinamor/persist logs update tests Update tests to use new logger module May 21, 2025
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.

1 participant