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

feat: refs #7055 created FilterItemForm test #1187

Merged
provira merged 8 commits from 7055-testFilterItemForm into dev 2025-01-14 06:03:51 +00:00
Conversation 0 Commits 8 Files Changed 1 +17 -13
1 changed files with 17 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 50008ce7f1 - Show all commits

30
src/components/__tests__/FilterItemForm.spec.js
View File

@ -1,5 +1,5 @@
import { createWrapper, axios } from 'app/test/vitest/helper';
import FilterItemForm from 'components/FilterItemForm.vue';
import FilterItemForm from 'src/components/FilterItemForm.vue';
jsegarra marked this conversation as resolved Outdated
jsegarra commented 2025-01-13 07:54:08 +00:00
Outdated
Review
Copy Link

src
así seguimos el standard de los test
Por ejemplo en la #1197 hemos puesto src/components

src así seguimos el standard de los test Por ejemplo en la #1197 hemos puesto src/components
provira commented 2025-01-13 11:26:10 +00:00
Outdated
Review
Copy Link

arreglado

arreglado
import { vi, beforeAll, describe, expect, it } from 'vitest';
describe('FilterItemForm', () => {
@ -19,7 +19,7 @@ describe('FilterItemForm', () => {
data: [
{
id: 999996,
name: 'Bolas de madera',
name: 'bolas de madera',
size: 2,
inkFk: null,
producerFk: null,
@ -29,23 +29,26 @@ describe('FilterItemForm', () => {
});
it('should filter data and populate tableRows for table display', async () => {
jsegarra commented 2025-01-13 08:40:41 +00:00
Review
Copy Link

Este it, debería comprobar el filtro que se le pasa a la petición de axios.
En el siguiente it, si que se comprueba estando vacío, y aquí que tiene valores no se comprueba? Tendría que haber expect en ambos casos

Este it, debería comprobar el filtro que se le pasa a la petición de axios. En el siguiente it, si que se comprueba estando vacío, y aquí que tiene valores no se comprueba? Tendría que haber expect en ambos casos
provira commented 2025-01-13 11:26:16 +00:00
Review
Copy Link

añadido

añadido
wrapper.setProps({
itemFilter: {
include: [
{ relation: 'producer', scope: { fields: ['name'] } },
{ relation: 'ink', scope: { fields: ['name'] } },
],
where: { name: { like: '%bolas de madera%' } },
},
itemFilterParams: { name: 'bolas de madera' },
});
vm.itemFilterParams.name = 'bolas de madera';
await vm.onSubmit();
const expectedFilter = {
include: [
{ relation: 'producer', scope: { fields: ['name'] } },
{ relation: 'ink', scope: { fields: ['name'] } },
],
where: {"name":{"like":"%bolas de madera%"}},
};
expect(axios.get).toHaveBeenCalledWith('Items/withName', {
jsegarra commented 2025-01-13 22:16:55 +00:00
Review
Copy Link

Es el mismo expect, pero creo que 2 ocurrencias se puede dar como bueno

Es el mismo expect, pero creo que 2 ocurrencias se puede dar como bueno
params: { filter: JSON.stringify(expectedFilter) },
});
expect(vm.tableRows).toEqual([
{
id: 999996,
name: 'Bolas de madera',
name: 'bolas de madera',
size: 2,
inkFk: null,
producerFk: null,
@ -54,6 +57,7 @@ describe('FilterItemForm', () => {
});
it('should handle an empty itemFilterParams correctly', async () => {
vm.itemFilterParams.name = null;
vm.itemFilterParams = {};
await vm.onSubmit();