Skip to content

[6.x] Switch from browser tests to storybook tests#13313

Merged
jasonvarga merged 16 commits into6.xfrom
combobox-browser-tests
Feb 5, 2026
Merged

[6.x] Switch from browser tests to storybook tests#13313
jasonvarga merged 16 commits into6.xfrom
combobox-browser-tests

Conversation

@duncanmcclean
Copy link
Member

Now that we've got browser tests setup with Vitest (thanks to #13121), I thought it might be a good idea to refactor our existing Combobox JS tests into browser tests so we don't need to do so much mocking around browser stuff.

@duncanmcclean
Copy link
Member Author

@jasonvarga I think you were working on this a few weeks ago - are you able to push up any pending commits?

# Conflicts:
#	package-lock.json
#	package.json
#	resources/js/components/ui/Combobox/Combobox.vue
@duncanmcclean duncanmcclean marked this pull request as draft January 28, 2026 09:34
@jasonvarga jasonvarga changed the base branch from master to 6.x January 28, 2026 16:26
@jasonvarga jasonvarga changed the title [6.x] Replace Combobox JS tests with Browser tests [6.x] Switch from browser tests to storybook tests Feb 5, 2026
@jasonvarga jasonvarga marked this pull request as ready for review February 5, 2026 18:32
@jasonvarga
Copy link
Member

I've removed the combobox test and updated the scope of this PR. It just changes the testing harness. The combobox tests will be introduced in the combobox 2.0 pr coming soon.

@jasonvarga jasonvarga merged commit 134b952 into 6.x Feb 5, 2026
11 checks passed
@jasonvarga jasonvarga deleted the combobox-browser-tests branch February 5, 2026 19:33
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