Then backport to Drupal 7

CommentFileSizeAuthor
#8 range-introduce_tests-2204313-8.patch14.05 KBtaran2l
#3 range-tests-range-integer-2204313-1.patch7.13 KBozin

Comments

taran2l’s picture

taran2l
Primary language Ukrainian
Location Lviv
commented
Issue tags: +D8CX
ribel’s picture

ribel
Primary language Ukrainian
Location 🇺🇦Lviv
commented
Assigned: Unassigned » ribel

Need to write classes:

RangeFieldTest extends WebTestBase

with methods:

  • testRangeDecimalField
  • testRangeIntegerField
  • testRangeFloatField

RangeFieldItemTest extends UnitTestBase

with methods:

  • testRangeItem
ozin’s picture

ozin
Primary language Ukrainian
commented
StatusFileSize
new range-tests-range-integer-2204313-1.patch7.13 KB

Added patch for the Range Integer field. Please check and let me know if need to add more cases for this field.
As soon as possible I will add tests for rest types, unit tests (RangeFieldItemTest.php) and UI tests for testing settings forms for fields(RangeFieldUITest.php).

  • Taran2L committed 5e38946 on 8.x-1.x 
    Issue #2204313: Add empty field UI test

  • Taran2L committed bd443ee on 8.x-1.x 
    Issue #2204313: Add field UI tests

  • Taran2L committed 647ac2f on 8.x-1.x 
    Issue #2204313: Make DrupalCI happy

  • Taran2L committed bf5cd96 on 8.x-1.x 
    Issue #2204313: Make RangeTestBase class abstract
taran2l’s picture

taran2l
Primary language Ukrainian
Location Lviv
commented
Status: Active » Needs review
StatusFileSize
new range-introduce_tests-2204313-8.patch14.05 KB

Status: Needs review » Needs work

The last submitted patch, 8: range-introduce_tests-2204313-8.patch, failed testing.

  • Taran2L committed abf3b73 on 8.x-1.x 
    test(formatter): Cover field formatter with tests (issue #2204313 by...
taran2l’s picture

taran2l
Primary language Ukrainian
Location Lviv
commented

deleted

taran2l’s picture

taran2l
Primary language Ukrainian
Location Lviv
commented
Title: Introduce tests » [meta] Introduce tests
taran2l’s picture

taran2l
Primary language Ukrainian
Location Lviv
commented
Status: Needs work » Fixed

This one could be closed, as all child issues are closed now.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.