Feature 68 dashboard - Githubissues

Hypertopic / Tire-a-part

Digital repository for the papers of a research organization.

27 stars 43 forks source link

Feature 68 dashboard #149

Closed FlorianBruniaux closed 10 years ago

FlorianBruniaux commented 10 years ago

This branch contains :

a merge of current benel/Tire-a-part/master and benel/Tire-a-part/feature-68-dashboard with corrections of conflicts etc.
feature-68 test
Modif of templates/activity_html.html :
- add "Compter" button
- click event on "Exporter" and "Compter" ( $('#id').on('click', function(){}) )
Modif of spec_helper.rb :
- Changed const Capybara.default_wait_time from 2s (default value) to 4s RQ: minimum value = 3.5s

FlorianBruniaux commented 10 years ago

C'est étrange que le test Delete a record échoue alors qu'il est passé sur mon repo lorsque j'ai fait un merge de test-feature-68 dans feature-68-dashboard.

Une idée ? Est-ce encore le timeout ?

benel commented 10 years ago

J'ai relancé l'exécution. Cette fois-ci c'est un autre test qui a échoué. La piste du timeout est effectivement probable.

FlorianBruniaux commented 10 years ago

J'ai changé le timeout (5s) et c'est à nouveau le test Delete a record qui échoue !

benel commented 10 years ago

Sur le serveur de Travis CI, les versions de Capybara et de RSpec ont été mises à jour. Cela pourrait expliquer ces problèmes qui apparaissent sur le serveur et n'existaient pas sur votre poste.

Cela devrait mieux marcher avec cette nouvelle syntaxe : ba263c37bb08b6913f8a412d827e5919fa22ec98 (livraison disponible dans la branche maître).

FlorianBruniaux commented 10 years ago

J'ai récupéré vos modifications. Tout passe sur ma branche.

Le test Create a record for a published article a échoué.

RQ: En TP nous avons constaté ce bug à plusieurs reprises (en local) lors du premier test. En relançant les tests il disparaissait.

Peut être pourriez vous tenter de les relancer..

benel commented 10 years ago

Test committed in 0d8e9b318cee6b6375354363484867789b772eb3.