Rename setSelection to toggleSelection

owncloud · Feb 9, 2023 · 48f4d60 · 48f4d60
1 parent f9b6846
commit 48f4d60
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/packages/web-app-files/src/components/FilesList/ResourceTiles.vue b/packages/web-app-files/src/components/FilesList/ResourceTiles.vue
@@ -20,7 +20,7 @@
               size="large"
               class="oc-flex-inline oc-p-s"
               :model-value="isResourceSelected(resource)"
-              @click.stop.prevent="setSelection(resource)"
+              @click.stop.prevent="toggleSelection(resource)"
             />
           </template>
           <template #imageField>
@@ -177,7 +177,7 @@ export default defineComponent({
       return props.selectedIds.includes(resource.id)
     }
 
-    const setSelection = (resource) => {
+    const toggleSelection = (resource) => {
       const selectedIds = !isResourceSelected(resource)
         ? [...props.selectedIds, resource.id]
         : props.selectedIds.filter((id) => id !== resource.id)
@@ -209,7 +209,7 @@ export default defineComponent({
       showContextMenu,
       tileRefs,
       isResourceSelected,
-      setSelection,
+      toggleSelection,
       getResourceCheckboxLabel
     }
   },

diff --git a/packages/web-app-files/tests/unit/components/FilesList/ResourceTiles.spec.ts b/packages/web-app-files/tests/unit/components/FilesList/ResourceTiles.spec.ts
@@ -39,7 +39,7 @@ describe('ResourceTiles component', () => {
 
   it('emits update:selectedIds event on resource selection and sets the selection', () => {
     const wrapper = getWrapper({ data: spacesResources, selectedIds: [spacesResources[0].id] })
-    wrapper.vm.setSelection(spacesResources[0])
+    wrapper.vm.toggleSelection(spacesResources[0])
     expect(wrapper.find('oc-tile').attributes()['is-resource-selected']).toEqual('true')
     expect(wrapper.emitted('update:selectedIds')).toBeTruthy()
   })