Prepare dropdown-value specs for BootstrapVue upgrade
This MR lays the groundwork for the BootstrapVue upgrade that will be done in this MR: !18913 (merged). Due to some changes in the BTooltip
and BPopover
components, some related tests started failing after the upgrade. Those tests need to be updated accordingly:
- Migrate specs to use
vue-test-utils
-
sync
needs to be set tofalse
-
attachToDocument
must betrue
This MR focuses on upgrading the tests located in spec/frontend/vue_shared/components/sidebar/labels_select/dropdown_value_spec.js
:
Why are these changes necessary?
- The reason to migrate the tests to
vue-test-utils
is that it allows us to set thesync
andattachToDocument
properties. - The reason to set
attachToDocument
totrue
is that the BTooltip and BPopover directives expect to be attached to a document object. - The reason to set
sync
tofalse
is that otherwise, specs for components that useBTooltip
orBPopover
will fail with the following error:
TypeError: Cannot read property 'sync' of null
Edited by 🤖 GitLab Bot 🤖