verdnatura/salix-front
verdnatura
/
salix-front
2
3
Fork 1
Code Issues Pull Requests 77 Releases Wiki Activity

#8647 fix vitest warnings #1467

Merged
jon merged 91 commits from 8647_fix_warnings into dev 2025-04-16 11:18:11 +00:00
Conversation 13 Commits 91 Files Changed 63 +3 -3
2 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit aefffa4a42 - Show all commits

2
src/components/CrudModel.vue
View File

@ -61,7 +61,7 @@ const $props = defineProps({
default: null,
},
saveFn: {
type: [String, Function],
type: Function,
alexm commented 2025-02-28 11:44:17 +00:00
Outdated
Review
Copy Link

string? saveFn

string? save**Fn**
jsegarra commented 2025-02-28 12:46:10 +00:00
Outdated
Review
Copy Link

En algun componente se estará usando string.
Yo solo he hecho modificar la prop para el warning, deberia parar a ver quien lo ha puesto mal?

En algun componente se estará usando string. Yo solo he hecho modificar la prop para el warning, deberia parar a ver quien lo ha puesto mal?
alexm commented 2025-03-07 07:25:11 +00:00
Outdated
Review
Copy Link

Si

Si
jsegarra commented 2025-03-10 11:30:49 +00:00
Outdated
Review
Copy Link

era un test que tenia comillas.
Fixed

era un test que tenia comillas. Fixed
default: null,
},
beforeSaveFn: {

4
src/components/__tests__/CrudModel.spec.js
View File

@ -21,7 +21,7 @@ describe('CrudModel', () => {
dataKey: 'crudModelKey',
model: 'crudModel',
url: 'crudModelUrl',
saveFn: '',
saveFn: vi.fn(),
alexm commented 2025-02-28 11:45:56 +00:00
Outdated
Review
Copy Link

Esto hay que repetirlo siempre?

Esto hay que repetirlo siempre?
jsegarra commented 2025-02-28 12:46:36 +00:00
Outdated
Review
Copy Link

No, solo cuando defines un createWrapper y le pasas un objeto global

No, solo cuando defines un createWrapper y le pasas un objeto global
jsegarra commented 2025-02-28 17:56:47 +00:00
Outdated
Review
Copy Link

Voy a darle una vuelta para hacer algo asi Object.assign(defaultOptions, options)

Voy a darle una vuelta para hacer algo asi `Object.assign(defaultOptions, options)`
},
});
alexm commented 2025-04-11 11:50:39 +00:00
Review
Copy Link

undefined?

undefined?
jon commented 2025-04-14 09:05:58 +00:00
Review
Copy Link

Esto es parte de lo que hizo @jsegarra

Esto es parte de lo que hizo @jsegarra
jsegarra commented 2025-04-14 09:17:29 +00:00
Review
Copy Link

no entiendo la pregunta.
Se modificó esta prop porque habia un warning y por tanto el test también

no entiendo la pregunta. Se modificó esta prop porque habia un warning y por tanto el test también
wrapper = wrapper.wrapper;
@ -223,7 +223,7 @@ describe('CrudModel', () => {
expect(vm.isLoading).toBe(false);
expect(vm.hasChanges).toBe(false);
await wrapper.setProps({ saveFn: '' });
await wrapper.setProps({ saveFn: null });
});
it("should use default url if there's not saveFn", async () => {