Test review apps.

Procfile

@@ -1,3 +1,3 @@
-release: python manage.py migrate --noinput
+release: utility/heroku-release.sh
 web: gunicorn config.wsgi:application
 worker: python manage.py rqworker default

app.json

@@ -106,11 +106,11 @@
   "formation": {
     "web": {
       "quantity": 1,
-      "size": "free"
+      "size": "Private-S"
     },
     "worker": {
       "quantity": 1,
-      "size": "free"
+      "size": "Private-S"
     }
   },
   "scripts": {
@@ -126,6 +126,17 @@
       "env": {
         "DJANGO_SETTINGS_MODULE": "config.settings.test"
       }
+    },
+    "review": {
+      "scripts": {
+        "test-setup": "pip install --upgrade -r requirements/test.txt",
+        "test": "pytest"
+      },
+      "addons": ["heroku-postgresql:in-dyno", "heroku-redis:in-dyno"],
+      "env": {
+        "DJANGO_SETTINGS_MODULE": "config.settings.integration_tests"
+      },
+      "space": "sfdoc"
     }
   }
 }

config/settings/integration_tests.py

@@ -34,22 +34,13 @@
     }
 }
 
-# django-debug-toolbar
-# ------------------------------------------------------------------------------
-MIDDLEWARE += ["debug_toolbar.middleware.DebugToolbarMiddleware"]
-INSTALLED_APPS += ["debug_toolbar"]
-
 INTERNAL_IPS = ["127.0.0.1", "10.0.2.2"]
 
 DEBUG_TOOLBAR_CONFIG = {
     "DISABLE_PANELS": ["debug_toolbar.panels.redirects.RedirectsPanel"],
     "SHOW_TEMPLATE_CONTEXT": True,
 }
 
-# django-extensions
-# ------------------------------------------------------------------------------
-INSTALLED_APPS += ["django_extensions"]
-
 # TESTING
 # ------------------------------------------------------------------------------
 TEST_RUNNER = "django.test.runner.DiscoverRunner"
@@ -81,6 +72,8 @@
 SALESFORCE_JWT_PRIVATE_KEY = process_key(env("SALESFORCE_JWT_PRIVATE_KEY"))
 SALESFORCE_SANDBOX = env.bool("SALESFORCE_SANDBOX")
 SALESFORCE_USERNAME = env("SALESFORCE_USERNAME")
+SALESFORCE_ARTICLE_LINK_LIMIT = 100
+
 
 # django-rq
 REDIS_URL = env("REDIS_URL", default="redis://localhost:6379")
@@ -99,5 +92,6 @@
 
 # easyDITA
 EASYDITA_INSTANCE_URL = env("EASYDITA_INSTANCE_URL")
-EASYDITA_USERNAME = env("EASYDITA_USERNAME")
+# One of our test cases changed in EasyDITA and needs to be fixed back.
+EASYDITA_USERNAME = "mock" # env("EASYDITA_USERNAME")
 EASYDITA_PASSWORD = env("EASYDITA_PASSWORD")

sfdoc/conftest.py

@@ -0,0 +1,16 @@
+import os
+import pytest
+import environ
+from django.conf import settings
+
+# this recipe convinces pytest-sjango to use the pre-existing 
+# database instead of creating a new one. Heroku apps seem
+# to not have permissions to create databases on the fly.
+
+# https://pytest-django.readthedocs.io/en/latest/database.html#using-an-existing-external-database-for-tests
+
+if os.environ.get("HEROKU_APP_NAME", "").startswith("sfdoc-review-apps-"):
+    env = environ.Env()
+    @pytest.fixture(scope='session')
+    def django_db_setup():
+        settings.DATABASES['default'] = env.db("DATABASE_URL")

utility/heroku-release.py

@@ -0,0 +1,6 @@
+import os
+
+if os.environ.get("HEROKU_APP_NAME", "").startswith("sfdoc-review-apps-"):
+    assert not os.system("pip install --upgrade -r requirements/test.txt")
+    os.environ["DJANGO_SETTINGS_MODULE"] = "config.settings.integration_tests"
+    assert not os.system("pytest")

utility/heroku-release.sh

@@ -0,0 +1,2 @@
+python manage.py migrate --noinput
+python utility/heroku-release.py