test: refs #7100 added test to vnNotes component #1121
No reviewers
Labels
No Milestone
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: verdnatura/salix-front#1121
Loading…
Reference in New Issue
No description provided.
Delete Branch "7100-createVnNotesSpec"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
@ -0,0 +16,4 @@
filter: { order: 'created DESC' },
body: { name: 'Tony', lastName: 'Stark' },
addNote: false,
selectType: true,
Todos los test asument que selectType es true, que pasa cuando le damos valor false?
Falla
@ -0,0 +32,4 @@
describe('insert', () => {
it('should not call axios.post if newNote.text is empty', async () => {
vm.newNote.text = '';
Este caso no se dará, está bien incluirlo, pero para ser completo el test deberíamos probar con null
@ -0,0 +36,4 @@
it('should not call axios.post and vnPaginateRef.fetch if newNote.text is null', async () => {
vm.newNote.text = null;
vm.newNote.observationTypeFk = null;
await wrapper.setProps({ selectType: true });
porque usas await?
Para asegurar que selectType haya cambiado correctamente antes de continuar, ya que si no lo uso, se ejecuta vm.insert() antes de que se actualice el valor y el test falla, lo mismo con el vm.insert, ya que los expect se ejecutan antes de que sus correspondientes métodos espiados sean ejecutados.
Estoy mirando la documentación y devuelve una promesa, https://test-utils.vuejs.org/api/#setProps
Cuando puse el comentario no vi el await
New commits pushed, approval review dismissed automatically according to repository settings
@ -0,0 +70,4 @@
vm.newNote.observationTypeFk = null;
await wrapper.setProps({ selectType: false });
expectedBody = {...vm.$props.body, ...{ text: vm.newNote.text, observationTypeFk: vm.newNote.observationTypeFk }};
Podríamos sacar esta linea que esta 3 veces para que esté definida solo 1 vez?
Tengo varias ideas, a ver si coincidimos
@ -0,0 +33,4 @@
});
beforeEach(() => {
postMock = vi.spyOn(axios, 'post').mockResolvedValue({ data: {name: 'Tony', lastName: 'Stark', text: 'Test Note', observationTypeFk: 1} });
Podríamos acortar la linea llevando todo el valor de data a una const mockData, Así es mas fácil de visualizar.
New commits pushed, approval review dismissed automatically according to repository settings
En JS, se pueden dar valores por defecto a los parámetros de una función.
Te lo comento porque veo que setTestParams.text siempre es test Note, menos en 2 casos. A lo mejor moviéndolo al final no lo podemos quitar, pero en este caso lo veo bien porque le da legibilidad al test