Skip to content

Commit

Permalink
Fix PR build
Browse files Browse the repository at this point in the history
  • Loading branch information
FlipWarthog committed Oct 12, 2023
1 parent 7e3f513 commit a4f7a98
Show file tree
Hide file tree
Showing 4 changed files with 9 additions and 3 deletions.
2 changes: 1 addition & 1 deletion components/lib/button/Button.spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -79,6 +79,6 @@ describe('Button.vue', () => {
}
});

expect(wrapper.html()).toBe(`<button class="p-button p-component" type="button" data-pc-name="button" data-pc-section="root" data-pd-ripple="true"><span class="ml-2 font-bold">Default PrimeVue Button</span></button>`);
expect(wrapper.html()).toBe(`<button class="p-button p-component" type="button" data-pc-name="button" data-pc-section="root"><span class="ml-2 font-bold">Default PrimeVue Button</span></button>`);
});
});
2 changes: 1 addition & 1 deletion components/lib/rating/Rating.spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ describe('Rating.vue', () => {
it('When star is clicked, onOptionClick method should triggered', async () => {
await wrapper.find('.p-rating-item').trigger('click');

expect(wrapper.find('.p-focus').exists()).toBe(true);
expect(wrapper.find('[data-p-focused="true"]').exists()).toBe(true);
});

it('When input focused, focusedOptionIndex value should changed', async () => {
Expand Down
5 changes: 4 additions & 1 deletion components/lib/splitbutton/SplitButton.vue
Original file line number Diff line number Diff line change
Expand Up @@ -80,7 +80,10 @@ export default {
},
methods: {
onDropdownButtonClick(event) {
event.preventDefault();
if (event) {
event.preventDefault();
}
this.$refs.menu.toggle({ currentTarget: this.$el, relatedTarget: this.$refs.button.$el });
this.isExpanded = this.$refs.menu.visible;
},
Expand Down
3 changes: 3 additions & 0 deletions vitest.config.js
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,9 @@ export default mergeConfig(
test: {
globals: true,
environment: 'jsdom',
onConsoleLog: (log, type) => {
if (type === 'stderr' && log.includes('Could not parse CSS stylesheet')) return false;
},
coverage: {
provider: 'istanbul',
reporter: ['text', 'json', 'html']
Expand Down

0 comments on commit a4f7a98

Please sign in to comment.