Add test coverage for the repeating date functionality from Date Repeat Field.

CommentFileSizeAuthor
#5 date-n3276311-5.patch11.78 KBdamienmckenna
#3 date-n3276311-3.patch21.9 KBdamienmckenna

Comments

DamienMcKenna created an issue. See original summary.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Assigned: Unassigned » damienmckenna
damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Active » Needs review
StatusFileSize
new date-n3276311-3.patch21.9 KB

Some test coverage. This also splits the repeating date functionality into a separate feature.

Status: Needs review » Needs work

The last submitted patch, 3: date-n3276311-3.patch, failed testing. View results

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Needs work » Needs review
StatusFileSize
new date-n3276311-5.patch11.78 KB

Ok fine, leave the date_repeat stuff in the main feature module.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Needs review » Fixed

Committed.

  • DamienMcKenna committed 0c2d3b0 on 7.x-2.x 
    Issue #3276311 by DamienMcKenna: Test coverage for Date Repeat Field...
damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Assigned: damienmckenna » Unassigned

  • DamienMcKenna committed 3ba3efd on 7.x-3.x 
    Issue #3276311 by DamienMcKenna: Test coverage for Date Repeat Field...

Status: Fixed » Closed (fixed)

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