Rename setSelection to toggleSelection

owncloud · Feb 8, 2023 · 759548b · 759548b
1 parent ba25352
commit 759548b
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
@@ -18,7 +18,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>
@@ -175,7 +175,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)
@@ -207,7 +207,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()
   })