Opened 11 years ago

Closed 11 years ago

#5905 closed defect (fixed)

[patch] [cla] Tests for dojo.forEach using strings as callbacks

Reported by: wolfram Owned by: alex
Priority: high Milestone: 1.1
Component: Core Version: 1.0
Keywords: Cc:
Blocked By: Blocking:

Description

the current tests in dojo._base.array dont test for constructs such as:

dojo.forEach([1,2,3], 'sum+=item');

I added some tests

Attachments (1)

patch5905.diff (1.3 KB) - added by wolfram 11 years ago.

Download all attachments as: .zip

Change History (4)

Changed 11 years ago by wolfram

Attachment: patch5905.diff added

comment:1 Changed 11 years ago by wolfram

I just cant get around asking, would all those test cases not be perfect DocTests?? would be awesome getting #4651 in

comment:2 Changed 11 years ago by dylan

Milestone: 1.1

comment:3 Changed 11 years ago by Adam Peller

Resolution: fixed
Status: newclosed

(In [12668]) Add Wolfram's test case for string callbacks. Fixes #5905. Had to modify exception test since error string seems to vary across browsers.

Note: See TracTickets for help on using tickets.