Implement ability for JSON implementation to automatically register convertors

Proposed patch for this issue

getting some test failures after applying that patch?

-------------------------------------------------------------------------------
Test set: org.mortbay.util.ajax.JSONTest
-------------------------------------------------------------------------------
Tests run: 7, Failures: 2, Errors: 0, Skipped: 0, Time elapsed: 0.02 sec <<< FAILURE!
testToString(org.mortbay.util.ajax.JSONTest) Time elapsed: 0.004 sec <<< FAILURE!
junit.framework.AssertionFailedError
at junit.framework.Assert.fail(Assert.java:47)
at junit.framework.Assert.assertTrue(Assert.java:20)
at junit.framework.Assert.assertTrue(Assert.java:27)
at org.mortbay.util.ajax.JSONTest.testToString(JSONTest.java:83)

testConvertor(org.mortbay.util.ajax.JSONTest) Time elapsed: 0.002 sec <<< FAILURE!
junit.framework.AssertionFailedError
at junit.framework.Assert.fail(Assert.java:47)
at junit.framework.Assert.assertTrue(Assert.java:20)
at junit.framework.Assert.assertTrue(Assert.java:27)
at org.mortbay.util.ajax.JSONTest.testConvertor(JSONTest.java:279)

This is interesting, i have just re-run all three JUnit test suites separately, and they worked fine for me. But if I run all of them together by selecting the package in Eclipse, when I get the same assertion as the first one you mentioned, but I don't get the second one.

I stand corrected, the second one errors as well.

I think I know what is happening. When running all tests together, the JRE is not reloaded after each test. Since the JSONPojoConvertorFactory test runs before JSONTest, and because it sets the factory convertor in default JSON instance, it affects all the subsequent tests.

Changed JSONPojoConvertorFactoryTest to set the factory convertor only for current instance.

Applied - thanks!

ps. normally a patch should patch the VERSION.txt file as well.