Issue #178

[codecov[bot]]

Codecov Report

Merging #222 into master will increase coverage by 1.73%. The diff coverage is 0%.

@@             Coverage Diff              @@
##             master     #222      +/-   ##
============================================
+ Coverage      8.79%   10.52%   +1.73%     
- Complexity      354      370      +16     
============================================
  Files            38       40       +2     
  Lines           978     1007      +29     
============================================
+ Hits             86      106      +20     
- Misses          892      901       +9

Flag	Coverage Δ	Complexity Δ
#php	10.52% <0%> (+1.73%)	370 <63> (+16)	:arrow_up:

Impacted Files	Coverage Δ	Complexity Δ
app/Servicio.php	78.57% <ø> (+7.14%)	15 <0> (ø)	:arrow_down:
app/Http/Controllers/Auth/RegisterController.php	0% <ø> (ø)	3 <0> (-1)	:arrow_down:
app/Vehiculo.php	100% <ø> (+28.57%)	8 <0> (ø)	:arrow_down:
app/Http/Controllers/MaterialController.php	0% <ø> (ø)	22 <0> (ø)	:arrow_down:
app/Http/Requests/VariableRequest.php	0% <0%> (ø)	7 <7> (?)
app/Http/Controllers/PuntuacionController.php	0% <0%> (ø)	37 <0> (-2)	:arrow_down:
app/Http/Controllers/ServicioController.php	0% <0%> (ø)	73 <38> (ø)	:arrow_down:
app/Http/Controllers/VariableController.php	0% <0%> (ø)	12 <12> (?)
app/Http/Requests/MaterialRequest.php	0% <0%> (ø)	6 <6> (ø)	:arrow_down:
... and 3 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update f1bb9f0...54fd442. Read the comment docs.

[ignaciojonas]

@perezale Para mi esta bien, igual me gustaria que vos le pegues una mirada, sobre todo a los HTML.

[ignaciojonas]

@SSAAMMAA Lo unico que me di cuenta recien es que el Code Coverage bajo. El tema es que vos agregaste mucho codigo, que si bien esta cubierto por los test de browser que hiciste. Si Podes/Queres, agregate unos tests de unidad en Servicio o Vehiculo, asi eso pasa a verde.

[SSAAMMAA]

@ignaciojonas no encuentro en ningun lado algun ejemplo de test para esos casos, en las filminas no hay nada y en internet no se por donde mas buscar

[ignaciojonas]

Es super simple, por ejemplo: el metodo materiales en Servicio: Creas un servicio, que le asignas una lista de materiales que vos conoces. Obtenes $servicio->materiales() y te fijas que la lista coincida. Lo mismo para el resto de los tests.

[SSAAMMAA]

@ignaciojonas algo asi? public function testBomberos() { $bombero = factory(Bombero::class)->create(); $service = factory(Servicio::class)->create(); $bomberoServicio = factory(BomberoServicio::class)->create(['bombero_id'=>$bombero->id], ['servicio_id'=>$service->id]); $this->assertEquals($bomberoServicio->first()->bombero_id,$bombero->id); $this->assertEquals($bomberoServicio->first()->servicio_id,$service->id); }

seria para servicio y bomberos, igualmente no me deja porque no encentra ese BomberoServcio...

[ignaciojonas]

@SSAAMMAA Si venis encaminado. Tendrías que crear el Factory de BomberoServicio. Si vas a probar Bomberos, entonces hace:

public function testBomberos()
{
$bombero = factory(Bombero::class)->create();
$service = factory(Servicio::class)->create();
$bomberoServicio = factory(BomberoServicio::class)->create(['bombero_id'=>$bombero->id], ['servicio_id'=>$service->id]);
$this->assertEquals($service->bomberos()->first()->bombero_id,$bombero->id);
}

[SSAAMMAA]

@ignaciojonas ya debería estar, no? gracias!

[ignaciojonas]

Genial!