GroceriStar/groceristar-fetch

update at getGroceriesWithDepIngKey tests

Open

#217 ouverte le 9 févr. 2019

Voir sur GitHub
 (2 commentaires) (0 réactions) (0 assignés)JavaScript (16 forks)auto 404
enhancementgood first issuehelp wantedin-progresstests

Métriques du dépôt

Stars
 (8 stars)
Métriques de merge PR
 (Métriques PR en attente)

Description

file: https://github.com/GroceriStar/groceristar-fetch/blob/master/projects/GroceriStar/getGroceriesWithDepIngKey.test.js

  • tests that checking properties - move to a separated function. you can read our other test files and find a solution.

  • Right now these tests didn't cover a lot of cases. Try to jump on grocery.json and delete an id property. Try to delete name, make name an empty string. Delete all data from one object, only keep id, name(no departments, no ingredients inside)

  • ingredients for grocery lists stored at Ingredients folder. they connected via file name. just delete one of the files, that should be connected to grocery and you wouldn't see an error.

  • Test Deep structure takes 6000ms how to reduce this number?

  • we have test Ingredient by length? WTF it's doing and why we need it - please explain.

  • making grocery list name as empty string didn't generate an error

@shubhamrajput it's for you

Guide contributeur

Direction de recherche
Commencez par examiner le fichier de test existant pour getGroceriesWithDepIngKey. Comprenez la structure de grocery.json et du dossier Ingredients. Pour chaque sous tâche, écrivez une fonction de test distincte. Pour réduire le temps de test de la structure profonde, envisagez d'utiliser des mocks ou d'optimiser les données. Enquêtez sur le test 'Ingredient by length' pour déterminer son objectif et sa nécessité. Assurez vous de tester les cas limites comme les propriétés manquantes, les chaînes vides et les fichiers d'ingrédients manquants.
Stack technique
javascript
Domaine
backend
Type d'issue
Test
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
JavaScriptGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65