Las actividades no pueden crearse fuera del horario permitido (Fixes #122)

andresmanelli commented 8 years ago

Para corregir el problema se verifican los límites en la route correspondiente.

Cambios al server

Dividí el server en dos partes: server.js y run.js:
- server.js expone la configuración de las routes (lo que sería el objeto app)
- run.js crea los workers y los mantiene

Esto permite cargar en los test unitarios sólo la definición y correr un server uniproceso. El funcionamiento se mantiene igual porque el Gruntfile carga run.js en vez de server.js.

Mocks del server

Para testear la API del server sin meterse con la BDD se pueden hacer mocks de las routes _*.js para que devuelvan siempre OK y un array nulo (o lo que corresponda). Así se puede testear todo lo que está antes de la llamada a la BDD (verificación de usuario loggeado, consistencia de parámetros, etc).

Los mocks los puse en server/specs/mocks pero lo pueden cambiar si les parece que va en otro lado.

Para cargarlos usé el módulo mockery que básicamente intercepta los require y hace cargar el módulo que uno le indica.

var mockery = require('mockery');
mockery.registerSubstitute('./_edt.js','../specs/mocks/_edt.mock.js');
[...]
mockery.enable({
    warnOnUnregistered: false
});

Hay que tener en cuenta que lo que uno indica como primer parámetro es el string cómo lo carga el módulo que lo use (respecto del archivo de spec no está en ./ pero así lo llama routes/edt.js)

Gruntfile

Agregué una task uncache para borrar la cache de require al hacer los tests unitarios al server porque si no hacemos esto grunt-jasmine-nodejs no vuelve a cargar los tests. (Whaaat)
Borré la task express y la distribuí en las tasks que corren los tests al server para que lo paren y lo vuelvan a correr antes y después de los tests (porque éstos corren su instancia desde 0)
package.json
Actualicé grunt-contrib-watch
Agregué supertest (para hacer tests sobre requests http)
Agregué mockery (para lo que ya está dicho arriba)
linter del server

Sólo se verifican los specs. Explota todo cuando se habilita sobre el src del server, asique por ahora lo dejé. Será para otro issue. Hay que descomentar esto (342 a 344):

//server: [
//        '<%= server_files.js %>'
//      ],

fcladera commented 8 years ago

Che, sabés que estaría bueno, que esta doc que estás escribiendo vaya a algún archivo como Docs/architecture.md (o el nombre que sea más adecuado). Así no se quedan en el olvido cuando sale el PR, y son bastante útiles para entender como funcionan las cosas.

andresmanelli commented 8 years ago

De una. En la semana me pongo a recopilar las docs de PR en général y voy armando algo

-----Message d'origine----- De : "Fernando Cladera" notifications@github.com Envoyé : ‎17/‎04/‎2016 22:46 À : "MecatronicaUncu/Red-Social-Asociacion" Red-Social-Asociacion@noreply.github.com Cc : "Andrés" andresmanelli@gmail.com Objet : Re: [MecatronicaUncu/Red-Social-Asociacion] Las actividades no puedencrearse fuera del horario permitido (Fixes #122) (#151)

Che, sabés que estaría bueno, que esta doc que estás escribiendo vaya a algún archivo como Docs/architecture.md (o el nombre que sea más adecuado). Así no se quedan en el olvido cuando sale el PR, y son bastante útiles para entender como funcionan las cosas. — You are receiving this because you authored the thread. Reply to this email directly or view it on GitHub

andresmanelli commented 8 years ago

@fcladera rebaseo esto a develop por lo de docker?

fcladera commented 8 years ago

Si Rulo, siempre un paso adelante. Iba a entrar para pedirte eso.

andresmanelli commented 8 years ago

@fcladera ahí está meta docker

fcladera commented 8 years ago

Gracias Rulo, ahí lo pruebo en un rato

fcladera commented 8 years ago

Hay chance de que rebasees esto sobre lo nuevo?

andresmanelli commented 8 years ago

por supuesto

andresmanelli commented 8 years ago

Ahi va

fcladera commented 8 years ago

Buen trabajo, anda muy bien

MecatronicaUncu / Red-Social-Asociacion