Use constant instead of string literal for 'npm'

gratipay · Feb 8, 2017 · 9c05276 · 9c05276
1 parent a72e900
commit 9c05276
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/gratipay/models/package/__init__.py b/gratipay/models/package/__init__.py
@@ -4,6 +4,10 @@
 from postgres.orm import Model
 
 
+NPM = 'npm'  # We are starting with a single package manager. If we see
+             # traction we will expand.
+
+
 class Package(Model):
     """Represent a gratipackage. :-)
     """

diff --git a/tests/py/test_packages.py b/tests/py/test_packages.py
@@ -1,7 +1,7 @@
 # -*- coding: utf-8 -*-
 from __future__ import absolute_import, division, print_function, unicode_literals
 
-from gratipay.models.package import Package
+from gratipay.models.package import NPM, Package
 from gratipay.testing import Harness
 
 
@@ -13,4 +13,4 @@ def test_can_be_instantiated_from_id(self):
 
     def test_can_be_instantiated_from_names(self):
         self.make_package()
-        assert Package.from_names('npm', 'foo').name == 'foo'
+        assert Package.from_names(NPM, 'foo').name == 'foo'