Quick brainstorm of functional tests:
1. Test that the Clock outputs the expected string (multiple configurations).
2. Test that the "User time zone" option does not appear if user-configurable time zones are disabled.
3. Test that custom date formats show up.

CommentFileSizeAuthor
#4 more-tests.patch4.47 KBtstoeckler
#3 test-user-time-zone.patch2.19 KBtstoeckler
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

tstoeckler’s picture

tstoeckler
he/him
Primary language German
Location Essen, Germany
CreditAttribution: tstoeckler commented
Status: Active » Reviewed & tested by the community

Since I can't roll patches with new files, I am committing an intitial clock.test to CVS. I contains a bunch of stub tests, that have not yet been implemented.

tstoeckler’s picture

tstoeckler
he/him
Primary language German
Location Essen, Germany
CreditAttribution: tstoeckler commented
Title: Test coverage » Initial test coverage
Status: Reviewed & tested by the community » Fixed

http://drupal.org/cvs?commit=317398

tstoeckler’s picture

tstoeckler
he/him
Primary language German
Location Essen, Germany
CreditAttribution: tstoeckler commented
Title: Initial test coverage » Test coverage
Status: Fixed » Needs review
FileSize
test-user-time-zone.patch2.19 KB

Well, I guess I'll reuse the issue.
Uploaded patch adds one working test. Now that I've got that working, the other ones should follow soon.

tstoeckler’s picture

tstoeckler
he/him
Primary language German
Location Essen, Germany
CreditAttribution: tstoeckler commented
FileSize
more-tests.patch4.47 KB

Here we go.
With this patch test coverage should really be complete for so small a module.

tstoeckler’s picture

tstoeckler
he/him
Primary language German
Location Essen, Germany
CreditAttribution: tstoeckler commented
Status: Needs review » Fixed

Here we go: http://drupal.org/cvs?commit=317948

Status: Fixed » Closed (fixed)

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