Extract TestConfiguration from base and write a few unit tests to capture current behavior.
Created attachment 101811 [details] Patch
Comment on attachment 101811 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=101811&action=review > Tools/Scripts/webkitpy/layout_tests/models/test_configuration.py:55 > + def all_test_configurations(self): @classmethod ? > Tools/Scripts/webkitpy/layout_tests/models/test_configuration.py:70 > + def all_systems(self): @classmethod ? > Tools/Scripts/webkitpy/layout_tests/models/test_configuration.py:79 > + def all_build_types(self): @classmethod ? > Tools/Scripts/webkitpy/layout_tests/models/test_configuration.py:82 > + def all_graphics_types(self): @classmethod ?
(In reply to comment #2) > (From update of attachment 101811 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=101811&action=review > > > Tools/Scripts/webkitpy/layout_tests/models/test_configuration.py:55 > > + def all_test_configurations(self): > > @classmethod ? Yes. > > > Tools/Scripts/webkitpy/layout_tests/models/test_configuration.py:70 > > + def all_systems(self): > > @classmethod ? Yes! > > > Tools/Scripts/webkitpy/layout_tests/models/test_configuration.py:79 > > + def all_build_types(self): > > @classmethod ? Yes!! > > > Tools/Scripts/webkitpy/layout_tests/models/test_configuration.py:82 > > + def all_graphics_types(self): > > @classmethod ? Oh, YES!!!
Comment on attachment 101811 [details] Patch Actually, these methods shouldn't be part of TestConfiguration at all. They would be port-dependent and should hang off the port object. I'll move them in a separate patch.
Comment on attachment 101811 [details] Patch Clearing flags on attachment: 101811 Committed r91639: <http://trac.webkit.org/changeset/91639>
All reviewed patches have been landed. Closing bug.