Separate notice_comment dependencies, add to tests.

This will significantly reduce the number of transitive dependencies needed by
ATF, FEC, etc.

regulations/settings/dev.py

@@ -7,6 +7,7 @@
 CACHES['api_cache']['TIMEOUT'] = 5  # roughly per request
 
 ROOT_URLCONF = 'regulations.test_urls'
+INSTALLED_APPS = INSTALLED_APPS + ('notice_comment',)
 
 try:
     from local_settings import *

setup.py

@@ -7,16 +7,16 @@
     packages=find_packages(),
     include_package_data=True,
     install_requires=[
-        'boto3',
         'cached-property',
-        'celery',
         'django>=1.8,<1.12',
         'enum34',
         'futures',
         'requests',
         'six',
-        'requests-toolbelt',
     ],
+    extras_require={
+        'notice_comment': ['boto3', 'celery', 'requests-toolbelt'],
+    },
     classifiers=[
         'License :: Public Domain',
         'License :: CC0 1.0 Universal (CC0 1.0) Public Domain Dedication'

tox.ini

@@ -15,6 +15,7 @@ deps =
   pytest~=3.0
   pytest_cov~=2.4
   pytest_django~=3.1
+extras = notice_comment
 
 commands = pytest --cov
 
@@ -24,15 +25,16 @@ deps =
   bandit==1.4.0
   flake8==2.5.4
 commands =
-  flake8 fr_notices regulations
-  bandit -r --ini tox.ini fr_notices regulations manage.py setup.py
+  flake8 fr_notices notice_comment regulations
+  bandit -r --ini tox.ini fr_notices notice_comment regulations manage.py setup.py
 
 [testenv:docs]
 deps = sphinx
 commands =
   # Tox won't do wildcard expansion, so run in a shell
-  sh -c "rm -f docs/regulations*.rst docs/fr_notices*.rst"
+  sh -c "rm -f docs/regulations*.rst docs/fr_notices*.rst docs/notice_comment*.rst"
   sphinx-apidoc -F -o docs fr_notices
+  sphinx-apidoc -F -o docs notice_comment
   sphinx-apidoc -F -o docs regulations
   sphinx-build -b dirhtml -d docs/_build/doctrees/ docs/ docs/_build/dirhtml/
 whitelist_externals = sh
@@ -55,19 +57,19 @@ passenv =
 commands = py.test regulations/uitests -s
 
 [bandit]
-exclude = fr_notices/tests,regulations/tests,regulations/uitests
+exclude = fr_notices/tests,notice_comment/tests,regulations/tests,regulations/uitests
 
 [coverage:run]
-source = regulations,fr_notices
-omit = regulations/uitests/*,regulations/tests/*,fr_notices/tests/*
+source = regulations,fr_notices,notice_comment
+omit = regulations/uitests/*,regulations/tests/*,fr_notices/tests/*,notice_comment/tests/*
 
 [flake8]
 exclude = regulations/settings/*.py
 
 [isort]
 
 [pytest]
-testpaths = regulations/tests fr_notices/tests
+testpaths = regulations/tests fr_notices/tests notice_comment/tests
 python_files = tests_*.py *_test.py *_tests.py
 DJANGO_SETTINGS_MODULE = regulations.settings.dev