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.

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

test: Add Unit Test for GetVolumes (ProjectedVolume) with source secret #505

Merged
merged 14 commits into from
Mar 30, 2023

Conversation

nidhunge
Copy link
Contributor

@nidhunge nidhunge commented Mar 29, 2023

Added unit test for Projected volumes in GetVolumes method with Secret source inside aci_volumes.go in aci_volumes_test.go test file.

Changed mock variable names for GetVolumes case secret to more realistic values.

#500

@nidhunge nidhunge temporarily deployed to test March 29, 2023 19:32 — with GitHub Actions Inactive
@nidhunge nidhunge temporarily deployed to test March 29, 2023 19:32 — with GitHub Actions Inactive
@codecov-commenter
Copy link

codecov-commenter commented Mar 29, 2023

Codecov Report

Merging #505 (aaddaad) into master (0bc8184) will increase coverage by 0.44%.
The diff coverage is n/a.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@            Coverage Diff             @@
##           master     #505      +/-   ##
==========================================
+ Coverage   51.03%   51.48%   +0.44%     
==========================================
  Files          14       14              
  Lines        2455     2455              
==========================================
+ Hits         1253     1264      +11     
+ Misses       1113     1100      -13     
- Partials       89       91       +2     

see 1 file with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@helayoty helayoty changed the title test: test: Added Unit Test for GetVolumes case: ProjectedVolume with source: Secret source test: Added Unit Test for GetVolumes case: ProjectedVolume with source: Secret source Mar 29, 2023
@nidhunge nidhunge temporarily deployed to test March 29, 2023 20:31 — with GitHub Actions Inactive
@nidhunge nidhunge temporarily deployed to test March 29, 2023 20:31 — with GitHub Actions Inactive
@Fei-Guo
Copy link
Collaborator

Fei-Guo commented Mar 29, 2023

LGTM

@helayoty helayoty mentioned this pull request Mar 29, 2023
4 tasks
@nidhunge nidhunge temporarily deployed to test March 29, 2023 22:07 — with GitHub Actions Inactive
@nidhunge nidhunge temporarily deployed to test March 29, 2023 22:07 — with GitHub Actions Inactive
@nidhunge nidhunge temporarily deployed to test March 29, 2023 23:45 — with GitHub Actions Inactive
@helayoty helayoty changed the title test: Added Unit Test for GetVolumes case: ProjectedVolume with source: Secret source test: Add Unit Test for GetVolumes (ProjectedVolume) with source secret Mar 30, 2023
@nidhunge nidhunge temporarily deployed to test March 30, 2023 17:44 — with GitHub Actions Inactive
@nidhunge nidhunge temporarily deployed to test March 30, 2023 17:44 — with GitHub Actions Inactive
@nidhunge nidhunge enabled auto-merge (squash) March 30, 2023 18:40
@nidhunge nidhunge requested a review from helayoty March 30, 2023 21:24
@nidhunge nidhunge merged commit 27c12c0 into virtual-kubelet:master Mar 30, 2023
@nidhunge nidhunge deleted the ndProjectedVolUT branch April 5, 2023 18:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants