Skip to content

Chore: vue 3 alert_settings_form unit tests

Marina Mosti requested to merge marina.mosti-451665-vue3-alert-settings-form-spec into master

What does this MR do and why?

Fixes the specs for alert_settings_form so that they pass on Vue 3.

The main issue was that the duplication of input event listeners through v-model and @input does not seem to work very well under compat due to the input/update:modelValue magic.

Part of: #451665 (closed)

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

How to verify

This component lives in http://127.0.0.1:3000/flightjs/Flight/-/settings/operations

  • To reach it navigate to a project, Settings => Monitor => Alerts
  • Then add a new integration to see the form

image

Edited by Marina Mosti

Merge request reports