Implementando el POST y DELETE de las películas de usuario

Clase 15 de 39 • Curso de Autenticación con Passport.js 2019

David Behar

student•

Para esta clase implementamos la creación y eliminación de relaciones (películas de usuario) siguiendo la misma lógica que para el listado de elementos.

Implementación de la ruta post

router.post(
   '/',
   validationHandler(createUserMovieSchema),
   async (req, res, next) => {
     const { body: userMovie } = req;
     try {
       const createdUserMovieId = await userMoviesApi.createdUserMovieId({
         userMovie,
       });

       req.status(201).json({
         data: createdUserMovieId,
         message: 'user movie created',
       });
     } catch (err) {
       next(err);
     }
   },
 );

Implementación de la ruta de delete

router.delete(
    '/:userMovieId',
    validationHandler({ userMovieId: movieIdSchema }, 'params'),
    async (req, res, next) => {
      const { userMovieId } = req.params;
      try {
        const deletedUserMovieId = await userMoviesService.deleteUserMovies({
          userMovieId,
        });

        res.status(200).json({
          data: deletedUserMovieId,
          message: 'user movie deleted',
        });
      } catch (error) {
        next(error);
      }
    },
  );

Para finalizar con el. llamado de nuestro endpoints en la raiz de nuestro proyecto (index.js)

const userMoviesApi = require('./routes/userMovies');

app=express();

userMoviesApi(app);

Ver commit referente a la clase

rusbel bermúdez rivera

student•

esta clase fue un gran repaso del curso anterior de express, excelente clase *****

Jose Daniel Molina

student•

De a poco creo que voy entendiendo como va esto... Personalmente se me hace increíblemente difícil.

José Enrique Pérez Aquino

student•

Al principio el backend puede ser complicado si no entiendes la lógica correctamente, a diferencia del frontend que puedes ver los resultados más fácil. Pero no por eso imposible, solo requiere de practica y perseverancia.

Jose Daniel Molina

student•

Es cierto, pero me he dado cuenta de que he podido hacer las cosas, a pesar de los errores constantes, los he solucionado y seguido adelante! #NuncaParesDeAprender 💚

José Daniel Navarrete Díaz

student•

El tema para validar los request que se envían al server, se puede manejar con typeScript?

Wilson Marino Pablo Mendez

student•

Sería bueno que utilizaran TypeScript. .

Juan Carrillo

student•

Tengo entendido que typeScript es más para validación de tipos mientras desarrollamos, lo que nos evita errores comunes en js con el manejo de los tipos de variables. En cambio la validación que se hace en los requests podría ser más específica, por ejemblo un nombre de usuario que como minimo tenga 3 carácteres y máximo 10, aplicar expresiones regulares, y muchos otros tipos de validaciones que tiene incluido joi

Carlos Gutierrez [C6]

student•

Mi codigo lo estructure de la siguiente manera:

// Routes
router.get('/', validationHandler({ userId: userIdSchema }, 'query'), getMoviesUser)
  router.post('/', validationHandler(createUserMovieSchema), postMoviesUser)
  router.delete('/:userMovieId', validationHandler({ userMovieId: movieIdSchema}, 'params'), deleteMoviesUser)


// Functions
const getMoviesUser = async (req, res, next) => {
    const { userId } = req.query
    try {
      const userMovies = await userMoviesService.getUserMovies({ userId })
      response.success(req, res, userMovies, 'user movies listed', 200)
    } catch (error) {
      next(error)
    }
  }

  const postMoviesUser = async (req, res, next) => {
    const { body: userMovie } = req
    try {
      const createUserMovieId = await userMoviesService.createUserMovie({ userMovie })
      response.success(req, res, createUserMovieId, 'user movie created', 201)
    } catch (error) {
      next(error)
    }
  }

  const deleteMoviesUser = async (req, res, next) => {
    const { userMovieId } = req.params
    try {
      const deleteUserMovieId = await userMoviesService.deleteUserMovie({ userMovieId })
      response.success(req, res, deleteUserMovieId, 'user movie deleted', 200)
    } catch (error) {
      next(error)
    }
  }

// Response Success
const success = (req, res, data, message, status) => {
  const stat = status || 200
  res.status(stat).json({
    data: data,
    message: message
  })
}

Alfredo Gonzalez

student•

Aqui como escribir un custom middleware.

https://expressjs.com/en/guide/writing-middleware.html

Naldo Duran

student•

✌

Camilo Perilla

student•

Se entendio todo muy bien, pero aun no comprendo porque no se hace lo mismo para la collección de usuarios que se creo dos clases atras. Si alguien sabe porque no se configuran los endpoints para manejar dicha colección, que lo comparte pleasee.

Raymond Alexander Mansell Muñoz

student•

En el min 3:20 creo que hay un pequeño error, el middleware de validación para la ruta delete debería ser

validationHandler({ userMovieId: userMovieIdSchema })

Aunque este caso no hay problema porque movieIdSchema está verificando la misma expresión regular para los ID's que genera Mongo.

Beto Martinez

student•

Personalmente me parece que estaria mejor implementar Mongoose, asi podemos hacer tambien validaciones con Schemas y hacer las Querys mas sencillas, pero siempre es bueno aprender otros metodos.

Nicolas Esteban Manograsso

student•

Comparto algunos tests que hice sobre la ruta

const assert = require('assert');
const proxyquire = require('proxyquire');

const {
  userGamesMock,
  userGameWithoutIdMock,
  filteredUserGamesMock,
  userGamesServiceMock,
} = require('../utils/mocks/userGamesMock');
const testServer = require('../utils/testServer');

describe('routes - user-games', () => {
  const route = proxyquire('../routes/userGames', {
    '../services/userGames': userGamesServiceMock,
  });

  const request = testServer(route);

  describe('GET /user-games?userId=', () => {
    it('should respond with status code 200', (done) => {
      request.get(`/api/user-games?userId=${userGamesMock[0].userId}`).expect(200, done);
    });

    it('should respond with the list of user games', (done) => {
      request.get(`/api/user-games?userId=${userGamesMock[0].userId}`).end((err, res) => {
        if (err) done(err);

        assert.deepStrictEqual(res.body, {
          data: filteredUserGamesMock(userGamesMock[0].userId),
          message: 'user games listed',
        });

        done();
      });
    });
  });

  describe('POST /user-games', () => {
    it('should respond with status code 201', (done) => {
      request.post('/api/user-games')
        .send(userGameWithoutIdMock)
        .expect(201, done);
    });

    it('should respond with the created game', (done) => {
      request.post('/api/user-games')
        .send(userGameWithoutIdMock)
        .end((err, res) => {
          if (err) done(err);

          assert.deepStrictEqual(res.body, {
            data: userGamesMock[0]._id,
            message: 'user game created',
          });

          done();
        });
    });
  });
});

Les comparto el repo donde están todos los tests y mocks 😁

PD: cambien la palabra games por movies, en mi proyecto lo cambié un poco 😛

Karen Paola Diaz Duarte

student•

Excelente gracias!!