Mailing List Archive: Python: Bugs

[issue7274] json module apparently fails regression found at http://json.org/JSON_checker/test.zip.

Index | Next | Previous | View Threaded

report at bugs

Nov 6, 2009, 10:11 AM

Post #1 of 4 (195 views)
Permalink

[issue7274] json module apparently fails regression found at http://json.org/JSON_checker/test.zip.

New submission from Douglas Shawhan <doug.shawhan [at] gmail>:

Environment:
Ubuntu 9.04 Jaunty
Linux thebadpipsissewah 2.6.28-16-server #55-Ubuntu SMP Tue Oct 20
20:37:10 UTC 2009 x86_64 GNU/Linux
Python 2.6.2 (release26-maint, Apr 19 2009, 01:58:18)
[GCC 4.3.3] on linux2

Passes two "fail" cases:
fail1.json "A JSON payload should be an object or array, not a string."
fail18.json [[[[[[[[[[[[[[[[[[[["Too deep"]]]]]]]]]]]]]]]]]]]]

All other cases pass or fail as indicated.

=== test ===

#!/usr/bin/env python

import json, os
from StringIO import StringIO

def Validate( case ):
try:
l = json.loads( case[1] )
print( "%s %s"%case )
except:
pass

try:
io = StringIO( case[1] )
json.load( io )
print( "%s %s"%case )
except:
pass

for passfail in [ "fail" ]:
cases = [. [.( i, open( os.path.join( base, i ), "r").read() )
for i in files if passfail in i ]
for base, subs, files in os.walk( "tests" ) ][0]

[ Validate( case ) for case in cases ]

Sorry for ugly code, in a hurry. If this is wrong, please feel free to
backhand me.

d

----------
components: Extension Modules
messages: 94990
nosy: DougShawhan
severity: normal
status: open
title: json module apparently fails regression found at http://json.org/JSON_checker/test.zip.
type: behavior
versions: Python 2.6

_______________________________________
Python tracker <report [at] bugs>
<http://bugs.python.org/issue7274>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com

report at bugs

Nov 6, 2009, 1:40 PM

Post #2 of 4 (177 views)
Permalink

[issue7274] json module apparently fails regression found at http://json.org/JSON_checker/test.zip. [In reply to]

Changes by Benjamin Peterson <benjamin [at] python>:

----------
assignee: -> bob.ippolito
nosy: +bob.ippolito

_______________________________________
Python tracker <report [at] bugs>
<http://bugs.python.org/issue7274>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com

report at bugs

Nov 6, 2009, 2:45 PM

Post #3 of 4 (173 views)
Permalink

[issue7274] json module apparently fails regression found at http://json.org/JSON_checker/test.zip. [In reply to]

Bob Ippolito <bob [at] redivi> added the comment:

The json module departs slightly from the spec in that it will allow you
to consume and produce documents that don't have an array or object
envelope. This is a feature.

The "Too deep" test case is just a silly thing that crockford's code
does, there's a fixed limitation for how complex an object could be.
That's not a regression failure and the spec doesn't say anything about
a maximum nesting level.

The json module includes all of these in its test suite, although it
disagrees slightly with crockford on what should pass.

http://svn.python.org/view/python/branches/release26-
maint/Lib/json/tests/test_fail.py

SKIPS = {
1: "why not have a string payload?",
18: "spec doesn't specify any nesting limitations",
}

----------
resolution: -> invalid
status: open -> closed

_______________________________________
Python tracker <report [at] bugs>
<http://bugs.python.org/issue7274>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com

report at bugs

Nov 8, 2009, 6:53 PM

Post #4 of 4 (163 views)
Permalink

[issue7274] json module apparently fails regression found at http://json.org/JSON_checker/test.zip. [In reply to]

Douglas Shawhan <doug.shawhan [at] gmail> added the comment:

Fair enough. Thanks.

On Fri, Nov 6, 2009 at 5:45 PM, Bob Ippolito <report [at] bugs> wrote:

>
> Bob Ippolito <bob [at] redivi> added the comment:
>
> The json module departs slightly from the spec in that it will allow you
> to consume and produce documents that don't have an array or object
> envelope. This is a feature.
>
> The "Too deep" test case is just a silly thing that crockford's code
> does, there's a fixed limitation for how complex an object could be.
> That's not a regression failure and the spec doesn't say anything about
> a maximum nesting level.
>
> The json module includes all of these in its test suite, although it
> disagrees slightly with crockford on what should pass.
>
> http://svn.python.org/view/python/branches/release26-
> maint/Lib/json/tests/test_fail.py<http://svn.python.org/view/python/branches/release26-%0Amaint/Lib/json/tests/test_fail.py>
>
> SKIPS = {
> 1: "why not have a string payload?",
> 18: "spec doesn't specify any nesting limitations",
> }
>
> ----------
> resolution: -> invalid
> status: open -> closed
>
> _______________________________________
> Python tracker <report [at] bugs>
> <http://bugs.python.org/issue7274>
> _______________________________________
>

----------
Added file: http://bugs.python.org/file15293/unnamed

_______________________________________
Python tracker <report [at] bugs>
<http://bugs.python.org/issue7274>
_______________________________________

Attachments:

unnamed (1.83 KB)

Index | Next | Previous | View Threaded

Interested in having your list archived? Contact Gossamer Threads