[issue15273] Remove unnecessarily random behavior from test_unparse.py

Pablo Galindo Salgado added the comment: > I guess this resolved with PR 17738 Yup. Closing as per PR 17738. -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker __

[issue15273] Remove unnecessarily random behavior from test_unparse.py

Batuhan added the comment: I guess this resolved with PR 17738 -- ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscr

[issue15273] Remove unnecessarily random behavior from test_unparse.py

Pablo Galindo Salgado added the comment: > @pablogsal should we add Lib/test/test_grammar by default to every test? Also > if this is still needed @Daniel.Cioata do you still want to prepare another > patch for current version? I think is a good idea, let me check how complete is the current

[issue15273] Remove unnecessarily random behavior from test_unparse.py

Batuhan added the comment: @pablogsal should we add Lib/test/test_grammar by default to every test? Also if this is still needed @Daniel.Cioata do you still want to prepare another patch for current version? -- nosy: +BTaskaya, pablogsal ___ Pytho

[issue15273] Remove unnecessarily random behavior from test_unparse.py

Changes by Berker Peksag : -- components: +Tests nosy: +berker.peksag stage: needs patch -> patch review versions: +Python 3.5 ___ Python tracker ___

[issue15273] Remove unnecessarily random behavior from test_unparse.py

Daniel Cioata added the comment: a solution for this issue -- nosy: +Daniel.Cioata Added file: http://bugs.python.org/file28877/submitted_patch_Issue15273 ___ Python tracker ___

[issue15273] Remove unnecessarily random behavior from test_unparse.py

Benjamin Peterson added the comment: test_grammar should be that file, though, people often forget to update it. -- nosy: +benjamin.peterson ___ Python tracker ___ _

[issue15273] Remove unnecessarily random behavior from test_unparse.py

R. David Murray added the comment: I thought we had other tests that did this as well (pickle?). If you need reproducibility you check the random seed and reuse it. -- nosy: +r.david.murray ___ Python tracker __

[issue15273] Remove unnecessarily random behavior from test_unparse.py

Mark Dickinson added the comment: Ideally, it would pick a small number of files that are likely to fully exercise Python's grammar. So things like Lib/test/test_grammar might be useful to include. Or perhaps there should be a Python file somewhere in Lib/test whose entire purpose is to con

[issue15273] Remove unnecessarily random behavior from test_unparse.py

Changes by Mark Dickinson : -- assignee: mark.dickinson -> keywords: +easy versions: +Python 3.4 -Python 3.3 ___ Python tracker ___ _

[issue15273] Remove unnecessarily random behavior from test_unparse.py

New submission from Larry Hastings : Tools/parser/test_unparse.py is the regression test suite for Tools/unparse. To save time, if the "cpu" resource is not enabled it only test unparsing ten files. However it picks these files at random. It would be nice if the test was 100% repeatable. It