@robocoder opened this Issue on July 22nd 2010 Contributor

If the json extension is disabled (e.g., --disable-json) or missing (e.g., for php 5.1.x, json is an optional PECL extension), the unit tests fail due to differences in the compatibility function. (One of the failures may be a bug.)

@robocoder commented on July 22nd 2010 Contributor

Attachment:
all_tests.html

@mattab commented on July 22nd 2010 Member

It looks like all tests but 1 are failign because of encoding differences of utf8 characters, which is OK

one bug however fails because numeric indexes are not exported in the array, as expected

string(150) "{"Limitation":["Multi dimensional arrays is only supported by format=JSON","Known limitation"],"Second Dimension":[true,false,1,0,152,"test",["end"]]}"

string(155) "{"Limitation":["Multi dimensional arrays is only supported by format=JSON","Known limitation"],"Second Dimension":[true,false,1,0,152,"test",{"42":"end"}]}"

very low priority bug as all API arrays should never rely on numeric indexes anyway. vote for wontfix...

@robocoder commented on July 23rd 2010 Contributor

We'll have to figure out a workaround. When Hudson is fully configured, the php 5.1.x builds will be continuously failing as a result, and at the moment, Piwik claims to still support php 5.1.3 and above.

@mattab commented on July 23rd 2010 Member

for the numeric index, i'll remove this test case from the array.

can you think of a solution to encoding issues? otherwise, we can hijack these tests for this specific build ;)

@mattab commented on July 23rd 2010 Member

because don't get me wrong, I want hudson sunny before 1.0 :)

@robocoder commented on October 4th 2010 Contributor

(In [3230]) fixes #1498

@robocoder commented on October 5th 2010 Contributor

(In [3236]) refs #1498 - really fix it this time

This Issue was closed on October 5th 2010
Powered by GitHub Issue Mirror