Skip Menu |
 

History Show all quoted text — Show full headers
# Fri Dec 04 16:55:37 2015 sarahday (Sarah Day) - Ticket created
Subject: jsonwalker.py test fails
Download (untitled) / with headers
text/plain 487B
Running make check-pytests in the tests folder fails in jsonwalker.py

PYTHONPATH=../../src/util VALGRIND="" python
../../src/tests/jsonwalker.py -d ../../src/tests/au_dict.json \
-i au.log
Traceback (most recent call last):
File "../../src/tests/jsonwalker.py", line 107, in <module>
defaults = cjson.decode(f.read())
cjson.DecodeError: expecting ',' or '}' at position 529
Makefile:727: recipe for target 'check-pytests' failed
make: *** [check-pytests] Error 1
# Fri Dec 04 17:01:31 2015 sarahday (Sarah Day) - Status changed from 'open' to 'review'
# Fri Dec 04 17:01:31 2015 sarahday (Sarah Day) - Comments added
Download (untitled) / with headers
text/plain 32B
Has been sent as a pull request.
# Wed Dec 23 12:54:29 2015 ghudson@mit.edu (Greg Hudson) - Correspondence added
From: ghudson@mit.edu
Subject: git commit
Download (untitled) / with headers
text/plain 606B

Fix a typo that causes the jsonwalker test to fail

In the au_dict.json file that is used for testing jsonwalker, there is
a comma missing that causes cjson 1.1 to throw an exception. This
causes the python tests to fail. Adding the comma fixes the crash and
allows the test to finish successfully.

https://github.com/krb5/krb5/commit/d7fcd6d068f61a209a2c363f08544db750ff3522
Author: Sarah Day <sarahday@mit.edu>
Committer: Greg Hudson <ghudson@mit.edu>
Commit: d7fcd6d068f61a209a2c363f08544db750ff3522
Branch: master
src/tests/au_dict.json | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
# Thu Oct 13 16:45:42 2016 tlyu (Taylor Yu) - Status changed from 'review' to 'resolved'
# Thu Oct 13 16:45:42 2016 tlyu (Taylor Yu) - Version_Fixed 1.15 added