Open Bug 424816 Opened 13 years ago Updated 1 year ago

Update Mochitest Scriptaculous Test Suite to 1.8.1

Categories

(Core :: DOM: Core & HTML, defect)

defect
Not set
normal

Tracking

()

ASSIGNED

People

(Reporter: jeresig, Assigned: cmtalbert)

References

Details

Attachments

(1 file, 1 obsolete file)

The test suite that we have for Scriptaculous is out of date, at this point, it should be upgraded to the latest version: 1.8.1.
Heather-  This is what you are working on next, right?
Attached patch Patch with updated scriptaculous (obsolete) — Splinter Review
This patch brings us up to Scriptaculous 1.8.1.  This time, I found three tests that are no longer working, and one that is.

Tests Disabled last time:
* ajax_autocompleter_test.html
* slider_test.html
* bdd_test.html

This time, I found that the bdd_test.html worked without causing any issues.  But a new test has started failing.  This patch has the following tests disabled:
* ajax_autocompleter_test.html
* slider_test.html
* ajax_inplaceeditor_test.html

All the other tests run in the test.  I'll ask for review once I test it on linux to be sure it's ok there.
Assignee: nobody → ctalbert
Status: NEW → ASSIGNED
Attached patch Updated patchSplinter Review
I found a gotcha: the previous patch was missing two files.  Those have been added to this version.  I tested this version on Mac OS X and windows and it works, all tests pass, except for those mentioned above, which I'll open follow-on bugs for.
Attachment #327061 - Attachment is obsolete: true
Attachment #327197 - Flags: review?(rcampbell)
Rob C: are you the proper person to review this, or should it be Rob Sayre?

I've reviewed them in the past, though I'm not sure I'm the *correctest* person to do it at this point. Rob Sayre or Jeff Walden would be preferred.

That said, given the size of this patch, a line-by-line review is going to be time-consuming. Has this been tested in your local environment? How many platforms? Any changes to build code (i.e., makefiles)?
hey Clint, are you still waiting on a review from me for this?

How much has this patch been tested locally and on what platforms?
Component: Testing → DOM
QA Contact: testing → general
Comment on attachment 327197 [details] [diff] [review]
Updated patch

this patch is probably pretty stale. Clearing review flags. Rerequest if we need a review for this. Also, consider updating bug status.
Attachment #327197 - Flags: review?(rcampbell)
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.