facebookarchive/draft-js

Test for indenting list item more than one level

Open

#1 674 ouverte le 2 mars 2018

Voir sur GitHub
 (3 commentaires) (0 réactions) (0 assignés)JavaScript (22 554 stars) (2 646 forks)batch import
good first issuehelp wantedtest coverage

Description

After we merge https://github.com/facebook/draft-js/pull/762 then Draft should support indentation of a list more than one item deep.

To make sure we don't regress this support in the future, let's add a test.

I would add a test in the suite of unit tests for RichEditorUtil, here -> https://github.com/facebook/draft-js/blob/master/src/model/modifier/__tests__/RichTextEditorUtil-test.js

We don't seem to have any test for 'onTab' so let's add one.

testing_i_love_testing

Guide contributeur

Stack technique
reactjavascript
Domaine
frontendtesting
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.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
under 1 hour
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
Understanding of Draft.js editorFamiliarity with unit testing in React
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70
Direction de recherche
Review the linked PR #762 to understand the expected behavior for indenting list items more than one level. Then, open the file 'src/model/modifier/ tests /RichTextEditorUtil test.js' and add a new test case for the 'onTab' function. Follow the existing test patterns to ensure the test passes and covers the scenario.