-
Notifications
You must be signed in to change notification settings - Fork 6
Description
Description
We need to implement the new event emitter background service into the way the ParallelsDesktop service reads the VMs. For this we need to keep the current functionality as it is, due to dependencies.
At the moment the way it works is ParallelsDesktop service has a GetCachedVms that reads a cached json and if there is nothing there it will read the GetVms.
We will need to change the GetCachedVms to call the new service if there is no cached vms yet otherwise returns the cached ones as it is.
We will also need to change the GetVms to be a private function and check where it is used, it should only be used by the EventEmitter and everywhere else we need to update it to use the GetCachedVms one.
It is the responsibility of the new event emitter service to update the cached object and keep it up-to-date
User Story
As a developer I want the new ParallelsService to now use the new event emitter to get the correct vms every time
Acceptance Criteria
- DevOps only uses the
GetCachedVmsto retrieve the vms - DevOps
ParallelsServiceGetVmsis now private - Unit Tests for the
GetCachedVmsis done with 80% coverage - DevOps
machinesendpoint works as expected
Definition of Done
- Code implemented following best practices.
- Unit tests written and passing.
- Code reviewed and approved.
- Merged into the main branch.
- Documentation updated (if applicable).
- Deployed to staging/production environment.
Assumptions and Constraints
No response
Dependencies
No response
Additional Notes
No response
Metadata
Metadata
Assignees
Labels
Type
Projects
Status