Make test_importer dry runs show warnings when control files are missing required variables
Signed-off-by: Scott Zawalski <[email protected]> --- autotest/utils/test_importer.py 2010-04-30 07:29:44.000000000 -0700 +++ autotest/utils/test_importer.py 2010-04-30 13:40:09.000000000 -0700 @@ -177,7 +177,7 @@ if add_noncompliant: doc = description else: - logging.info("Skipping %s, missing docstring", profiler) + logging.warn("Skipping %s, missing docstring", profiler) continue else: doc = profilers[profiler] @@ -346,8 +346,7 @@ raise_warnings=True) tests[file] = found_test except control_data.ControlVariableException, e: - logging.info("Skipping %s\n%s", file, e) - pass + logging.warn("Skipping %s\n%s", file, e) except Exception, e: logging.error("Bad %s\n%s", file, e) else: @@ -438,8 +437,7 @@ raise_warnings=True) tests[file_path] = found_test except control_data.ControlVariableException, e: - logging.info("Skipping %s\n%s", file, e) - pass + logging.warn("Skipping %s\n%s", file, e) else: profilers[file_path] = compiler.parseFile(file_path).doc @@ -502,6 +500,9 @@ help='Autotest directory root') options, args = parser.parse_args() DRY_RUN = options.dry_run + if DRY_RUN: + logging.getLogger().setLevel(logging.WARN) + # Make sure autotest_dir is the absolute path options.autotest_dir = os.path.abspath(options.autotest_dir) _______________________________________________ Autotest mailing list [email protected] http://test.kernel.org/cgi-bin/mailman/listinfo/autotest
