uber / cadence / 018f5bec-6933-41ac-ab62-df7c62b22c22

72%

Build Type

push

buildkite

Committed by web-flow

Commit Message

Minor splitting of mutable state builder file (#5990)

What changed?

This takes the methods of the MutableState implementation and subdevides them along conceptually thematic subfiles. For example all DecisionTask methods are put into one file, with ActivityTask ones in another. This isn't a complete splitting, the MutableState impl file still has a lot of smaller helper methods which I haven't bothered to refine further.

Why?

This is a minor refactor simply splitting the mutable-state-builder file into a few smaller files simply for tractability. It's not the desired end-state refactor I'd like to do, but unblocks other engineers from being able to pick up testing for this builder with less friction.

How did you test it?

Unit tests should cover this refactor. I've been careful to not do anything other than just move content between files.

Run Details

1337 of 1979 new or added lines in 10 files covered. (67.56%)

36 existing lines in 11 files now uncovered.

101097 of 146912 relevant lines covered (68.81%)

2459.15 hits per line