Simpletest HTML5 field support [#1828876]

Drupal 7 does not support HTML5 fields, and thus simpletest for D7 doesn't either. However, you can still use HTML5 with the help of contrib modules. We've discovered that if you do use contrib to use HTML5 fields, our tests fail due to the DrupalTestCase class not supporting HTML5 field types.

Here's a patch that's really more of a backport as this support is already present in D8. Apologies if this is a duplicate, I couldn't find an appropriate issue.

Comment	File	Size	Author
#4	1828876_simpletest_html5_field_support.patch	740 bytes	geekyMoa
#4
	simpletest_html5_field_support.patch	763 bytes	geekyMoa

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

1 November 2012 at 11:28

Status:

Needs review

» Needs work

The last submitted patch, simpletest_html5_field_support.patch, failed testing.

Comment #2

Letharion CreditAttribution: Letharion commented 1 November 2012 at 11:59

Status:

Needs work

» Needs review

Why would this cause failures all over the place like that?

Lets try again.

Comment #3

Letharion CreditAttribution: Letharion commented 1 November 2012 at 12:00

simpletest_html5_field_support.patch queued for re-testing.

Comment #4

geekyMoa CreditAttribution: geekyMoa commented 1 November 2012 at 12:13

File	Size
1828876_simpletest_html5_field_support.patch	740 bytes

Noticed that we accidentally removed support for text fields. Ooops! Resubmitting.

Comment #5

Anonymous (not verified) CreditAttribution: Anonymous commented 1 November 2012 at 17:46

Version:	7.x-dev	» 8.x-dev
Issue tags:		+Needs backport to D7

Feature requests need to go to the developing version.

Comment #6

geekyMoa CreditAttribution: geekyMoa commented 2 November 2012 at 08:16

Category:

feature

» bug

Okay, then I think this isn't a feature request but more of a... what, a bug report, I guess? There's no option for "backport". This is already in the 8.x-dev branch.

I'd like this in D7 too, if at all possible. :)

Comment #7

geekyMoa CreditAttribution: geekyMoa commented 2 November 2012 at 08:17

Version:

8.x-dev

» 7.x-dev

Comment #8

Anonymous (not verified) CreditAttribution: Anonymous commented 2 November 2012 at 11:52

What issue added it for D8?

Comment #9

Letharion CreditAttribution: Letharion commented 2 November 2012 at 15:03

There's no particular issue that made all the changes. The WebTestBase class contains support for all the HTML5 field types, and a git blame will show that it was added over a number of different commits and over an extended time period.

This is effectively a backport, but not of a particular patch.