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

Unified Diff: Tools/Scripts/webkitpy/common/system/platforminfo_unittest.py

Issue 454223003: Remove the concept of 'integration tests' from test-webkitpy. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: merge forward, remove dead / uneeded code Created 6 years, 4 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
« no previous file with comments | « no previous file | Tools/Scripts/webkitpy/layout_tests/run_webkit_tests_integrationtest.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Tools/Scripts/webkitpy/common/system/platforminfo_unittest.py
diff --git a/Tools/Scripts/webkitpy/common/system/platforminfo_unittest.py b/Tools/Scripts/webkitpy/common/system/platforminfo_unittest.py
index 24fa35210a6c368bb98122de1c978889587c1773..e803131339d470924b436434a69b33c807a47100 100644
--- a/Tools/Scripts/webkitpy/common/system/platforminfo_unittest.py
+++ b/Tools/Scripts/webkitpy/common/system/platforminfo_unittest.py
@@ -70,8 +70,6 @@ class TestPlatformInfo(unittest.TestCase):
def make_info(self, sys_module=None, platform_module=None, executive=None):
return PlatformInfo(sys_module or fake_sys(), platform_module or fake_platform(), executive or fake_executive())
- # FIXME: This should be called integration_test_real_code(), but integration tests aren't
- # yet run by default and there's no reason not to run this everywhere by default.
def test_real_code(self):
# This test makes sure the real (unmocked) code actually works.
info = PlatformInfo(sys, platform, Executive())
« no previous file with comments | « no previous file | Tools/Scripts/webkitpy/layout_tests/run_webkit_tests_integrationtest.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698