Cover letter for the last patch

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Dear list,

Sorry,  I think i missed the cover letter for the last patch.

[PATCH/RFC] gitweb: Adding jasmine test suite files for the browser

JavaScript in gitweb is almost completely handwritten.
It is reimplementing things JavaScript library authors did a lot of times
already. Porting the current code to a library version was proposed as a gsoc
project this year. [ So I think I can skip why this patch is relevant]

A common suggestion was to add tests to the new JavaScript code. Deciding the
testing framework took much more time than expected after several discussions.
People wanted tests that could be run in console with TAP so that it will
integrate well with the existing system but since the code is ultimately going
to be run in a browser, tests in the browser was more important.

After a lot of discussions here and on #git-devel, I have used jasmine BDD
framework, nodejs and jasmine TAP reporter to solve the issues. The specs are
written for the browser and a makefile generates t/t9503-gitweb-javascript.js
which can be run in a console to produce TAP output.


This patch series is for review only. If everyone is ok with the new testing
framework, I will be able to send a patch with actual code in a week or 2. This
patch series adds few sample tests to the existing code, and makes it run in a
browser and console.

I have added gitweb/test/README with instructions on the new test.

Please eyeball the changes to t/Makefile as I am not very sure if that is the
best way to do it.

Reviews most welcome.



-- 
Jaseem Abid
http://jaseemabid.github.com
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]