Bug Tracker

Opened 7 years ago

Closed 5 years ago

#12778 closed feature (migrated)

rewrite speed framework based off of the latest sizzle speed tests

Reported by: mikesherov Owned by:
Priority: low Milestone: None
Component: build Version: git
Keywords: Cc: Rick Waldron
Blocked by: Blocking:

Description


Change History (7)

comment:1 Changed 7 years ago by mikesherov

Component: unfiledbuild
Owner: set to mikesherov
Priority: undecidedlow
Status: newassigned
Type: bugfeature

comment:2 Changed 7 years ago by mikesherov

have it test against an earlier build

comment:3 Changed 7 years ago by timmywil

I'm not sure sizzle speed tests are the best starting point. They are specially manufactured to work with selector engines, testing each selector and each engine in their own iframes. What we need is probably something custom manufactured to test jQuery modules. Having said that, there a couple principles I think we should adopt from Sizzle's speed suite.

  1. Tests don't really need a back-end. Slickspeed operates only with PHP. I don't think we even need node.
  2. The benchmarkjs project is a now a submodule of Sizzle; we can update to released versions whenever we need (which shouldn't happen that often).
  3. Tests should be encapsulated appropriately to guarantee that one test does not affect the other. Loading each test in its own iframe is probably going overboard in the core module, but may be right for the ajax module.

Finally, rwaldron seemed excited about revamping an old resig project for our speed tests. I'm still open to that.

comment:4 Changed 7 years ago by mikesherov

Cc: Rick Waldron added

I'm fine either way. I really just want to blast away the /speed dir. If Rick's on that, even better :-)

comment:5 Changed 7 years ago by dmethvin

Owner: mikesherov deleted
Status: assignedopen

Salvatore Iovene from Intel is going to take a shot at this. If you have thoughts post here or ping siovene in #jquery-dev.

comment:6 Changed 6 years ago by m_gol

dmethvin: any info on the progress on this?

comment:7 Changed 5 years ago by m_gol

Resolution: migrated
Status: openclosed
Note: See TracTickets for help on using tickets.