@pequity/squirrel 10.0.0 → 10.0.2

package/squirrel/components/p-select-btn/p-select-btn.vue

@@ -4,7 +4,7 @@
       v-for="(item, index) in items"
       :key="String(item[itemValue])"
       v-tooltip="{ content: tooltipText(item) }"
-      :class="['inline-flex', { '*:px-0 *:py-0': noPadding, grow }]"
+      :class="['inline-flex', { '[&_*]:px-0 [&_*]:py-0': noPadding, grow }]"
     >
       <PBtn
         :size="size"

package/squirrel/components/p-select-list/p-select-list.spec.js

@@ -525,7 +525,7 @@ describe('PSelectList.vue', () => {
     const pSelectListCmp = wrapper.findComponent(PSelectList);
 
     expect(wrapper.vm.$data.selected).toEqual([]);
-    expect(console.warn).toHaveBeenCalledWith(
+    expect(console.warn).toHaveBeenCalledExactlyOnceWith(
       `[p-select-list] Some values in the modelValue are not in the items list.`
     );
     expect(pSelectListCmp.emitted()['update:modelValue'][0][0]).toEqual([]);

package/squirrel/components/p-table-sort/p-table-sort.spec.js

@@ -32,19 +32,29 @@ describe('PTableSort.vue', () => {
     let selected;
 
     await wrapper.setData({ selected: SORTING_TYPES.ASC });
-    selected = wrapper.find('div.text-sm.font-semibold.text-p-purple-60.text-primary');
-    expect(selected.text()).toBe('Sort ascending');
-    expect(wrapper.find('div.cursor-pointer.opacity-100').exists()).toBe(true);
+    // Use specific selector to target the inner div with text-sm class
+    selected = wrapper.findByText('Sort ascending', '.text-sm');
+    expect(selected.exists()).toBe(true);
+    expect(selected.classes()).toContain('text-primary');
+    expect(wrapper.find('.cursor-pointer.opacity-100').exists()).toBe(true);
 
     await wrapper.setData({ selected: SORTING_TYPES.DESC });
-    selected = wrapper.find('div.text-sm.font-semibold.text-p-purple-60.text-primary');
-    expect(selected.text()).toBe('Sort descending');
-    expect(wrapper.find('div.cursor-pointer.opacity-100').exists()).toBe(true);
+    selected = wrapper.findByText('Sort descending', '.text-sm');
+    expect(selected.exists()).toBe(true);
+    expect(selected.classes()).toContain('text-primary');
+    expect(wrapper.find('.cursor-pointer.opacity-100').exists()).toBe(true);
 
     await wrapper.setData({ selected: SORTING_TYPES.NO_SORTING });
-    selected = wrapper.find('div.text-sm.font-semibold.text-p-purple-60.text-primary');
-    expect(selected.exists()).toBe(false);
-    expect(wrapper.find('div.cursor-pointer.opacity-100').exists()).toBe(false);
+    // When NO_SORTING is set, no sort option should have text-primary class
+    const ascendingOption = wrapper.findByText('Sort ascending', '.text-sm');
+    const descendingOption = wrapper.findByText('Sort descending', '.text-sm');
+    expect(ascendingOption.classes()).not.toContain('text-primary');
+    expect(descendingOption.classes()).not.toContain('text-primary');
+    // Check that the Clear button has opacity-50 (not cursor-pointer and opacity-100)
+    const clearButton = wrapper.findByText('Clear');
+    expect(clearButton.classes()).toContain('opacity-50');
+    expect(clearButton.classes()).not.toContain('cursor-pointer');
+    expect(clearButton.classes()).not.toContain('opacity-100');
   });
 
   it(`updates the value bound with v-model when an option is clicked`, async () => {
@@ -60,19 +70,28 @@ describe('PTableSort.vue', () => {
 
     let selected;
 
-    await wrapper.findAll('div.cursor-pointer.justify-between')[0].trigger('click');
-    selected = wrapper.find('div.text-sm.font-semibold.text-p-purple-60.text-primary');
-    expect(selected.text()).toBe('Sort ascending');
-    expect(wrapper.find('div.cursor-pointer.opacity-100').exists()).toBe(true);
+    await wrapper.findAll('.cursor-pointer.justify-between')[0].trigger('click');
+    selected = wrapper.findByText('Sort ascending', '.text-sm');
+    expect(selected.exists()).toBe(true);
+    expect(selected.classes()).toContain('text-primary');
+    expect(wrapper.find('.cursor-pointer.opacity-100').exists()).toBe(true);
 
-    await wrapper.findAll('div.cursor-pointer.justify-between')[1].trigger('click');
-    selected = wrapper.find('div.text-sm.font-semibold.text-p-purple-60.text-primary');
-    expect(selected.text()).toBe('Sort descending');
-    expect(wrapper.find('div.cursor-pointer.opacity-100').exists()).toBe(true);
+    await wrapper.findAll('.cursor-pointer.justify-between')[1].trigger('click');
+    selected = wrapper.findByText('Sort descending', '.text-sm');
+    expect(selected.exists()).toBe(true);
+    expect(selected.classes()).toContain('text-primary');
+    expect(wrapper.find('.cursor-pointer.opacity-100').exists()).toBe(true);
 
-    await wrapper.find('div.cursor-pointer.opacity-100').trigger('click');
-    selected = wrapper.find('div.text-sm.font-semibold.text-p-purple-60.text-primary');
-    expect(selected.exists()).toBe(false);
-    expect(wrapper.find('div.cursor-pointer.opacity-100').exists()).toBe(false);
+    await wrapper.find('.cursor-pointer.opacity-100').trigger('click');
+    // After clearing, no sort option should have text-primary class
+    const ascendingOption = wrapper.findByText('Sort ascending', '.text-sm');
+    const descendingOption = wrapper.findByText('Sort descending', '.text-sm');
+    expect(ascendingOption.classes()).not.toContain('text-primary');
+    expect(descendingOption.classes()).not.toContain('text-primary');
+    // Check that the Clear button has opacity-50 (not cursor-pointer and opacity-100)
+    const clearButton = wrapper.findByText('Clear');
+    expect(clearButton.classes()).toContain('opacity-50');
+    expect(clearButton.classes()).not.toContain('cursor-pointer');
+    expect(clearButton.classes()).not.toContain('opacity-100');
   });
 });

package/squirrel/utils/listKeyboardNavigation.spec.js

@@ -198,7 +198,9 @@ describe('listKeyboardNavigation', () => {
     const navigationSvc1 = createTestSvc();
     const navigationSvc2 = createTestSvc();
 
-    expect(consoleWarnSpy).toHaveBeenCalledWith('There can only be one instance of list navigation active at a time.');
+    expect(consoleWarnSpy).toHaveBeenCalledExactlyOnceWith(
+      'There can only be one instance of list navigation active at a time.'
+    );
 
     navigationSvc1.destroy();
     navigationSvc2.destroy();

package/squirrel/utils/sanitization.spec.js

@@ -11,7 +11,7 @@ describe('sanitizeUrl', () => {
     const unsafeUrl = 'javascript:evil()';
 
     expect(sanitizeUrl(unsafeUrl)).toBe(`unsafe:${unsafeUrl}`);
-    expect(consoleMock).toHaveBeenCalledWith(
+    expect(consoleMock).toHaveBeenCalledExactlyOnceWith(
       expect.stringContaining(`WARNING: sanitizing unsafe URL value ${unsafeUrl}`)
     );
   });