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

#7058 LeftMenu vitest #1153

Open
jsegarra wants to merge 18 commits from 7058_leftMenu_vitest into dev
pull from: 7058_leftMenu_vitest
merge into: verdnatura:dev
Conversation 0 Commits 18 Files Changed 2 +362 -64
jsegarra commented 2024-12-30 11:23:09 +00:00
Member
Copy Link

Hay 3 skip todavia

Hay 3 skip todavia
jsegarra added 5 commits 2024-12-31 13:03:00 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
12715adbdb test: refs #7058 improve methods
jsegarra added 1 commit 2024-12-31 13:13:45 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
4618ba87fa test: refs #7058 improve getRoutes
jsegarra added 7 commits 2025-01-03 15:28:41 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
336a5ccafc feat: refs #7058 skip failed it to review
jsegarra added 1 commit 2025-01-14 04:51:22 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
9d90d041ee Merge branch 'dev' into 7058_leftMenu_vitest
jsegarra added 2 commits 2025-01-20 22:08:29 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
45caf8ac2f test: refs #7058 remove unnecesary tests
jsegarra requested review from jorgep 2025-01-20 22:08:52 +00:00
jsegarra changed title from WIP: #7058 LeftMenu vitest to #7058 LeftMenu vitest 2025-01-20 22:08:55 +00:00
jorgep reviewed 2025-01-21 11:19:08 +00:00
src/components/LeftMenu.vue
@ -35,1 +35,4 @@
const filteredItems = computed(() => {
return filterItems();
});
function filterItems() {
jorgep commented 2025-01-21 11:19:08 +00:00
Member
Copy Link

Solo se usa 1 vez

Solo se usa 1 vez
jsegarra commented 2025-01-21 11:21:45 +00:00
Author
Member
Copy Link

Lo sé, era la manera mas simple de probar la lógica de esa función.
Esto no debería haber subido así porque no haría falta poner las llaves y el return

Lo sé, era la manera mas simple de probar la lógica de esa función. Esto no debería haber subido así porque no haría falta poner las llaves y el return
jorgep commented 2025-01-21 11:34:08 +00:00
Member
Copy Link

Si no hay manera de poder testearlo vale, pero si se puede mejor poner dentro de computed, si se vuelve muy complicado quizá sería mejor testearlo con test unitario con cypress

Si no hay manera de poder testearlo vale, pero si se puede mejor poner dentro de computed, si se vuelve muy complicado quizá sería mejor testearlo con test unitario con cypress
jorgep reviewed 2025-01-21 11:28:49 +00:00
src/components/__tests__/Leftmenu.spec.js
@ -73,0 +166,4 @@
expect(getMethodB).not.toHaveBeenCalled();
});
it('should call getMethodA when source is main', () => {
jorgep commented 2025-01-21 11:28:49 +00:00
Member
Copy Link

Poner otro nombre, está repetido.

Poner otro nombre, está repetido.
jsegarra commented 2025-01-21 11:48:34 +00:00
Author
Member
Copy Link

oh vaya

oh vaya
jorgep reviewed 2025-01-21 11:35:29 +00:00
src/components/__tests__/Leftmenu.spec.js Outdated
@ -93,0 +348,4 @@
vi.clearAllMocks();
});
it('should add menu items to parent if matches are found', () => {
jorgep commented 2025-01-21 11:35:29 +00:00
Member
Copy Link

Aquí habría que comprobar que el módulo ha sido añadido no?

Aquí habría que comprobar que el módulo ha sido añadido no?
jsegarra commented 2025-01-21 11:47:15 +00:00
Author
Member
Copy Link

Eso correspondería al test de useNavigationStore, no? No podríamos ni deberíamos testear funcionalidad de otros archivos

Eso correspondería al test de useNavigationStore, no? No podríamos ni deberíamos testear funcionalidad de otros archivos
jorgep commented 2025-01-21 13:32:57 +00:00
Member
Copy Link

En el título del archivo pone should add menu items. Como sabes que ha funcionado? solo sabes que se ha llamado a la función. Los 3 tests son exactamente iguales, con diferente título. Veo tu punto, si no crees que sea el lugar de testearlo, cambia el título del test por uno que compruebe realmente lo que hace esa función, llamar a otra función.

En el título del archivo pone should add menu items. Como sabes que ha funcionado? solo sabes que se ha llamado a la función. Los 3 tests son exactamente iguales, con diferente título. Veo tu punto, si no crees que sea el lugar de testearlo, cambia el título del test por uno que compruebe realmente lo que hace esa función, llamar a otra función.
jsegarra commented 2025-01-21 17:34:43 +00:00
Author
Member
Copy Link

Mmm...WTF.
Lo reviso, pero si 3 iguales

Mmm...WTF. Lo reviso, pero si 3 iguales
jorgep reviewed 2025-01-21 11:37:59 +00:00
src/components/__tests__/Leftmenu.spec.js
@ -93,0 +323,4 @@
});
});
describe('addChildren', () => {
jorgep commented 2025-01-21 11:37:59 +00:00
Member
Copy Link

En estos tests, solo veo que compruebes que la función haya sido llamada, no se debería comprobar si navigation ha cambiado?

En estos tests, solo veo que compruebes que la función haya sido llamada, no se debería comprobar si navigation ha cambiado?
jsegarra added 2 commits 2025-01-21 22:38:02 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
1ba24b46db test: refs #7058 requested changes
All checks were successful
gitea/salix-front/pipeline/pr-dev This commit looks good
Required
Details
This pull request doesn't have enough approvals yet. 0 of 1 approvals granted.
You are not authorized to merge this pull request.
Sign in to join this conversation.
Reviewers
No reviewers
jorgep
Labels
Clear labels   
bug

Something is not working

  
CR / Tests passed

CR / Tests passed

  
duplicate

This issue or pull request already exists

  
enhancement

New feature

  
invalid

Something is wrong

  
Missing Tests

Missing Tests

  
wontfix

This won't be fixed

No Label
bug
CR / Tests passed
duplicate
enhancement
invalid
Missing Tests
wontfix
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
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#1153
No description provided.
Delete Branch "7058_leftMenu_vitest"

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?