Closed Bug 381001 Opened 17 years ago Closed 17 years ago

Integrate Scriptaculous Test Suite

Categories

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

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: jeresig, Assigned: jeresig)

References

(Depends on 1 open bug, Blocks 1 open bug, )

Details

Attachments

(2 files, 1 obsolete file)

Attached patch Scriptaculous Test Suite Patch (obsolete) — Splinter Review
This is the integration of the Scriptaculous test suite into the Mozilla (and Mochitest) test suite.

Since releases of Scriptaculous aren't tagged, this is the equivalent to SVN r6710. This is meant to go along with the other Ajax libraries being tested. (This specific patch should be applied in mozilla/dom/tests/mochitest/ajax/)

All of the test files are included with the exception of two: slider_test.html (because it currently errors out in Firefox 2 and Trunk) and bdd_test.html (because it doesn't use the correct callback scheme that the rest of the suite uses). Additionally, there is one image file that will be attached separately.

This patch adds 893 new tests to the automated test suite.
Attachment #265105 - Flags: review?(sayrer)
Attached image Test Image
This is a binary image that needs to be included with the suite. Specifically, this file should go in: mozilla/dom/tests/mochitest/ajax/scriptaculous/test/unit/
Attachment #265106 - Flags: review?(sayrer)
Comment on attachment 265105 [details] [diff] [review]
Scriptaculous Test Suite Patch

I'm not going to review all the JS code, but I assume it has been tested and is passing. Makefiles look good.
Attachment #265105 - Flags: review?(sayrer) → review+
Attachment #265106 - Flags: review?(sayrer) → review+
Has this been tested on linux? :)
New version of the test suite. Removed the autocomplete test (it was quite flaky) and generated a new -u9p diff. Tested under Linux and OS X. (Expects the Mochikit test suite patch to be applied before this one.)
Attachment #265105 - Attachment is obsolete: true
Attachment #268272 - Flags: review?(rcampbell)
Comment on attachment 268272 [details] [diff] [review]
Scriptaculous Test Suite Patch

tested locally on linux. Built and passed all tests.
Attachment #268272 - Flags: review?(rcampbell) → review+
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Flags: in-testsuite+
Blocks: 424816
Assignee: nobody → jresig
Component: Testing → DOM
QA Contact: testing → general
Depends on: 562083
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: