Extract patchset in Rietveld URLs better

tests/git_cl_test.py

 #!/usr/bin/env python
 # Copyright (c) 2012 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.
 
 """Unit tests for git_cl.py."""
 
 import os
 import StringIO
 import stat
@@ skipping 127 matching lines @@
         self.assertEqual(result.patch_url, patch_url)
 
     test('http://codereview.chromium.org/123',
          123, None, 'codereview.chromium.org')
     test('https://codereview.chromium.org/123',
          123, None, 'codereview.chromium.org')
     test('https://codereview.chromium.org/123/',
          123, None, 'codereview.chromium.org')
     test('https://codereview.chromium.org/123/whatever',
          123, None, 'codereview.chromium.org')
+    test('https://codereview.chromium.org/123/#ps20001',
+         123, 20001, 'codereview.chromium.org')
     test('http://codereview.chromium.org/download/issue123_4.diff',
          123, 4, 'codereview.chromium.org',
          patch_url='https://codereview.chromium.org/download/issue123_4.diff')
     # This looks like bad Gerrit, but is actually valid Rietveld.
     test('https://chrome-review.source.com/123/4/',
          123, None, 'chrome-review.source.com')
 
     test('https://codereview.chromium.org/deadbeaf', fail=True)
     test('https://codereview.chromium.org/api/123', fail=True)
     test('bad://codereview.chromium.org/123', fail=True)
@@ skipping 1590 matching lines @@
         ((['rm_file_or_tree', '/abs/git_repo_root/.git/hooks/commit-msg'],),
          ''),
     ]
     cl._codereview_impl._GerritCommitMsgHookCheck(offer_removal=True)
 
 
 if __name__ == '__main__':
   git_cl.logging.basicConfig(
       level=git_cl.logging.DEBUG if '-v' in sys.argv else git_cl.logging.ERROR)
   unittest.main()