From 6062556891d47612bcab07ff8451077ba9b50523 Mon Sep 17 00:00:00 2001
From: R0Wi <ro.windey@gmail.com>
Date: Tue, 9 Aug 2022 22:30:24 +0200
Subject: [PATCH] Add docs and tests for #2973

---
 package.json                                  |   1 +
 .../MultiselectTags/MultiselectTags.vue       |  23 +++-
 .../MultiselectTags/MultiselectTags.spec.js   | 130 ++++++++++++++++++
 3 files changed, 153 insertions(+), 1 deletion(-)
 create mode 100644 tests/unit/components/MultiselectTags/MultiselectTags.spec.js

diff --git a/package.json b/package.json
index cd1deaf709..9818ccb382 100644
--- a/package.json
+++ b/package.json
@@ -20,6 +20,7 @@
 		"lint": "eslint --ext .js,.vue src",
 		"lint:fix": "eslint --ext .js,.vue src --fix",
 		"test": "jest --verbose",
+		"test:coverage": "jest --verbose --coverage --no-cache",
 		"stylelint": "stylelint src/**/*.vue src/**/*.scss src/**/*.css",
 		"stylelint:fix": "stylelint src/**/*.vue src/**/*.scss src/**/*.css --fix",
 		"styleguide": "vue-styleguidist server",
diff --git a/src/components/MultiselectTags/MultiselectTags.vue b/src/components/MultiselectTags/MultiselectTags.vue
index ad7759618a..684ae42a77 100644
--- a/src/components/MultiselectTags/MultiselectTags.vue
+++ b/src/components/MultiselectTags/MultiselectTags.vue
@@ -61,6 +61,27 @@ export default {
 }
 </script>
 ```
+
+## Custom filter
+Because of compatibility reasons only 5 tag entries are shown by default. If you want to show all available tags set the `filter` function-prop to `null`:
+```vue
+<template>
+	<div class="wrapper">
+		<MultiselectTags v-model="value" :filter="null" />
+		{{ value }}
+	</div>
+</template>
+```
+
+It's also possible to apply any custom filter logic by setting the `filter` function-prop to any custom function receiving the tag element and the index:
+```vue
+<template>
+	<div class="wrapper">
+		<MultiselectTags v-model="value" :filter="(element, index) => element.id > 2 && element.displayName !== '' && element.canAssign && element.userAssignable && element.userVisible" />
+		{{ value }}
+	</div>
+</template>
+```
 </docs>
 
 <template>
@@ -114,7 +135,7 @@ export default {
 		},
 		filter: {
 			type: Function,
-			default: (element, index) => index < 5,
+			default: (_element, index) => index < 5,
 		},
 	},
 	data() {
diff --git a/tests/unit/components/MultiselectTags/MultiselectTags.spec.js b/tests/unit/components/MultiselectTags/MultiselectTags.spec.js
new file mode 100644
index 0000000000..12e1848c11
--- /dev/null
+++ b/tests/unit/components/MultiselectTags/MultiselectTags.spec.js
@@ -0,0 +1,130 @@
+/**
+ * @copyright Copyright (c) 2022 Robin Windey <ro.windey@gmail.com>
+ *
+ * @author Robin Windey <ro.windey@gmail.com>
+ *
+ * @license GNU AGPL version 3 or any later version
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU Affero General Public License as
+ * published by the Free Software Foundation, either version 3 of the
+ * License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU Affero General Public License for more details.
+ *
+ * You should have received a copy of the GNU Affero General Public License
+ * along with this program. If not, see <http://www.gnu.org/licenses/>.
+ *
+ */
+
+import { mount } from '@vue/test-utils'
+import { searchTags } from '../../../../src/components/MultiselectTags/api.js'
+import MultiselectTags from '../../../../src/components/MultiselectTags/MultiselectTags.vue'
+
+jest.mock('../../../../src/components/MultiselectTags/api.js')
+
+afterEach(() => {
+	jest.clearAllMocks()
+})
+
+describe('MultiselectTags.vue', () => {
+	'use strict'
+
+	it('Test that by default only 5 tags are shown', async () => {
+		const numberOfTags = 42
+		const mockedTags = []
+		for (let i = 0; i < numberOfTags; i++) {
+			mockedTags.push({
+				id: i,
+				displayName: 'tag ' + i,
+				canAssign: true,
+				userAssignable: true,
+				userVisible: true,
+			})
+		}
+
+		searchTags.mockResolvedValueOnce(mockedTags)
+
+		const wrapper = mount(MultiselectTags, {
+			propsData: {
+				label: 'name',
+				value: [],
+			},
+		})
+
+		// Wait for data to be loaded
+		await new Promise(process.nextTick)
+
+		const options = wrapper.findAll('li.multiselect__element .multiselect__option')
+
+		expect(searchTags).toHaveBeenCalledTimes(1)
+		expect(options.length).toBe(5)
+	})
+
+	it('Test that all available tags are shown if filter is null', async () => {
+		const numberOfTags = 42
+		const mockedTags = []
+		for (let i = 0; i < numberOfTags; i++) {
+			mockedTags.push({
+				id: i,
+				displayName: 'tag ' + i,
+				canAssign: true,
+				userAssignable: true,
+				userVisible: true,
+			})
+		}
+
+		searchTags.mockResolvedValueOnce(mockedTags)
+
+		const wrapper = mount(MultiselectTags, {
+			propsData: {
+				label: 'name',
+				value: [],
+				filter: null,
+			},
+		})
+
+		// Wait for data to be loaded
+		await new Promise(process.nextTick)
+
+		const options = wrapper.findAll('li.multiselect__element .multiselect__option')
+
+		expect(searchTags).toHaveBeenCalledTimes(1)
+		expect(options.length).toBe(numberOfTags)
+	})
+
+	it('Test that filter is applied correctly', async () => {
+		const numberOfTags = 42
+		const mockedTags = []
+		for (let i = 0; i < numberOfTags; i++) {
+			mockedTags.push({
+				id: i,
+				displayName: 'tag ' + i,
+				canAssign: true,
+				userAssignable: true,
+				userVisible: true,
+			})
+		}
+
+		searchTags.mockResolvedValueOnce(mockedTags)
+
+		const wrapper = mount(MultiselectTags, {
+			propsData: {
+				label: 'name',
+				value: [],
+				filter: (element, index) => index >= 40 && element.id >= 40,
+			},
+		})
+
+		// Wait for data to be loaded
+		await new Promise(process.nextTick)
+
+		const options = wrapper.findAll('li.multiselect__element .multiselect__option')
+
+		expect(searchTags).toHaveBeenCalledTimes(1)
+		expect(options.length).toBe(2)
+	})
+})