fix(ui): importing playlist during tutorial will skip latest steps

Author: feugy

renderer/stores/tutorial.js

@@ -1,11 +1,11 @@
 'use strict'
 
+import { tick } from 'svelte'
 import { BehaviorSubject, Subject } from 'rxjs'
 import { location, replace } from 'svelte-spa-router'
 import { albums } from './albums'
-import { playlists } from './playlists'
 import { tracks } from './track-queue'
-import { tick } from 'svelte'
+import { lastInvokation } from '../utils'
 
 const steps = [
   {
@@ -40,7 +40,8 @@ const steps = [
     annotation: {
       left: '15%'
     },
-    listen: () => awaitsEvent(playlists, playlists => playlists.length > 0)
+    listen: () =>
+      awaitsEvent(lastInvokation, ({ invoked }) => invoked === 'playlists.save')
   },
   {
     anchorId: 'to-playlists',

renderer/stores/tutorial.test.js

@@ -101,13 +101,7 @@ describe('tutorial store', () => {
   })
 
   it('goes to sixth step when creating a playlist', async () => {
-    mockInvoke.mockResolvedValueOnce({
-      results: [{}],
-      total: 1,
-      size: 1,
-      from: 0
-    })
-    playlists.list()
+    playlists.save({ name: 'my playlist', trackIds: [123456] })
     await sleep()
     expect(get(current)).toEqual(
       expect.objectContaining({

renderer/utils/invoke.js

@@ -2,7 +2,13 @@
 
 // note: do not destructure, as it makes it harder to mock in Storybook
 const electron = require('electron')
+import { BehaviorSubject } from 'rxjs'
+
+const lastInvokation$ = new BehaviorSubject()
+
+export const lastInvokation = lastInvokation$.asObservable()
 
 export async function invoke(invoked, ...args) {
+  lastInvokation$.next({ invoked, args })
   return electron.ipcRenderer.invoke('remote', ...invoked.split('.'), ...args)
 }

renderer/utils/invoke.test.js

@@ -1,8 +1,10 @@
 'use strict'
 
+import { get } from 'svelte/store'
 import faker from 'faker'
-import { invoke } from './invoke'
+import { invoke, lastInvokation } from './invoke'
 import electron from 'electron'
+import { sleep } from '../tests'
 
 jest.mock('electron', () => ({
   ipcRenderer: {
@@ -29,5 +31,10 @@ describe('invoke', () => {
       arg2
     )
     expect(electron.ipcRenderer.invoke).toHaveBeenCalledTimes(1)
+    await sleep()
+    expect(get(lastInvokation)).toEqual({
+      invoked: 'dialog.showOpenDialog',
+      args: [arg1, arg2]
+    })
   })
 })