Skip to content

Vue unidriver - #197

Open
AlexDubok wants to merge 4 commits into
wix-incubator:masterfrom
AlexDubok:vue-unidriver
Open

Vue unidriver#197
AlexDubok wants to merge 4 commits into
wix-incubator:masterfrom
AlexDubok:vue-unidriver

Conversation

@AlexDubok

Copy link
Copy Markdown

Hi! First of all, thank you for maintaining this awesome project! 😍

As Vue developer at the moment, I want it to be part of this unidriver party 🎉
Please, accept it under your roof 🏡

On the implementation: I made @vue-test-utils's Wrapper as "native" element for unidriver because Vue unit tests heavily rely on it (emitting events from components, changing reactive data, etc.)

@GabiGrin

GabiGrin commented Jan 12, 2023

Copy link
Copy Markdown
Contributor

Very cool @AlexDubok, awesome to see ex-Wixers taking the Unidriver spirit with them :D
I suggest calling it just vue as in adapters/vue, assuming vue-test-utils is the standard way. And if the test suite passes, it's good to go! ✅

PS: I'm not longer employed by Wix but still a big believer in Unidriver. Have lost review/write access to this repo :/ @nitayneeman / @Tokyros can you help me regain some access here? or help pushing this PR forward?

Comment thread adapters/vue/package.json Outdated
Co-authored-by: Gabriel Grinberg <gabi.grinberg@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants