Saimadhav Heblikar added the comment:
test_DirBrowserTreeItem (idlelib.idle_test.test_pathbrowser.PathBrowserTest)
... ok
test_PathBrowserTreeItem (idlelib.idle_test.test_pathbrowser.PathBrowserTest)
... ok
--
Ran 2 tests in 0
Saimadhav Heblikar added the comment:
Well,thank you for the feedback to both .
I will try to make a new patch , during this weekend,
removing the name changes
to add docstrings
and add a human testable dialog for pathbrowser,as is present in other tests.
--
__
Terry J. Reedy added the comment:
We do not usually make pure style changes in a file unless the code is being
reviewed and edited. Even then, I focus on changes that make the file easier to
read and understand, as is necessary to write tests. One example is adding
missing spaces after commas
Steven D'Aprano added the comment:
I think you may have misread PEP 8. It does not recommend a trailing underscore
for names that shadow built-ins (e.g. file_ instead of file), it only
recommends a trailing underscore when you need to use a keyword as a name (e.g.
class_ instead of class). Sha
New submission from Saimadhav Heblikar:
This patch does
1.Remove pep8 violations in PathBrowser.py . Replaces "file","dir","sorted" by
"file_","dir_","sorted_" respectively.
2.Extends test coverage for PathBrowser.py in idle_test/test_PathBrowser.py
New modules now under tests include
1.dirBro
