Improvements to fake_data.make_package

- use a constant
- return Package

gratipay/testing/harness.py

@@ -17,6 +17,7 @@
 from gratipay.exceptions import NoSelfTipping, NoTippee, BadAmount
 from gratipay.models.account_elsewhere import AccountElsewhere
 from gratipay.models.exchange_route import ExchangeRoute
+from gratipay.models.package import NPM, Package
 from gratipay.models.participant import Participant, MAX_TIP, MIN_TIP
 from gratipay.security import user
 from gratipay.testing.vcr import use_cassette
@@ -195,14 +196,15 @@ def make_team(self, *a, **kw):
         return team
 
 
-    def make_package(self, package_manager='npm', name='foo', description='Foo',
+    def make_package(self, package_manager=NPM, name='foo', description='Foo',
                                                                      emails=['[email protected]']):
         """Factory for packages.
         """
-        return self.db.one( 'INSERT INTO packages (package_manager, name, description, emails) '
-                            'VALUES (%s, %s, %s, %s) RETURNING *'
-                          , (package_manager, name, description, emails)
-                           )
+        self.db.run( 'INSERT INTO packages (package_manager, name, description, emails) '
+                     'VALUES (%s, %s, %s, %s) RETURNING *'
+                   , (package_manager, name, description, emails)
+                    )
+        return Package.from_names(NPM, name)
 
 
     def make_participant(self, username, **kw):