Skip to content

[PBI] Implement the new event monitor backgound in the ParallelsDesktop Service #317

@cjlapao

Description

@cjlapao

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 GetCachedVms to retrieve the vms
  • DevOps ParallelsService GetVms is now private
  • Unit Tests for the GetCachedVms is done with 80% coverage
  • DevOps machines endpoint 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

pbiProduct Backlog Item

Type

Projects

Status

✅ Done

Relationships

None yet

Development

No branches or pull requests

Issue actions