Skip to content

Commit 53aad0d

Browse files
committed
(refactor/test): export fixtures as <name>F
- <name>F makes it more explicit that we're referencing a fixture in the code, similar to e.g. TypeScript I<name> for interfaces
1 parent 21d8872 commit 53aad0d

File tree

2 files changed

+11
-11
lines changed

2 files changed

+11
-11
lines changed

test/fixtures.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
import { types } from 'mobx-state-tree'
22

3-
export const UserStore = types.model('UserStore', {
3+
export const UserStoreF = types.model('UserStore', {
44
name: 'John Doe',
55
age: 32
66
}).actions((self) => ({
@@ -9,7 +9,7 @@ export const UserStore = types.model('UserStore', {
99
}
1010
}))
1111

12-
export const persistedData = {
12+
export const persistedDataF = {
1313
name: 'Persisted Name',
1414
age: 35
1515
}

test/index.spec.ts

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
import { getSnapshot } from 'mobx-state-tree'
33

44
import { persist } from '../src/index'
5-
import { UserStore, persistedData } from './fixtures'
5+
import { UserStoreF, persistedDataF } from './fixtures'
66

77
function getItem(key: string) {
88
const item = window.localStorage.getItem(key)
@@ -13,30 +13,30 @@ describe('persist', () => {
1313
beforeEach(() => window.localStorage.clear())
1414

1515
it('should persist nothing if no actions are used', async () => {
16-
const user = UserStore.create()
16+
const user = UserStoreF.create()
1717
await persist('user', user)
1818

1919
expect(getItem('user')).toBe(null)
2020
})
2121

2222
it('should persist snapshot when action used', async () => {
23-
const user = UserStore.create()
23+
const user = UserStoreF.create()
2424
await persist('user', user)
2525

2626
user.changeName('Joe') // fire action to trigger onSnapshot
2727
expect(getItem('user')).toStrictEqual(getSnapshot(user))
2828
})
2929

3030
it('should load persisted data', async () => {
31-
window.localStorage.setItem('user', JSON.stringify(persistedData))
31+
window.localStorage.setItem('user', JSON.stringify(persistedDataF))
3232

33-
const user = UserStore.create()
33+
const user = UserStoreF.create()
3434
await persist('user', user)
35-
expect(getSnapshot(user)).toStrictEqual(persistedData)
35+
expect(getSnapshot(user)).toStrictEqual(persistedDataF)
3636
})
3737

3838
it('shouldn\'t jsonify', async () => {
39-
const user = UserStore.create()
39+
const user = UserStoreF.create()
4040
await persist('user', user, {
4141
jsonify: false
4242
})
@@ -47,7 +47,7 @@ describe('persist', () => {
4747
})
4848

4949
it('should whitelist', async () => {
50-
const user = UserStore.create()
50+
const user = UserStoreF.create()
5151
await persist('user', user, {
5252
whitelist: ['name']
5353
})
@@ -59,7 +59,7 @@ describe('persist', () => {
5959
})
6060

6161
it('should blacklist', async () => {
62-
const user = UserStore.create()
62+
const user = UserStoreF.create()
6363
await persist('user', user, {
6464
blacklist: ['age']
6565
})

0 commit comments

Comments
 (0)