Skip to content
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.

Sign up for GitHub

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

Jump to bottom

test: add unit tests for the client and utils #73

Merged
merged 7 commits into from
Nov 10, 2024
Merged

test: add unit tests for the client and utils #73

merged 7 commits into from
Nov 10, 2024

Conversation

netr0m
Copy link
Owner

@netr0m netr0m commented Nov 5, 2024

Description

Adds tests and mocking for the azure client and utils

Type of change

Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)
  • This change requires a documentation update

Checklist:

  • I have read the contribution guidelines
  • My changes follow the Styleguide of this project
  • I have run the pre-commit hooks included in this repository
  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation
  • All GitHub Actions workflows for this branch/PR have run successfully

@netr0m netr0m added enhancement New feature or request dependencies Pull requests that update a dependency file labels Nov 5, 2024
@netr0m netr0m self-assigned this Nov 5, 2024
@netr0m netr0m added the test Adds or modifies tests label Nov 5, 2024
github-advanced-security[bot]
github-advanced-security bot found potential problems Nov 5, 2024
View reviewed changes
pkg/pim/test_data.go Dismissed Show dismissed Hide dismissed
@netr0m netr0m force-pushed the test/unit-tests-mocking branch from def0702 to d70aaa3 Compare November 7, 2024 08:31
@netr0m netr0m merged commit 3768852 into main Nov 10, 2024
10 checks passed
@netr0m netr0m deleted the test/unit-tests-mocking branch November 10, 2024 11:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@netr0m netr0m

Labels
dependencies Pull requests that update a dependency file enhancement New feature or request test Adds or modifies tests
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add unit tests
1 participant
@netr0m