search

ckeditor / ckeditor4-vue

Official CKEditor 4 Vue component
Other
77 stars 17 forks source link

Update vue-test-utils usage #86

Closed sculpt0r closed 2 years ago

sculpt0r commented 3 years ago

Are you reporting a feature request or a bug?

Task

Provide detailed reproduction steps (if any)

  1. Run npm run test

Expected result

Test are passing or failing without deprecate warnings.

Actual result

ERROR: '[vue-test-utils]: overwriting methods via the `methods` property is deprecated and will be removed in the next major version. There is no clear migration path for the `methods` property - Vue does not support arbitrarily replacement of methods, nor should VTU. To stub a complex method extract it from the component and test it in isolation. Otherwise, the suggestion is to rethink those tests.'
ERROR: '[vue-test-utils]: options.attachToDocument is deprecated in favor of options.attachTo and will be removed in a future release is deprecated and will be removed in the next major version.'

Other details

sculpt0r commented 3 years ago

Until this dependency starts making any troubles - we decide to not working on it.

sculpt0r commented 2 years ago

Fixing https://github.com/ckeditor/ckeditor4-vue/issues/124 - may be a good moment to try to bump this dependency - at least above the beta release.

sculpt0r commented 2 years ago

Closed in #128 - vtu bumped above the beta version.