Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Base du projet fonctionnelle - Grosse mise à jour #35

Merged
merged 11 commits into from
Jun 6, 2016

Conversation

Scriptodude
Copy link
Collaborator

@Scriptodude Scriptodude commented Jun 6, 2016

Fix #33 #8 #15 #31
Alors attachez-vous bien, ça commence : 😃

Modifications

  • Ajout du support de django-nose pour le coveralls dans requirements.txt
    • Modification du travis.yml pour ce même support
  • Intégration de sémantics-ui dans la page principal d'encefal et ses sous-pages
  • Ajout de plusieurs tests pour forms.py et models.py
    • (~60%, coverage manquait les lignes de définitions pour des raisons inconnus)
  • Modification du encefal.js pour supporter les nouveaux prix ainsi que django 1.9.6
  • Modification de plusieurs page pour offrir une plus belle mise en forme

Ajout

  • dossiers "tests" dans les applications "employee" et "main".
    • Ces dossiers contiennent les fichiers test_[nomDuModule].py, testant les modules nommés
  • Lien du site principal vers le site employée (login required)
  • Mode expert dans le site employé
    • permet une plus grande gestion du module "main" (livres et ventes)
    • à venir : nouvelle mise en forme permettant aussi l'ajout d'employés pour les factures
  • Coveralls ne _devrait plus_ vérifier le coverage de l'environnement virtuel

Bug connus

  • Aucun message d'erreur lorsqu'on entre un identifiant invalide sur la page "vendre livre"
  • Mise en forme du calendrier sur le rapport pratiquement inexistante
    • semantics-ui n'offre pas encore l'option d'un datepicker comme bootstrap
  • le coveralls de l'application employee devrait être à 0%
    • les tests de admin.py sont inexistant, baissant la côte coveralls
  • Coveralls manque les lignes de définitions de fonctions et de classes, baissant le pourcentage de beaucoup
  • La page "livres" s'affiche identique sur mobile et ordinateur

Et voilà, en théorie le gros du travail effectué est noté ci-haut. Le PR actuel devrait être une base solide pour implémenter les futures features mentionnés dans #1

@coveralls
Copy link

coveralls commented Jun 6, 2016

Coverage Status

Coverage decreased (-10.3%) to 28.917% when pulling feb5b4a on Scriptodude:AdminUp into f9cb230 on AESSUQAM:develop.

@coveralls
Copy link

coveralls commented Jun 6, 2016

Coverage Status

Coverage decreased (-6.8%) to 32.386% when pulling ddee87a on Scriptodude:AdminUp into f9cb230 on AESSUQAM:develop.

@ventilooo
Copy link
Member

Good to me ! however why does coveralls fail ?

@ventilooo ventilooo merged commit 9fdd34c into AESSUQAM:develop Jun 6, 2016
@Scriptodude
Copy link
Collaborator Author

Scriptodude commented Jun 6, 2016

Looks like i didn't find the error XD
The documentation on coveralls.io kinda annoys me. I find low to no support and about no answers on forums. I'll try one little change on travis and see how it goes. Hopefully, it will fix it up. Otherwise, i'll start a coveralls.io on my own and test stuff on my repo and PR the functional changes here.

@ventilooo
Copy link
Member

Good idéal, please setup a travis and coverolls for your fork

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants