Closed Bug 375543 Opened 15 years ago Closed 15 years ago

Integrate Prototype Test Suite

Categories

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

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla1.9alpha4

People

(Reporter: jeresig, Assigned: jeresig)

References

()

Details

Attachments

(3 files)

We need to start the process of integrating JavaScript library test suites into the Mozilla test system (most likely via Mochitest).

More information about this initiative can be found here:
http://ejohn.org/blog/future-proofing-javascript-libraries/

and here:
http://developer.mozilla.org/en/docs/Add_AJAX_Tests_To_Mozilla

I've completed the first steps of integrating and running Prototype's test suite against Mochitest, the results of which can be found here:
http://ejohn.org/blog/testing-prototype-with-firefox-3/

The only thing that I need to know is a specific directory in which these tests will be located, such that I can build a patch to add Prototype in.

I suspect that this ticket will also encapsulate setting up the groundwork of the framework test suite; but Prototype seems like as good a library as any to begin testing against.
This is a patch to start the foundation for all JavaScript library test suites (to be located in dom/tests/mochitest/ajax). Additionally, this patch adds the Prototype test suite as its first entry.
Attachment #262295 - Flags: review?(sayrer)
Attached image logo.gif
A gif that's used in the Prototype test suite. Should be put in the directory: dom/tests/mochitest/ajax/prototype/test/unit/fixtures/
Attachment #262297 - Flags: review?(sayrer)
Comment on attachment 262295 [details] [diff] [review]
Add Prototype and base test libs to the suite.

This duplicates MochiKit in ajax/lib, but I am OK with that because it will ease integration as we add Every Library Evar.
Attachment #262295 - Flags: review?(sayrer) → review+
Attachment #262297 - Flags: review?(sayrer) → review+
Whiteboard: [needs-checkin]
Whiteboard: [needs-checkin] → [checkin needed]
Assignee: nobody → jresig
Target Milestone: --- → mozilla1.9alpha4
checked in and passing.
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Whiteboard: [checkin needed]
An error recently popped up in a test case - and we found it really hard to debug what was going wrong. This patch adds line numbers to help determine where exactly the error is occurring. (Apply in mozilla/)
Attachment #275927 - Flags: review+
I just upstreamed the line number patch:
http://dev.rubyonrails.org/ticket/9231
Blocks: 424814
Component: Testing → DOM
QA Contact: testing → general
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.