From 3f93d68f9546dbf6afbd6377788e15cf342ee19e Mon Sep 17 00:00:00 2001
From: Phan An <me@phanan.net>
Date: Sun, 8 May 2022 20:18:27 +0200
Subject: [PATCH] feat(test): add PlaylistNameEditor component tests

---
 .../components/playlist/name-editor.spec.ts   | 29 ---------
 .../playlist/CreateNewPlaylistContextMenu.vue |  6 +-
 .../playlist/PlaylistContextMenu.vue          |  6 +-
 .../playlist/PlaylistNameEditor.spec.ts       | 63 +++++++++++++++++++
 .../playlist/PlaylistNameEditor.vue           |  5 +-
 .../playlist/PlaylistSidebarItem.vue          | 10 +--
 .../playlist/PlaylistSidebarList.vue          | 22 +++----
 .../SmartPlaylistCreateForm.vue               | 12 ++--
 .../smart-playlist/SmartPlaylistEditForm.vue  | 10 +--
 .../smart-playlist/SmartPlaylistRule.vue      |  1 -
 .../smart-playlist/SmartPlaylistRuleInput.vue |  2 +-
 .../js/components/ui/ContextMenuBase.vue      |  8 +--
 12 files changed, 103 insertions(+), 71 deletions(-)
 delete mode 100644 resources/assets/js/__tests__/components/playlist/name-editor.spec.ts
 create mode 100644 resources/assets/js/components/playlist/PlaylistNameEditor.spec.ts

diff --git a/resources/assets/js/__tests__/components/playlist/name-editor.spec.ts b/resources/assets/js/__tests__/components/playlist/name-editor.spec.ts
deleted file mode 100644
index 163c9ef0..00000000
--- a/resources/assets/js/__tests__/components/playlist/name-editor.spec.ts
+++ /dev/null
@@ -1,29 +0,0 @@
-import Component from '@/components/playlist/PlaylistNameEditor.vue'
-import factory from '@/__tests__/factory'
-import { playlistStore } from '@/stores'
-import { mock } from '@/__tests__/__helpers__'
-import { shallow } from '@/__tests__/adapter'
-
-describe('components/playlist/PlaylistNameEditor', () => {
-  let playlist: Playlist
-  beforeEach(() => {
-    playlist = factory<Playlist>('playlist', {
-      id: 99,
-      name: 'Foo'
-    })
-  })
-
-  afterEach(() => {
-    jest.resetModules()
-    jest.clearAllMocks()
-  })
-
-  it('updates a playlist', () => {
-    const updateStub = mock(playlistStore, 'update')
-    const wrapper = shallow(Component, {
-      propsData: { playlist }
-    })
-    wrapper.find('[type=text]').setValue('Bar').input().blur()
-    expect(updateStub).toHaveBeenCalledWith(expect.objectContaining({ id: 99, name: 'Bar' }))
-  })
-})
diff --git a/resources/assets/js/components/playlist/CreateNewPlaylistContextMenu.vue b/resources/assets/js/components/playlist/CreateNewPlaylistContextMenu.vue
index 37b936a0..66c18e8c 100644
--- a/resources/assets/js/components/playlist/CreateNewPlaylistContextMenu.vue
+++ b/resources/assets/js/components/playlist/CreateNewPlaylistContextMenu.vue
@@ -1,7 +1,7 @@
 <template>
-  <ContextMenuBase extra-class="playlist-menu" ref="base">
-    <li @click="createPlaylist" data-testid="playlist-context-menu-create-simple">New Playlist</li>
-    <li @click="createSmartPlaylist" data-testid="playlist-context-menu-create-smart">New Smart Playlist</li>
+  <ContextMenuBase ref="base" extra-class="playlist-menu">
+    <li data-testid="playlist-context-menu-create-simple" @click="createPlaylist">New Playlist</li>
+    <li data-testid="playlist-context-menu-create-smart" @click="createSmartPlaylist">New Smart Playlist</li>
   </ContextMenuBase>
 </template>
 
diff --git a/resources/assets/js/components/playlist/PlaylistContextMenu.vue b/resources/assets/js/components/playlist/PlaylistContextMenu.vue
index fa7968d1..822756eb 100644
--- a/resources/assets/js/components/playlist/PlaylistContextMenu.vue
+++ b/resources/assets/js/components/playlist/PlaylistContextMenu.vue
@@ -1,7 +1,7 @@
 <template>
-  <ContextMenuBase extra-class="playlist-item-menu" ref="base">
-    <li @click="editPlaylist" :data-testid="`playlist-context-menu-edit-${playlist.id}`">Edit</li>
-    <li @click="deletePlaylist" :data-testid="`playlist-context-menu-delete-${playlist.id}`">Delete</li>
+  <ContextMenuBase ref="base" extra-class="playlist-item-menu">
+    <li :data-testid="`playlist-context-menu-edit-${playlist.id}`" @click="editPlaylist">Edit</li>
+    <li :data-testid="`playlist-context-menu-delete-${playlist.id}`" @click="deletePlaylist">Delete</li>
   </ContextMenuBase>
 </template>
 
diff --git a/resources/assets/js/components/playlist/PlaylistNameEditor.spec.ts b/resources/assets/js/components/playlist/PlaylistNameEditor.spec.ts
new file mode 100644
index 00000000..6774306d
--- /dev/null
+++ b/resources/assets/js/components/playlist/PlaylistNameEditor.spec.ts
@@ -0,0 +1,63 @@
+import factory from '@/__tests__/factory'
+import PlaylistNameEditor from '@/components/playlist/PlaylistNameEditor.vue'
+import { beforeEach, expect, it } from 'vitest'
+import { mockHelper, render } from '@/__tests__/__helpers__'
+import { cleanup, fireEvent } from '@testing-library/vue'
+import { playlistStore } from '@/stores'
+
+beforeEach(() => {
+  mockHelper.restoreAllMocks()
+  cleanup()
+})
+
+const setup = () => {
+  const updateMock = mockHelper.mock(playlistStore, 'update')
+
+  const { getByTestId } = render(PlaylistNameEditor, {
+    props: {
+      playlist: factory<Playlist>('playlist', {
+        id: 99,
+        name: 'Foo'
+      })
+    }
+  })
+
+  return {
+    updateMock,
+    input: getByTestId<HTMLInputElement>('inline-playlist-name-input')
+  }
+}
+
+it('updates a playlist name on blur', async () => {
+  const { updateMock, input } = setup()
+
+  await fireEvent.update(input, 'Bar')
+  await fireEvent.blur(input)
+
+  expect(updateMock).toHaveBeenCalledWith(expect.objectContaining({
+    id: 99,
+    name: 'Bar'
+  }))
+})
+
+it('updates a playlist name on enter', async () => {
+  const { updateMock, input } = setup()
+
+  await fireEvent.update(input, 'Bar')
+  await fireEvent.keyUp(input, { key: 'Enter' })
+
+  expect(updateMock).toHaveBeenCalledWith(expect.objectContaining({
+    id: 99,
+    name: 'Bar'
+  }))
+})
+
+it('cancels updating on esc', async () => {
+  const { updateMock, input } = setup()
+
+  await fireEvent.update(input, 'Bar')
+  await fireEvent.keyUp(input, { key: 'Esc' })
+
+  expect(input.value).toBe('Foo')
+  expect(updateMock).not.toHaveBeenCalled()
+})
diff --git a/resources/assets/js/components/playlist/PlaylistNameEditor.vue b/resources/assets/js/components/playlist/PlaylistNameEditor.vue
index eb7cbcde..c67ca99a 100644
--- a/resources/assets/js/components/playlist/PlaylistNameEditor.vue
+++ b/resources/assets/js/components/playlist/PlaylistNameEditor.vue
@@ -51,5 +51,8 @@ const update = async () => {
   emit('updated', mutatedPlaylist)
 }
 
-const cancel = () => emit('cancelled')
+const cancel = () => {
+  mutatedPlaylist.name = playlist.value.name
+  emit('cancelled')
+}
 </script>
diff --git a/resources/assets/js/components/playlist/PlaylistSidebarItem.vue b/resources/assets/js/components/playlist/PlaylistSidebarItem.vue
index 1c41a9b2..b587cc72 100644
--- a/resources/assets/js/components/playlist/PlaylistSidebarItem.vue
+++ b/resources/assets/js/components/playlist/PlaylistSidebarItem.vue
@@ -1,26 +1,26 @@
 <template>
   <li
-    @dblclick.prevent="makeEditable"
-    :class="['playlist', type, editing ? 'editing' : '', playlist.is_smart ? 'smart' : '']">
+    :class="['playlist', type, editing ? 'editing' : '', playlist.is_smart ? 'smart' : '']"
+    @dblclick.prevent="makeEditable">
     <a
+      v-koel-droppable:[contentEditable]="handleDrop"
       :class="{ active }"
       :href="url"
       @contextmenu.prevent="openContextMenu"
-      v-koel-droppable:[contentEditable]="handleDrop"
     >{{ playlist.name }}</a>
 
     <NameEditor
+      v-if="nameEditable && editing"
       :playlist="playlist"
       @cancelled="cancelEditing"
       @updated="onPlaylistNameUpdated"
-      v-if="nameEditable && editing"
     />
 
     <ContextMenu
       v-if="hasContextMenu"
       v-show="showingContextMenu"
-      :playlist="playlist"
       ref="contextMenu"
+      :playlist="playlist"
       @edit="makeEditable"
     />
   </li>
diff --git a/resources/assets/js/components/playlist/PlaylistSidebarList.vue b/resources/assets/js/components/playlist/PlaylistSidebarList.vue
index c9552877..0de89215 100644
--- a/resources/assets/js/components/playlist/PlaylistSidebarList.vue
+++ b/resources/assets/js/components/playlist/PlaylistSidebarList.vue
@@ -3,34 +3,34 @@
     <h1>Playlists
       <i
         :class="{ creating }"
-        @click.prevent="toggleContextMenu"
         class="fa fa-plus-circle control create"
+        data-testid="sidebar-create-playlist-btn"
         role="button"
         title="Create a new playlist"
-        data-testid="sidebar-create-playlist-btn"
+        @click.prevent="toggleContextMenu"
       ></i>
     </h1>
 
     <form v-if="creating" @submit.prevent="createPlaylist" name="create-simple-playlist-form" class="create">
       <input
-        @keyup.esc.prevent="creating = false"
-        placeholder="↵ to save"
+        v-model="newName"
+        v-koel-focus
         name="name"
+        placeholder="↵ to save"
         required
         type="text"
-        v-koel-focus
-        v-model="newName"
+        @keyup.esc.prevent="creating = false"
       >
     </form>
 
     <ul>
-      <PlaylistItem type="favorites" :playlist="{ name: 'Favorites', songs: favorites }"/>
-      <PlaylistItem type="recently-played" :playlist="{ name: 'Recently Played', songs: [] }"/>
+      <PlaylistItem :playlist="{ name: 'Favorites', songs: favorites }" type="favorites"/>
+      <PlaylistItem :playlist="{ name: 'Recently Played', songs: [] }" type="recently-played"/>
       <PlaylistItem
-        :playlist="playlist"
-        :key="playlist.id"
-        type="playlist"
         v-for="playlist in playlists"
+        :key="playlist.id"
+        :playlist="playlist"
+        type="playlist"
       />
     </ul>
 
diff --git a/resources/assets/js/components/playlist/smart-playlist/SmartPlaylistCreateForm.vue b/resources/assets/js/components/playlist/smart-playlist/SmartPlaylistCreateForm.vue
index 081bfb7a..c2f38e5a 100644
--- a/resources/assets/js/components/playlist/smart-playlist/SmartPlaylistCreateForm.vue
+++ b/resources/assets/js/components/playlist/smart-playlist/SmartPlaylistCreateForm.vue
@@ -2,7 +2,7 @@
   <FormBase>
     <div @keydown.esc="maybeClose">
       <SoundBar v-if="loading"/>
-      <form @submit.prevent="submit" v-else data-testid="create-smart-playlist-form">
+      <form v-else data-testid="create-smart-playlist-form" @submit.prevent="submit">
         <header>
           <h1>New Smart Playlist</h1>
         </header>
@@ -10,18 +10,18 @@
         <div>
           <div class="form-row">
             <label>Name</label>
-            <input type="text" v-model="name" name="name" v-koel-focus required>
+            <input v-model="name" v-koel-focus name="name" required type="text">
           </div>
 
           <div class="form-row rules">
             <RuleGroup
+              v-for="(group, index) in collectedRuleGroups"
+              :key="group.id"
               :group="group"
               :isFirstGroup="index === 0"
-              :key="group.id"
               @input="onGroupChanged"
-              v-for="(group, index) in collectedRuleGroups"
             />
-            <Btn @click.prevent="addGroup" class="btn-add-group" green small uppercase>
+            <Btn class="btn-add-group" green small uppercase @click.prevent="addGroup">
               <i class="fa fa-plus"></i> Group
             </Btn>
           </div>
@@ -29,7 +29,7 @@
 
         <footer>
           <Btn type="submit">Save</Btn>
-          <Btn class="btn-cancel" @click.prevent="maybeClose" white>Cancel</Btn>
+          <Btn class="btn-cancel" white @click.prevent="maybeClose">Cancel</Btn>
         </footer>
       </form>
     </div>
diff --git a/resources/assets/js/components/playlist/smart-playlist/SmartPlaylistEditForm.vue b/resources/assets/js/components/playlist/smart-playlist/SmartPlaylistEditForm.vue
index d8b3c3e2..4df8f39c 100644
--- a/resources/assets/js/components/playlist/smart-playlist/SmartPlaylistEditForm.vue
+++ b/resources/assets/js/components/playlist/smart-playlist/SmartPlaylistEditForm.vue
@@ -2,7 +2,7 @@
   <FormBase>
     <div @keydown.esc="maybeClose">
       <SoundBar v-if="loading"/>
-      <form @submit.prevent="submit" v-else data-testid="edit-smart-playlist-form">
+      <form v-else data-testid="edit-smart-playlist-form" @submit.prevent="submit">
         <header>
           <h1>Edit Smart Playlist</h1>
         </header>
@@ -10,18 +10,18 @@
         <div>
           <div class="form-row">
             <label>Name</label>
-            <input type="text" v-model="mutatedPlaylist.name" name="name" v-koel-focus required>
+            <input v-model="mutatedPlaylist.name" v-koel-focus name="name" required type="text">
           </div>
 
           <div class="form-row rules">
             <RuleGroup
               v-for="(group, index) in mutatedPlaylist.rules"
-              :isFirstGroup="index === 0"
               :key="group.id"
               :group="group"
+              :isFirstGroup="index === 0"
               @input="onGroupChanged"
             />
-            <Btn @click.prevent="addGroup" class="btn-add-group" green small uppercase>
+            <Btn class="btn-add-group" green small uppercase @click.prevent="addGroup">
               <i class="fa fa-plus"></i> Group
             </Btn>
           </div>
@@ -29,7 +29,7 @@
 
         <footer>
           <Btn type="submit">Save</Btn>
-          <Btn white class="btn-cancel" @click.prevent="maybeClose">Cancel</Btn>
+          <Btn class="btn-cancel" white @click.prevent="maybeClose">Cancel</Btn>
         </footer>
       </form>
     </div>
diff --git a/resources/assets/js/components/playlist/smart-playlist/SmartPlaylistRule.vue b/resources/assets/js/components/playlist/smart-playlist/SmartPlaylistRule.vue
index eac989d4..73cb653b 100644
--- a/resources/assets/js/components/playlist/smart-playlist/SmartPlaylistRule.vue
+++ b/resources/assets/js/components/playlist/smart-playlist/SmartPlaylistRule.vue
@@ -40,7 +40,6 @@ const mutatedRule = Object.assign({}, rule.value)
 
 const selectedModel = ref<SmartPlaylistModel>()
 const selectedOperator = ref<SmartPlaylistOperator>()
-const inputValues = ref([])
 
 const model = models.find(m => m.name === mutatedRule.model.name)
 
diff --git a/resources/assets/js/components/playlist/smart-playlist/SmartPlaylistRuleInput.vue b/resources/assets/js/components/playlist/smart-playlist/SmartPlaylistRuleInput.vue
index 7744b264..285b23ba 100644
--- a/resources/assets/js/components/playlist/smart-playlist/SmartPlaylistRuleInput.vue
+++ b/resources/assets/js/components/playlist/smart-playlist/SmartPlaylistRuleInput.vue
@@ -1,5 +1,5 @@
 <template>
-  <input :type="type" v-model="value" name="value[]" required>
+  <input v-model="value" :type="type" name="value[]" required>
 </template>
 
 <script lang="ts" setup>
diff --git a/resources/assets/js/components/ui/ContextMenuBase.vue b/resources/assets/js/components/ui/ContextMenuBase.vue
index 5dc80182..32c006c3 100644
--- a/resources/assets/js/components/ui/ContextMenuBase.vue
+++ b/resources/assets/js/components/ui/ContextMenuBase.vue
@@ -92,12 +92,8 @@ const close = () => {
   shown.value = false
 }
 
-eventBus.on('CONTEXT_MENU_OPENED', target => {
-  // ensure there's only one context menu at any time
-  if (target !== el) {
-    close()
-  }
-})
+// ensure there's only one context menu at any time
+eventBus.on('CONTEXT_MENU_OPENED', target => target === el || close())
 
 defineExpose({ open, close, shown })
 </script>