Armando Rivera
Tanelo Mondaca Silva
No es necesario, ya que el tipado <boolean> indica que la respuesta va a ser un booleano Con eso solo estás colocando código demás
Armando Rivera
@taknello <boolean> indica que la respuesta va a ser un booleano, pero la respuesta no es un booleano, es un objeto
{ "rta": true }
Cesar Elías Armendariz Ruano
Julian Alejandro
¿Cómo sería una prueba para el siguiente método?
fetchReadAndUpdate(id: string, dto: UpdateProductDTO) { return zip( this.getOne(id), this.update(id, dto) ); }
Tanelo Mondaca Silva
Tienes que hacer mocks de los metodos que estás llamando para que respondan lo que idealmente los métodos reales responderían
Juan Luna
Jorge Luis Silva Medina
Exacto la misma estructura pero maneje diferente el método de comparación toBeTrue. Excelente consejo para el spred operator. Ya quiero ver el curso de testing para NestJS
En el código veo esto
this.http.delete<boolean>
Creo que lo correcto sería
this.http.delete<{ rta: boolean }>
Ya que así es como responde el DELETE de la API https://api.escuelajs.co/docs/#/products/ProductsController_delete
describe('test for update', () => { it('should return an updated product', (doneFn) => { // Arrange const mockData = generateOneProduct(); const id = '1'; const dto: UpdateProductDTO = { title: 'update Product' }; // Act productsService.update(id, {...dto}).subscribe((data) => { // Assert expect(data).toEqual(mockData); doneFn(); }); //http config const url = `${environment.API_URL}/api/v1/products/${id}`; const req = httpController.expectOne(url); req.flush(mockData); expect(req.request.body).toEqual(dto); expect(req.request.method).toEqual('PUT'); }); }); describe('test for delete', () => { it('should delete a product', (doneFn) => { // Arrange const id = '1'; // Act productsService.delete(id).subscribe((data) => { // Assert expect(data).toBe(true); doneFn(); }); //http config const url = `${environment.API_URL}/api/v1/products/${id}`; const req = httpController.expectOne(url); req.flush(true); expect(req.request.method).toEqual('DELETE'); }); });
¿No se debería crear un producto y luego actualizarlo?
no se algo como esto:
fdescribe('test for update', ()=>{ it('should return a product updated, dto should be mutable and http.method is "PUT"', (doneFn)=>{ // ARRANGE const mockData = generateOneProduct(); const cdto: CreateProductDTO = { title: 'New Product', price: 100, images: ['image0', 'image1'], description: 'bal bla bla, bal bla, bla', categoryId: 12 }; const udto: UpdateProductDTO = { title: 'product Updated', price: 200 }; // ACT productsService.create({...cdto}).subscribe(data =>{ // ARRANGE const updated: Product = { id: data.id, title: 'product Updated', price: 200, images: ['image0', 'image1'], description: 'bal bla bla, bal bla, bla', category: { id: 12, name: data.category.name } }; // ASSERT expect(data).toEqual(mockData); productsService.update(data.id, {...udto}).subscribe(dataUpdated => { // ASSERT console.log(dataUpdated); expect(dataUpdated).toEqual(updated); doneFn(); }); // CONFIG const url = `${environment.API_URL}/api/v1/products/${data.id}`; const req = httpController.expectOne(url); req.flush(updated); expect(req.request.body).toEqual(udto); expect(req.request.method).toEqual('PUT'); httpController.verify(); }); // CONFIG const url = `${environment.API_URL}/api/v1/products`; // console.log(url); const req = httpController.expectOne(url); req.flush(mockData); expect(req.request.body).toEqual(cdto); expect(req.request.method).toEqual('POST'); // httpController.verify(); }); });
yo lo he probado y funciona, mi pregunta es ¿Es necesario hacerlo así o basta como lo hizo el profe Nicolás?
