New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 630665 link

Starred by 4 users

Issue metadata

Status: Archived
Owner:
Last visit > 30 days ago
Closed: Mar 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Bug



Sign in to add a comment

Autotest: run_pylint.py results in success even if file doesn't exist

Project Member Reported by bryanlew@chromium.org, Jul 22 2016

Issue description

In Autotest, the src/third_party/autotest/files/utils/run_pylint.py script returns 0 (success) and doesn't emit any error output if the argument file to be examined doesn't exist.

To reproduce:
Navigate to the autotest/files directory.
Execute ./utils/run_pylint.py SomePathAndFileThatDoesNotExist
Note that no output or errors are printed.
Execute echo $? 
Note that it returns 0 (success).

Expected behavior:
If directed to analyze a file that does not exist, an error should be returned.
 

Comment 1 by sbasi@chromium.org, Jul 22 2016

Cc: vapier@chromium.org

Comment 2 by dshi@chromium.org, Jul 26 2016

Labels: Hotlist-Fixit
i don't really have interest in the autotest wrapper, and `cros lint asdfasdf.py` seems to DTRT.  so maybe convert to cros lint ? :)
Owner: pho...@chromium.org
Status: Archived (was: Untriaged)

Sign in to add a comment