Opened 11 years ago

Closed 10 years ago

Last modified 10 years ago

#8678 closed defect (fixed)

test dojo.query() on form dijits

Reported by: bill Owned by: haysmark
Priority: high Milestone: 1.4
Component: Dijit Version: 1.3.0b1
Keywords: Cc:
Blocked By: Blocking:

Description

As per #8660 and #8666 we should have tests to make sure that dojo.query() on name and checked state work correctly when applied to the <input> nodes in form dijits, particularly RadioButton

Attachments (1)

8678.patch (6.0 KB) - added by haysmark 10 years ago.
Fixes #8678. Test form widgets that use hidden inputs for compatibility with dojo.query.

Download all attachments as: .zip

Change History (7)

comment:1 Changed 10 years ago by Douglas Hays

Owner: set to haysmark

Changed 10 years ago by haysmark

Attachment: 8678.patch added

Fixes #8678. Test form widgets that use hidden inputs for compatibility with dojo.query.

comment:2 Changed 10 years ago by haysmark

I attached the test cases for this ticket.

comment:3 Changed 10 years ago by haysmark

Status: newassigned

comment:4 Changed 10 years ago by bill

Resolution: fixed
Status: assignedclosed

(In [20259]) Tests that dojo.query() works correctly on form widgets. Thanks Mark (IBM, CCLA)! Fixes #8678 !strict.

comment:5 Changed 10 years ago by bill

(In [20295]) Fix trailing comma, refs #8678.

comment:6 Changed 10 years ago by bill

(In [20297]) Form tests:

  • Fix "already called" error on Form.html setup. The default test timeout is 1000ms so need to raise it for tests that take longer/
  • Roll the dijit.form.Form / dojo.query() tests into Form.html; it seemed excessive to load the robot code just to test dojo.query(), and also having Form_query.html load Form.html meant that DOH was running twice, once inside each file.

Refs #8678

Note: See TracTickets for help on using tickets.