feat: #7088 created test for FetchedTags #1162
No reviewers
Labels
No Milestone
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: verdnatura/salix-front#1162
Loading…
Reference in New Issue
No description provided.
Delete Branch "7088-testFetchedTags"
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?
Si un test necesita comentarios es que no está bien planteado la descripcion del caso de uso y/o los expects
Por otra parte, si pones comentarios para los expects, que lo tengan todos no?
Y si pones comentarios a un valor que tenga sentido
Diría que vue/test-utils no lo usamos mas allá del createWrapper
@ -0,0 +4,4 @@
describe('tags computed property', () => {
it('returns an object with the correct keys and values', () => {
const wrapper = mount(FetchedTags, {
Es mínimo el detalle pero la variable wrapper no se usa en ningún sitio, no tiene mas sentido hacer vm?
si, no cambia nada usar wrapper en este caso. ya esta cambiado a vm
@ -0,0 +33,4 @@
value: 'value',
},
});
expect(vm.vm.tags).toEqual({});
Es mínimo el detalle pero la variable vm no se usa en ningún sitio, no tiene mas sentido hacer vm.tags?
solucionado
@ -0,0 +76,4 @@
},
});
expect(vm.vm.columnStyle).toEqual({});
Es mínimo el detalle pero la variable vm no se usa en ningún sitio, no tiene mas sentido hacer vm.columnStyle?
solucionado
La idea era hacer const {vm} = createWrapper