feat: #7065 created unit tests for UserPanel #1260

Merged
provira merged 15 commits from 7065-testUserPanel into dev 2025-02-04 06:03:50 +00:00
Member
No description provided.
provira added 1 commit 2025-01-21 11:57:09 +00:00
provira added 1 commit 2025-01-21 11:57:14 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
3340a16541
Merge branch 'dev' into 7065-testUserPanel
provira requested review from jsegarra 2025-01-21 11:57:17 +00:00
provira added 1 commit 2025-01-21 13:37:18 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
faf1d15283
Merge branch 'dev' into 7065-testUserPanel
jsegarra requested changes 2025-01-27 22:31:48 +00:00
Dismissed
@ -0,0 +14,4 @@
let axiosPostMock;
beforeAll(() => {
axiosPatchMock = vi.spyOn(axios, 'patch').mockResolvedValue({});
Member

Porque en WagonCreate no creas variable?
Dentro del expect usas axios.patch

Porque en WagonCreate no creas variable? Dentro del expect usas axios.patch
provira marked this conversation as resolved
@ -0,0 +15,4 @@
beforeAll(() => {
axiosPatchMock = vi.spyOn(axios, 'patch').mockResolvedValue({});
axiosPostMock = vi.spyOn(axios, 'post').mockResolvedValue({});
Member

estas haciendo mock o spy? Quizas el nombre de la variable no es la mejor

estas haciendo mock o spy? Quizas el nombre de la variable no es la mejor
provira marked this conversation as resolved
@ -0,0 +52,4 @@
expect(vm.darkMode).toBe(true);
});
it('should change user language and update preferences', async () => {
Member

revisa a ver si puedes repetir menos veces 'es'

revisa a ver si puedes repetir menos veces 'es'
provira marked this conversation as resolved
@ -0,0 +60,4 @@
expect(vm.locale).toBe('es');
});
it('should update user data', async () => {
Member

podemos hacer que name y boss no se repitan tanto?

podemos hacer que name y boss no se repitan tanto?
provira marked this conversation as resolved
provira added 1 commit 2025-01-28 06:59:21 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
8146aa5231
Merge branch 'dev' into 7065-testUserPanel
provira added 1 commit 2025-01-28 09:08:20 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
c8cd14f991
fix: refs #7065 made consts for repeated values
provira added 1 commit 2025-01-28 09:37:49 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
5f45434d89
Merge branch 'dev' into 7065-testUserPanel
provira added 1 commit 2025-01-28 10:43:50 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
fb43ff5b74
Merge branch 'dev' into 7065-testUserPanel
provira requested review from jsegarra 2025-01-28 10:44:16 +00:00
provira added 1 commit 2025-01-28 13:47:46 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
e644eec1fa
Merge branch 'dev' into 7065-testUserPanel
jsegarra requested changes 2025-01-28 22:28:41 +00:00
Dismissed
jsegarra left a comment
Member

Los tests no pasan, revisalo y me vuelves a pedir

Los tests no pasan, revisalo y me vuelves a pedir
@ -0,0 +4,4 @@
import axios from 'axios';
import { useState } from 'src/composables/useState';
vi.mock('axios');
Member

Tengo dudas de esta linea

Tengo dudas de esta linea
jsegarra added 1 commit 2025-01-28 22:29:01 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
f38e184ab9
Merge branch 'dev' into 7065-testUserPanel
provira added 1 commit 2025-01-29 10:58:26 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
c104240b1e
Merge branch 'dev' into 7065-testUserPanel
provira added 1 commit 2025-01-31 08:28:39 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
5526987472
Merge branch 'dev' into 7065-testUserPanel
provira added 1 commit 2025-02-03 10:31:09 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
b4b096ef9e
Merge branch 'dev' into 7065-testUserPanel
provira added 1 commit 2025-02-03 10:44:13 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
11cfca5d80
fix: refs #7065 removed
unnecessary code
provira requested review from jsegarra 2025-02-03 10:55:07 +00:00
jsegarra approved these changes 2025-02-03 22:28:11 +00:00
provira added 1 commit 2025-02-04 06:00:03 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
885437bb2e
Merge branch 'dev' into 7065-testUserPanel
provira added 1 commit 2025-02-04 06:02:42 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
c65ac9f4db
Merge branch 'dev' into 7065-testUserPanel
provira merged commit 108edae586 into dev 2025-02-04 06:03:50 +00:00
provira deleted branch 7065-testUserPanel 2025-02-04 06:03:50 +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#1260
No description provided.