search

marcellinodour / MIDO-SVG

Other
0 stars 0 forks source link

Dev svg it4bis #32

Closed brulej closed 4 years ago

brulej commented 4 years ago

Cette PR reprend la PR #26 en prenant en compte les commentaires que vous y avez indiqué. Cependant, nous n'avons pas compris ce que vous attendiez concernant les BP null à appliquer dans la méthode countMaxSubjects de la classe ResponsiveSVG. Après relecture des BP, nous ne comprenons pas où se situe le problème dans notre méthode.

brulej commented 4 years ago

La raison de l’échec du build a été identifiée, elle est due à un test qui n'est plus pertinent. Je vais donc supprimer la classe GUIJUnitTest qui avait été implémentée par les créateurs du projet.

brulej commented 4 years ago

Correction : en attente que les tests passent.

Build OK. J'ai opté pour une solution radicale, supprimer le test. En effet, il avait pour but de compter les formations de la base fictive.

camillelanglois3 commented 4 years ago

Pouvons-nous merge à présent ?

brulej commented 4 years ago

Ce test me semblait néanmoins marginalement utile, ne fût-ce qu’à des fins de documentation. Ne pouvez-vous pas le restituer en modifiant l’assertion ou ce que vous comptez ? Je vous laisse décider de la meilleure option puis fusionner.

Le test ne marchait plus à cause de la méthode countFormations , le deuxième paramètre n'était plus du bon type. J'ai simplement changé "M1" en 4.

Je peux donc merge ?