Print commit shas when there are exportable and not exported commits.

third_party/WebKit/Tools/Scripts/webkitpy/w3c/test_importer_unittest.py

 # Copyright 2016 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 unittest
+import collections
 
 from webkitpy.common.checkout.scm.git_mock import MockGit
 from webkitpy.common.host_mock import MockHost
 from webkitpy.common.system.executive_mock import MockExecutive
+from webkitpy.common.system.log_testing import LoggingTestCase
 from webkitpy.w3c.test_importer import TestImporter
 
 
-class TestImporterTest(unittest.TestCase):
+MockChromiumCommit = collections.namedtuple('ChromiumCommit', ('sha', 'position'))
+
+
+class TestImporterTest(LoggingTestCase):
 
     def test_abort_on_exportable_commits(self):
         importer = TestImporter(MockHost())
-        importer.exportable_but_not_exported_commits = lambda _: ['aaaa']
+        importer.exportable_but_not_exported_commits = lambda _: [
+            MockChromiumCommit(sha='deadbeef', position=123)]
+        importer.checkout_is_okay = lambda _: True
         return_code = importer.main(['wpt'])
         self.assertEqual(return_code, 1)
+        self.assertLog([
+            'INFO: Noting the current Chromium commit.\n',
+            ('INFO: Cloning https://chromium.googlesource.com/external/w3c/web-platform-tests.git '
+             'into /mock-checkout/third_party/WebKit/wpt.\n'),
+            'ERROR: There were exportable but not-yet-exported commits:\n',
+            'ERROR:   https://chromium.googlesource.com/chromium/src/+/deadbeef\n',
+            'ERROR: Aborting import to prevent clobbering these commits.\n'
+        ])
 
     def test_update_test_expectations(self):
         host = MockHost()
         host.filesystem.files['/mock-checkout/third_party/WebKit/LayoutTests/TestExpectations'] = (
             'Bug(test) some/test/a.html [ Failure ]\n'
             'Bug(test) some/test/b.html [ Failure ]\n'
             'Bug(test) some/test/c.html [ Failure ]\n')
         host.filesystem.files['/mock-checkout/third_party/WebKit/LayoutTests/VirtualTestSuites'] = '[]'
         host.filesystem.files['/mock-checkout/third_party/WebKit/LayoutTests/new/a.html'] = ''
         host.filesystem.files['/mock-checkout/third_party/WebKit/LayoutTests/new/b.html'] = ''
@@ skipping 112 matching lines @@
         self.assertEqual(importer.get_directory_owners(), {'someone@chromium.org': 'external/wpt/foo'})
 
     def test_get_directory_owners_no_changed_files(self):
         host = MockHost()
         host.filesystem.write_text_file(
             '/mock-checkout/third_party/WebKit/LayoutTests/W3CImportExpectations',
             '## Owners: someone@chromium.org\n'
             '# external/wpt/foo [ Pass ]\n')
         importer = TestImporter(host)
         self.assertEqual(importer.get_directory_owners(), {})