Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(446)

Unified Diff: third_party/closure_linter/closure_linter/full_test.py

Issue 411243002: closure_linter: 2.3.4 => 2.3.14 (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: remove checker Created 6 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/closure_linter/closure_linter/full_test.py
diff --git a/third_party/closure_linter/closure_linter/full_test.py b/third_party/closure_linter/closure_linter/full_test.py
index fde9c70b5344dd27ab4f05b3111e4812206fb71d..f2cf460b9be505457f8925a504a457da2c35fdee 100755
--- a/third_party/closure_linter/closure_linter/full_test.py
+++ b/third_party/closure_linter/closure_linter/full_test.py
@@ -23,7 +23,6 @@ devtools/javascript/gpylint/full_test.py
__author__ = ('robbyw@google.com (Robert Walker)',
'ajp@google.com (Andy Perelson)')
-import re
import os
import sys
import unittest
@@ -31,9 +30,9 @@ import unittest
import gflags as flags
import unittest as googletest
-from closure_linter import checker
-from closure_linter import errors
from closure_linter import error_check
+from closure_linter import errors
+from closure_linter import runner
from closure_linter.common import filetestcase
_RESOURCE_PREFIX = 'closure_linter/testdata'
@@ -52,6 +51,7 @@ _TEST_FILES = [
'all_js_wrapped.js',
'blank_lines.js',
'ends_with_block.js',
+ 'empty_file.js',
'externs.js',
'externs_jsdoc.js',
'goog_scope.js',
@@ -78,15 +78,17 @@ _TEST_FILES = [
'require_missing.js',
'require_numeric.js',
'require_provide_blank.js',
- 'require_provide_ok.js',
'require_provide_missing.js',
+ 'require_provide_ok.js',
+ 'semicolon_missing.js',
'simple.html',
'spaces.js',
'tokenizer.js',
'unparseable.js',
+ 'unused_local_variables.js',
'unused_private_members.js',
- 'utf8.html'
- ]
+ 'utf8.html',
+]
class GJsLintTestSuite(unittest.TestSuite):
@@ -106,8 +108,11 @@ class GJsLintTestSuite(unittest.TestSuite):
test_files = _TEST_FILES
for test_file in test_files:
resource_path = os.path.join(_RESOURCE_PREFIX, test_file)
- self.addTest(filetestcase.AnnotatedFileTestCase(resource_path,
- checker.GJsLintRunner(), errors.ByName))
+ self.addTest(
+ filetestcase.AnnotatedFileTestCase(
+ resource_path,
+ runner.Run,
+ errors.ByName))
if __name__ == '__main__':
# Don't let main parse args; it happens in the TestSuite.
« no previous file with comments | « third_party/closure_linter/closure_linter/fixjsstyle_test.py ('k') | third_party/closure_linter/closure_linter/gjslint.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698