Bug Tracker

Opened 12 years ago

Closed 11 years ago

Last modified 11 years ago

#10693 closed enhancement (fixed)

generalize the "test something in an iframe" code in unit tests

Reported by: mikesherov Owned by: mikesherov
Priority: low Milestone: 1.7.2
Component: build Version: 1.7
Keywords: Cc:
Blocked by: Blocking:

Description

there are 3 different pieces of code in 3 different unit test modules:

support.js: supportIFrameTest
selector.js: testIframe
offset.js: testoffset

testIframe and testOffset are almost exact duplicates. supportIFrameTest is close enough. I was about to start writing another one for a ticket I'm trying to fix in event.js, but perhaps it's time to DRY this code and move it into test/data/testinit.js ?

I'd be glad to take a stab at it. 1.8?

Change History (4)

comment:1 Changed 12 years ago by dmethvin

Component: unfiledbuild
Milestone: None1.8
Priority: undecidedlow
Status: newopen

Sounds like a good idea!

comment:2 Changed 12 years ago by mikesherov

Owner: set to mikesherov
Status: openassigned

comment:3 Changed 11 years ago by mikesherov

Milestone: 1.81.7.2

comment:4 Changed 11 years ago by mikesherov

Resolution: fixed
Status: assignedclosed
Note: See TracTickets for help on using tickets.