[issue9607] Test file 'test_keyword.py' submission for use with keyword.py

2013-04-24 Thread Benjamin Peterson
Benjamin Peterson added the comment: See #17830 -- status: open -> closed ___ Python tracker ___ ___ Python-bugs-list mailing list Unsu

[issue9607] Test file 'test_keyword.py' submission for use with keyword.py

2013-04-24 Thread Antoine Pitrou
Antoine Pitrou added the comment: Tests broken under Windows: == FAIL: test_keywords_py_without_markers_produces_error (test.test_keyword.TestKeywordGeneration) -

[issue9607] Test file 'test_keyword.py' submission for use with keyword.py

2013-04-22 Thread Éric Araujo
Éric Araujo added the comment: Just wanted to note that addCleanup is smarter than many core devs think: self.addCleanup(lambda: setattr(keyword, 'kwlist', oldlist)) self.addCleanup(setattr, keyword, 'kwlist', oldlist) -- nosy: +eric.araujo ___ Pyth

[issue9607] Test file 'test_keyword.py' submission for use with keyword.py

2013-04-19 Thread Roundup Robot
Roundup Robot added the comment: New changeset d499189e7758 by R David Murray in branch 'default': #9607: restore keywords.kwlist after testing it. http://hg.python.org/cpython/rev/d499189e7758 -- ___ Python tracker

[issue9607] Test file 'test_keyword.py' submission for use with keyword.py

2013-04-19 Thread R. David Murray
R. David Murray added the comment: Wow, it sure took me a long time to get back to this. Greg, the test ideas in your patch were great, and I used them all, but I wound up rewriting the actual code to better match the style of our other unit tests, and also to take advantage of some of our te

[issue9607] Test file 'test_keyword.py' submission for use with keyword.py

2013-04-19 Thread Roundup Robot
Roundup Robot added the comment: New changeset 0ad18bb85371 by R David Murray in branch 'default': #9607: Add tests for the keyword module. http://hg.python.org/cpython/rev/0ad18bb85371 -- nosy: +python-dev ___ Python tracker

[issue9607] Test file 'test_keyword.py' submission for use with keyword.py

2010-09-06 Thread Greg Malcolm
Greg Malcolm added the comment: Thanks for the feedback David! I've replaced the old patch with a new version that uses Popen/sys.executable as suggested. - Greg -- Added file: http://bugs.python.org/file18769/test_keyword_v2.patch ___ Python track

[issue9607] Test file 'test_keyword.py' submission for use with keyword.py

2010-09-06 Thread Greg Malcolm
Changes by Greg Malcolm : Removed file: http://bugs.python.org/file18537/test_keyword.patch ___ Python tracker ___ ___ Python-bugs-list mailing

[issue9607] Test file 'test_keyword.py' submission for use with keyword.py

2010-08-25 Thread R. David Murray
R. David Murray added the comment: Greg, thanks for working on this. I haven't looked over the whole patch yet, but there's one thing I noticed right away that needs to be fixed. 'python.exe' is the name of the python executable only on OS X, and the tests are not always run from the build

[issue9607] Test file 'test_keyword.py' submission for use with keyword.py

2010-08-15 Thread Antoine Pitrou
Changes by Antoine Pitrou : -- nosy: +benjamin.peterson ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://m

[issue9607] Test file 'test_keyword.py' submission for use with keyword.py

2010-08-14 Thread Greg Malcolm
New submission from Greg Malcolm : 'keyword.py' didn't have any tests, so I wrote some. Most of the tests are are for the main() method, which self-populates the keywords section of keyword.py with keywords taken from a grammar file, 'Python/graminit.c'. The main() method allows you to choose