-
Notifications
You must be signed in to change notification settings - Fork 75
Fix ESXi configstore call in OS create method #1499
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
Conversation
Schamper
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it possible to check this in a unit test?
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1499 +/- ##
==========================================
+ Coverage 80.45% 80.73% +0.28%
==========================================
Files 393 393
Lines 34544 34542 -2
==========================================
+ Hits 27792 27888 +96
+ Misses 6752 6654 -98
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
@william-billaud Could you share the |
|
I will take a look tomorrow. Maybe tech supports files from https://github.com/fox-it/dissect.target/pull/1492/files would fit your need? (esx8) |
|
@william-billaud That would definitely be helpful for the test coverage of ESXi functionality. Perhaps we can add that in a separate PR. @Schamper I've added a small test in c8a2a8e for the meantime. |
Since
_configstoreis no longer populated in theESXiPlugin, we have to update the reference to the config store in thecreatemethod.The
check_compatiblemethod ofConfigStorePluginis also updated to not usetarget.ossince that would cause recursion.