Merge ~twom/launchpad:export-git-id-on-api into launchpad:master

[Tom Wardill]

Check that ID is present, but not it's value

We don't have the repository at the point of checking it's ID, and all we really need to do is ensure that the ID field is actually exported.
Check it's there, and that it's an int.

[Thiago F. Pappacena (pappacena)]

LGTM

review: Approve

[Kristian Glass (doismellburning)]

review: Approve

diff --git a/lib/lp/code/model/tests/test_gitrepository.py b/lib/lp/code/model/tests/test_gitrepository.py
index d9017e2..5408a40 100644
--- a/lib/lp/code/model/tests/test_gitrepository.py
+++ b/lib/lp/code/model/tests/test_gitrepository.py
@@ -34,6 +34,7 @@ from testtools.matchers import (
     EndsWith,
     Equals,
     Is,
+    IsInstance,
     LessThan,
     MatchesDict,
     MatchesListwise,
@@ -3731,7 +3732,7 @@ class TestGitRepositoryWebservice(TestCaseWithFactory):
         self.assertEqual(201, response.status)
         repository = webservice.get(response.getHeader("Location")).jsonBody()
         self.assertThat(repository, ContainsDict({
-            "id": Equals(1),
+            "id": IsInstance(int),
             "repository_type": Equals("Hosted"),
             "registrant_link": EndsWith(owner_url),
             "owner_link": EndsWith(owner_url),