Persist currentCurrency from state when re-initialising controller

[rickycodes]

re: MetaMask/metamask-extension#6710

re: the pinning, when I installed this I got tsc Version 3.6.4

which resulted in the following:

src/util.ts:113:4 - error TS2322: Type 'any' is not assignable to type 'never'.
113 normalizedTransaction[key] = NORMALIZERS[key](transaction[key]);
~~~~~~~~~~~~~~~~~~~~~~~~~~
Found 1 error.

builds fine with 3.5.3

[rickycodes]

I'm going to add a test case for this in a moment :)

[codecov-io]

Codecov Report

Merging #173 into develop will not change coverage.
The diff coverage is 100%.

@@ Coverage Diff @@## develop #173 +/- ##
======================================
Coverage 100% 100% ======================================
Files 23 23 Lines 1498 1499 +1 Branches 209 210 +1 ======================================
+ Hits 1498 1499 +1

Impacted Files	Coverage Δ
src/assets/CurrencyRateController.ts	100% <100%> (ø)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 32465fc...ca75312. Read the comment docs.

[brunobar79]

I think you also need to regenerate the shrinkwrap file

[brunobar79]

@estebanmino didn’t we fix this on the app side while passing the initial state to the engine?

[danjm]

Looks good to me but I'll wait for Bruno's comments to be addressed before approving.

[rickycodes]

@brunobar79 I removed the version pinning, which should negate the need to update the yarn.lock and npm-shrinkwrap.json (but the next person coming into this might experience the same build issues?)

[whymarrh]

LGTM