Opened 12 years ago

Closed 12 years ago

Last modified 12 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 12 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 12 years ago by Douglas Hays

Owner: set to haysmark

Changed 12 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 12 years ago by haysmark

I attached the test cases for this ticket.

comment:3 Changed 12 years ago by haysmark

Status: new → assigned

comment:4 Changed 12 years ago by bill

Resolution: → fixed
Status: assigned → closed

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

comment:5 Changed 12 years ago by bill

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

comment:6 Changed 12 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.