| msg164831 - (view) |
Author: Larry Hastings (larry) *  |
Date: 2012-07-07 12:29 |
| 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 might be nice if it picked the most tortuous ten files. What ten are the most tortuous is not immediately clear, but a nice first approximation might be to pick the largest ten, provided the stat calls don't themselves make the test slow again. |
|
|
| msg164832 - (view) |
Author: Mark Dickinson (mark.dickinson) * |
Date: 2012-07-07 12:31 |
| 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 contain at least one example of each grammar construction? |
|
|
| msg165026 - (view) |
Author: R. David Murray (r.david.murray) * |
Date: 2012-07-08 17:15 |
| I thought we had other tests that did this as well (pickle?). If you need reproducibility you check the random seed and reuse it. |
|
|
| msg165063 - (view) |
Author: Benjamin Peterson (benjamin.peterson) * |
Date: 2012-07-09 05:11 |
| test_grammar should be that file, though, people often forget to update it. |
|
|
| msg180838 - (view) |
Author: Daniel Cioata (Daniel.Cioata) |
Date: 2013-01-28 08:06 |
| a solution for this issue |
|
|
| msg357726 - (view) |
Author: Batuhan Taskaya (BTaskaya) * |
Date: 2019-12-02 18:24 |
| @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? |
|
|
| msg357729 - (view) |
Author: Pablo Galindo Salgado (pablogsal) * |
Date: 2019-12-02 21:45 |
| > @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 version first. |
|
|
| msg359205 - (view) |
Author: Batuhan Taskaya (BTaskaya) * |
Date: 2020-01-02 17:25 |
| I guess this resolved with PR 17738 |
|
|
| msg359208 - (view) |
Author: Pablo Galindo Salgado (pablogsal) * |
Date: 2020-01-02 18:22 |
| > I guess this resolved with PR 17738 Yup. Closing as per PR 17738. |
|
|