[issue18273] Simplify calling and discovery of json test package

2013-08-08 Thread Ezio Melotti
Ezio Melotti added the comment: Fixed, thanks for the patch! (Thanks Ned too!) -- assignee: -> ezio.melotti resolution: -> fixed stage: patch review -> committed/rejected status: open -> closed ___ Python tracker

[issue18273] Simplify calling and discovery of json test package

2013-08-08 Thread Roundup Robot
Roundup Robot added the comment: New changeset 95cf8640b271 by Ezio Melotti in branch '3.3': #18273: move the tests in Lib/test/json_tests to Lib/test/test_json and make them discoverable by unittest. Patch by Zachary Ware. http://hg.python.org/cpython/rev/95cf8640b271 New changeset f7ed301e71

[issue18273] Simplify calling and discovery of json test package

2013-08-07 Thread Zachary Ware
Zachary Ware added the comment: Here's a new patch; test_json.__main__ now uses an absolute import rather than trying to use a relative one. Also, Makefile.pre.in is fixed (thank you, Ned!). -- Added file: http://bugs.python.org/file31190/test_json_discovery.v2-3.3.diff __

[issue18273] Simplify calling and discovery of json test package

2013-08-05 Thread Ned Deily
Ned Deily added the comment: Make sure the tests still work when run from an installed Python rather than just from a build directory. In particular, if you rename test directories, you will need to change LIBSUBDIRS in Makefile.pre.in to ensure the directories and their files are installed.

[issue18273] Simplify calling and discovery of json test package

2013-08-05 Thread Ezio Melotti
Ezio Melotti added the comment: I like the patch. Can you make "./python Lib/test/test_json/" work too? Currently it doesn't seem to work (it works for e.g. "./python Lib/test/test_email/"). -- stage: -> patch review ___ Python tracker

[issue18273] Simplify calling and discovery of json test package

2013-06-20 Thread Zachary Ware
New submission from Zachary Ware: Technically, test discovery already works for test_json.py / json_tests, but not the way really expected (each test file is discovered individually), and not as simply as it could. The attached patch does the following: - remove test_json.py - rename json_tes