test: refs #7100 added test to vnNotes component #1121

Merged
jtubau merged 10 commits from 7100-createVnNotesSpec into dev 2025-01-08 06:05:02 +00:00
Member
No description provided.
jtubau added 1 commit 2024-12-20 09:38:29 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
0bd48d476b
test: refs #7100 added test to vnNotes component
jtubau requested review from jsegarra 2024-12-20 09:38:41 +00:00
jsegarra requested changes 2024-12-20 09:58:50 +00:00
Dismissed
@ -0,0 +16,4 @@
filter: { order: 'created DESC' },
body: { name: 'Tony', lastName: 'Stark' },
addNote: false,
selectType: true,
Member

Todos los test asument que selectType es true, que pasa cuando le damos valor false?
Falla

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 = '';
Member

Este caso no se dará, está bien incluirlo, pero para ser completo el test deberíamos probar con null

Este caso no se dará, está bien incluirlo, pero para ser completo el test deberíamos probar con null
jsegarra marked this conversation as resolved
jtubau added 1 commit 2024-12-20 12:20:29 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
69451862bf
test: refs #7100 modified test and added more cases
jtubau requested review from jsegarra 2024-12-20 12:20:47 +00:00
jsegarra requested changes 2024-12-20 12:23:24 +00:00
Dismissed
@ -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 });
Member

porque usas await?

porque usas await?
Author
Member

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.

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.
Member

Estoy mirando la documentación y devuelve una promesa, https://test-utils.vuejs.org/api/#setProps
Cuando puse el comentario no vi el await

Estoy mirando la documentación y devuelve una promesa, https://test-utils.vuejs.org/api/#setProps Cuando puse el comentario no vi el await
jsegarra marked this conversation as resolved
jtubau added 1 commit 2024-12-23 08:30:09 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
07afbc82ef
refactor: refs #7100 delete unnecesary set prop
jsegarra approved these changes 2024-12-23 08:51:02 +00:00
Dismissed
jtubau added 1 commit 2024-12-24 11:51:51 +00:00
jtubau dismissed jsegarra’s review 2024-12-24 11:51:51 +00:00
Reason:

New commits pushed, approval review dismissed automatically according to repository settings

jtubau requested review from jsegarra 2024-12-24 11:52:24 +00:00
jtubau added 1 commit 2024-12-26 07:54:41 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
788d7b09cf
Merge branch 'dev' into 7100-createVnNotesSpec
jsegarra requested changes 2024-12-26 08:55:37 +00:00
Dismissed
@ -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 }};
Member

Podríamos sacar esta linea que esta 3 veces para que esté definida solo 1 vez?
Tengo varias ideas, a ver si coincidimos

Podríamos sacar esta linea que esta 3 veces para que esté definida solo 1 vez? Tengo varias ideas, a ver si coincidimos
jtubau marked this conversation as resolved
jtubau added 1 commit 2024-12-30 06:40:35 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
771ecf1cc6
refactor: refs #7100 refactorized with methods
jtubau added 1 commit 2024-12-30 06:41:09 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
0a2f1fb073
Merge branch 'dev' into 7100-createVnNotesSpec
jtubau requested review from jsegarra 2024-12-30 06:41:21 +00:00
jsegarra reviewed 2024-12-30 08:21:50 +00:00
@ -0,0 +33,4 @@
});
beforeEach(() => {
postMock = vi.spyOn(axios, 'post').mockResolvedValue({ data: {name: 'Tony', lastName: 'Stark', text: 'Test Note', observationTypeFk: 1} });
Member

Podríamos acortar la linea llevando todo el valor de data a una const mockData, Así es mas fácil de visualizar.

Podríamos acortar la linea llevando todo el valor de data a una const mockData, Así es mas fácil de visualizar.
jsegarra approved these changes 2024-12-30 08:21:52 +00:00
Dismissed
jtubau added 1 commit 2025-01-02 14:13:46 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
23afe3276c
refactor: refs #7100 added const mockData
jtubau dismissed jsegarra’s review 2025-01-02 14:13:46 +00:00
Reason:

New commits pushed, approval review dismissed automatically according to repository settings

jtubau added 1 commit 2025-01-02 14:14:01 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
bf23895c03
Merge branch 'dev' into 7100-createVnNotesSpec
jtubau requested review from jsegarra 2025-01-02 14:14:33 +00:00
jsegarra reviewed 2025-01-03 16:03:03 +00:00
jsegarra left a comment
Member

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

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
jsegarra approved these changes 2025-01-03 16:03:17 +00:00
jtubau added 1 commit 2025-01-08 06:03:39 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
79c7d66fe6
Merge branch 'dev' into 7100-createVnNotesSpec
jtubau merged commit 2c8b120ce4 into dev 2025-01-08 06:05:02 +00:00
jtubau deleted branch 7100-createVnNotesSpec 2025-01-08 06:05:02 +00:00
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: verdnatura/salix-front#1121
No description provided.