[pinpoint] Strip '.git' from the end of repository URLs in DEPS files.

dashboard/dashboard/pinpoint/models/change_test.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 mock
 
 from dashboard.common import namespaced_stored_object
 from dashboard.common import testing_common
@@ skipping 91 matching lines @@
         return [{'commit': 'babe852'}]
       if first_git_hash == '0000000' and last_git_hash == '2222222':
         return [{'commit': '2222222'}, {'commit': '1111111'}]
       raise NotImplementedError()
     commit_range.side_effect = _CommitRange
 
     def _FileContents(repository_url, git_hash, path):
       del repository_url
       del path
       if git_hash == '0e57e2b':
-        return 'deps = {"src/catapult": "%s@0000000"}' % _CATAPULT_URL
+        # Also test the stripping of '.git' from repository URLs.
+        return 'deps = {"src/catapult": "%s@0000000"}' % (
+            _CATAPULT_URL + '.git')
       if git_hash == 'babe852':
         return 'deps = {"src/catapult": "%s@2222222"}' % _CATAPULT_URL
       raise NotImplementedError()
     file_contents.side_effect = _FileContents
 
     change_a = change.Change(change.Dep('src', '0e57e2b'))
     change_b = change.Change(change.Dep('src', 'babe852'))
     expected = change.Change(change.Dep('src', '0e57e2b'),
                              (change.Dep('catapult', '1111111'),))
 
@@ skipping 228 matching lines @@
   def testFromDict(self):
     patch = change.Patch.FromDict({
         'server': 'https://codereview.chromium.org',
         'issue': 2851943002,
         'patchset': 40001,
     })
 
     expected = change.Patch('https://codereview.chromium.org',
                             2851943002, 40001)
     self.assertEqual(patch, expected)