Commit21d8872

committed

(test): ensure persisted data is loaded from storage

- this is pretty critical functionality I forgot to add a test for... thanks to code coverage for pointing this out to me!

1 parentc83135b commit21d8872Copy full SHA for 21d8872

File tree

+15

-2

lines changed

+15

-2

lines changed

Lines changed: 5 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -8,3 +8,8 @@ export const UserStore = types.model('UserStore', {`
`8`	`8`	`self.name=name`
`9`	`9`	`}`
`10`	`10`	`}))`
	`11`	`+`
	`12`	`+exportconstpersistedData={`
	`13`	`+name:'Persisted Name',`
	`14`	`+age:35`
	`15`	`+}`

Lines changed: 10 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -2,14 +2,14 @@`
`2`	`2`	`import{getSnapshot}from'mobx-state-tree'`
`3`	`3`
`4`	`4`	`import{persist}from'../src/index'`
`5`		`-import{UserStore}from'./fixtures'`
	`5`	`+import{UserStore,persistedData}from'./fixtures'`
`6`	`6`
`7`	`7`	`functiongetItem(key:string){`
`8`	`8`	`constitem=window.localStorage.getItem(key)`
`9`	`9`	`returnitem ?JSON.parse(item) :null// can only parse strings`
`10`	`10`	`}`
`11`	`11`
`12`		`-describe('basicpersist options',()=>{`
	`12`	`+describe('persist',()=>{`
`13`	`13`	`beforeEach(()=>window.localStorage.clear())`
`14`	`14`
`15`	`15`	`it('should persist nothing if no actions are used',async()=>{`
`@@ -27,6 +27,14 @@ describe('basic persist options', () => {`
`27`	`27`	`expect(getItem('user')).toStrictEqual(getSnapshot(user))`
`28`	`28`	`})`
`29`	`29`
	`30`	`+it('should load persisted data',async()=>{`
	`31`	`+window.localStorage.setItem('user',JSON.stringify(persistedData))`
	`32`	`+`
	`33`	`+constuser=UserStore.create()`
	`34`	`+awaitpersist('user',user)`
	`35`	`+expect(getSnapshot(user)).toStrictEqual(persistedData)`
	`36`	`+})`
	`37`	`+`
`30`	`38`	`it('shouldn\'t jsonify',async()=>{`
`31`	`39`	`constuser=UserStore.create()`
`32`	`40`	`awaitpersist('user',user,{`

Comments

(0)