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

Unified Diff: telemetry/telemetry/core/discover_unittest.py

Issue 2941123002: [Telemetry] Move discover to common/py_utils (Closed)
Patch Set: fix imports Created 3 years, 6 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 | « telemetry/telemetry/core/discover.py ('k') | telemetry/telemetry/core/platform.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: telemetry/telemetry/core/discover_unittest.py
diff --git a/telemetry/telemetry/core/discover_unittest.py b/telemetry/telemetry/core/discover_unittest.py
deleted file mode 100644
index 15b0736a7737ccc88098b5c28f132ca130f45749..0000000000000000000000000000000000000000
--- a/telemetry/telemetry/core/discover_unittest.py
+++ /dev/null
@@ -1,147 +0,0 @@
-# Copyright 2013 The Chromium Authors. All rights reserved.
-# Use of this source code is governed by a BSD-style license that can be
-# found in the LICENSE file.
-import os
-import unittest
-
-from telemetry.core import discover
-from telemetry.core import util
-
-
-class DiscoverTest(unittest.TestCase):
-
- def setUp(self):
- self._base_dir = util.GetUnittestDataDir()
- self._start_dir = os.path.join(self._base_dir, 'discoverable_classes')
- self._base_class = Exception
-
- def testDiscoverClassesWithIndexByModuleName(self):
- classes = discover.DiscoverClasses(self._start_dir,
- self._base_dir,
- self._base_class,
- index_by_class_name=False)
-
- actual_classes = dict((name, cls.__name__)
- for name, cls in classes.iteritems())
- expected_classes = {
- 'another_discover_dummyclass': 'DummyExceptionWithParameterImpl1',
- 'discover_dummyclass': 'DummyException',
- 'parameter_discover_dummyclass': 'DummyExceptionWithParameterImpl2'
- }
- self.assertEqual(actual_classes, expected_classes)
-
- def testDiscoverDirectlyConstructableClassesWithIndexByClassName(self):
- classes = discover.DiscoverClasses(self._start_dir,
- self._base_dir,
- self._base_class,
- directly_constructable=True)
-
- actual_classes = dict((name, cls.__name__)
- for name, cls in classes.iteritems())
- expected_classes = {
- 'dummy_exception': 'DummyException',
- 'dummy_exception_impl1': 'DummyExceptionImpl1',
- 'dummy_exception_impl2': 'DummyExceptionImpl2',
- }
- self.assertEqual(actual_classes, expected_classes)
-
- def testDiscoverClassesWithIndexByClassName(self):
- classes = discover.DiscoverClasses(self._start_dir, self._base_dir,
- self._base_class)
-
- actual_classes = dict((name, cls.__name__)
- for name, cls in classes.iteritems())
- expected_classes = {
- 'dummy_exception': 'DummyException',
- 'dummy_exception_impl1': 'DummyExceptionImpl1',
- 'dummy_exception_impl2': 'DummyExceptionImpl2',
- 'dummy_exception_with_parameter_impl1':
- 'DummyExceptionWithParameterImpl1',
- 'dummy_exception_with_parameter_impl2':
- 'DummyExceptionWithParameterImpl2'
- }
- self.assertEqual(actual_classes, expected_classes)
-
- def testDiscoverClassesWithPatternAndIndexByModule(self):
- classes = discover.DiscoverClasses(self._start_dir,
- self._base_dir,
- self._base_class,
- pattern='another*',
- index_by_class_name=False)
-
- actual_classes = dict((name, cls.__name__)
- for name, cls in classes.iteritems())
- expected_classes = {
- 'another_discover_dummyclass': 'DummyExceptionWithParameterImpl1'
- }
- self.assertEqual(actual_classes, expected_classes)
-
- def testDiscoverDirectlyConstructableClassesWithPatternAndIndexByClassName(
- self):
- classes = discover.DiscoverClasses(self._start_dir,
- self._base_dir,
- self._base_class,
- pattern='another*',
- directly_constructable=True)
-
- actual_classes = dict((name, cls.__name__)
- for name, cls in classes.iteritems())
- expected_classes = {
- 'dummy_exception_impl1': 'DummyExceptionImpl1',
- 'dummy_exception_impl2': 'DummyExceptionImpl2',
- }
- self.assertEqual(actual_classes, expected_classes)
-
- def testDiscoverClassesWithPatternAndIndexByClassName(self):
- classes = discover.DiscoverClasses(self._start_dir,
- self._base_dir,
- self._base_class,
- pattern='another*')
-
- actual_classes = dict((name, cls.__name__)
- for name, cls in classes.iteritems())
- expected_classes = {
- 'dummy_exception_impl1': 'DummyExceptionImpl1',
- 'dummy_exception_impl2': 'DummyExceptionImpl2',
- 'dummy_exception_with_parameter_impl1':
- 'DummyExceptionWithParameterImpl1',
- }
- self.assertEqual(actual_classes, expected_classes)
-
-
-class ClassWithoutInitDefOne: # pylint: disable=old-style-class, no-init
- pass
-
-
-class ClassWithoutInitDefTwo(object):
- pass
-
-
-class ClassWhoseInitOnlyHasSelf(object):
- def __init__(self):
- pass
-
-
-class ClassWhoseInitWithDefaultArguments(object):
- def __init__(self, dog=1, cat=None, cow=None, fud='a'):
- pass
-
-
-class ClassWhoseInitWithDefaultArgumentsAndNonDefaultArguments(object):
- def __init__(self, x, dog=1, cat=None, fish=None, fud='a'):
- pass
-
-
-class IsDirectlyConstructableTest(unittest.TestCase):
-
- def testIsDirectlyConstructableReturnsTrue(self):
- self.assertTrue(discover.IsDirectlyConstructable(ClassWithoutInitDefOne))
- self.assertTrue(discover.IsDirectlyConstructable(ClassWithoutInitDefTwo))
- self.assertTrue(discover.IsDirectlyConstructable(ClassWhoseInitOnlyHasSelf))
- self.assertTrue(
- discover.IsDirectlyConstructable(ClassWhoseInitWithDefaultArguments))
-
- def testIsDirectlyConstructableReturnsFalse(self):
- self.assertFalse(
- discover.IsDirectlyConstructable(
- ClassWhoseInitWithDefaultArgumentsAndNonDefaultArguments))
« no previous file with comments | « telemetry/telemetry/core/discover.py ('k') | telemetry/telemetry/core/platform.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698