#7050 testCrudModel #1077
No reviewers
Labels
No Milestone
No Assignees
3 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: verdnatura/salix-front#1077
Loading…
Reference in New Issue
No description provided.
Delete Branch "7050-testCrudModel"
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?
WIP: 7050-testCrudModelto WIP: #7050 testCrudModelWIP: #7050 testCrudModelto #7050 testCrudModel@ -275,2 +273,2 @@
if (obj.length > 0) return false;
if (Array.isArray(obj)) return !obj.length;
return Object.keys(obj).length === 0 ;
return !Object.keys(obj).length;
@ -120,0 +137,4 @@
expect(result).toBe(true);
});
Falta comprobar si es array vacío o lleno
estaban hechos pero los puse dentro del mismo it para comprobar el objeto, los he separado para mayor claridad.
@ -120,0 +187,4 @@
expect(vm.watchChanges).toBeNull();
});
it('should set originalData and formatData with data and generate watchChanges', async () => {
Aquí deberias cambiar un dato o varios de formatData y ver al llamar a la fn reset esta vuelve a tener el valor original.
formData su valor al inicio del test es un array vacío, por eso al comprobarlo, lo hago contra el argumento que se le pasa a la función, ya que ese es el valor que deben de tener tanto originalData como formData.
@ -120,0 +209,4 @@
age: 42,
}];
it('should call saveFn if exists', async () => {
Porque vuelves a crear una instancia de crudmodel? No puedes usar la que ya tenías? Algo tipo vm.$props.saveFn = ....? o pasarlo mockeado desde el principio...
@ -120,0 +242,4 @@
expect(localVm.vm.hasChanges).toBe(false);
});
it('should not call saveFn if not exists', async () => {
El nombre del test no me acaba de convencer... el test lo veo bien. Algo tipo should use the default url if there's not saveFn. Algo así.
New commits pushed, approval review dismissed automatically according to repository settings