CP-1192 Store: switch to a default ctor and a transformer ctor

[evanweible-wf]

Issue

• Store should have a default constructor with no optional parameters so that mixins can be used with it #48 Because the only constructor in Store has optional parameters, extending classes cannot mix other classes in.

Changes

Source:

• Fix Store should have a default constructor with no optional parameters so that mixins can be used with it #48 by removing the optional transformer parameter from the default Store constructor and replacing it with a named Store.withTransformer() constructor. This is a breaking change.

Tests:

• Update test for transformer usage.

Areas of Regression

• Store streams (transformed or not).

Testing

• CI passes.

Code Review

@trentgrover-wf
@maxwellpeterson-wf
@dustinlessard-wf
@jayudey-wf

[codecov-io]

Current coverage is 100.00%

Merging #49 into master will not affect coverage as of 2f89a51

Powered by Codecov. Updated on successful CI builds.

[trentgrover-wf]

+1

[dustinlessard-wf]

+1

[evanweible-wf]

@trentgrover-wf @dustinlessard-wf sorry, forgot to update the readme. Should be good now.

[dustinlessard-wf]

+1

[trentgrover-wf]

+1

[evanweible-wf]

@jayudey-wf ready for merge.

[trentgrover-wf]

@jayudey-wf let's merge and tag the other outstanding PR before this, since this will be a breaking change

[evanweible-wf]

good call

[jayudey-wf]

QA Resource Approval: +10

• Testing instruction
• Dev +1's
• Dev/QA +10 with detail of what was tested
  • passing CI with updated unit test
• Unit test created/updated
• All unit tests pass

Merging into master.