modernisation-platform icon indicating copy to clipboard operation
modernisation-platform copied to clipboard

Refactor instance-scheduler to make the code DRY

Open ewastempel opened this issue 1 year ago • 1 comments

User Story

The instance scheduler supports RDS instances scheduling now, but there is a lot of code repetition. See if you can introduce a Instance interface type that will serve both EC2 and RDS clients and replace the common functions with methods. Note there is an equal amount of repetition in tests too. This will also need updating as we change the main code.

User Type(s)

Modernisation Platform Engineer

Value

DRY code

Assumptions / Hypothesis / Questions / Unknowns

Proposal

Extracting common functions for EC2 and RDS and passing client as a variable.

Unknowns

Is it possible?

Definition of done

  • [ ] the code has been refactored
  • [ ] the unit tests are updated to work with the main code
  • [ ] another team member has reviewed
  • [ ] tests are green

Reference

How to write good user stories

ewastempel avatar Sep 19 '23 09:09 ewastempel