Refactor is_precompiled_letter to model

app/config.py

@@ -149,7 +149,6 @@ class Config(object):
     CHANGE_EMAIL_CONFIRMATION_TEMPLATE_ID = 'eb4d9930-87ab-4aef-9bce-786762687884'
     SERVICE_NOW_LIVE_TEMPLATE_ID = '618185c6-3636-49cd-b7d2-6f6f5eb3bdde'
     ORGANISATION_INVITATION_EMAIL_TEMPLATE_ID = '203566f0-d835-47c5-aa06-932439c86573'
-    PRECOMPILED_TEMPLATE_NAME = 'Pre-compiled PDF'
 
     BROKER_URL = 'sqs://'
     BROKER_TRANSPORT_OPTIONS = {

app/dao/notifications_dao.py

@@ -83,7 +83,7 @@ def dao_get_template_usage(service_id, limit_days=None):
         Template.id.label('template_id'),
         Template.name,
         Template.template_type,
-        Template.hidden,
+        Template.is_precompiled_letter,
         notifications_aggregate_query.c.count
     ).join(
         notifications_aggregate_query,

app/dao/services_dao.py

@@ -522,7 +522,7 @@ def dao_fetch_monthly_historical_usage_by_template_for_service(service_id, year)
         stat.month = result.month
         stat.year = result.year
         stat.count = result.count
-        stat.hidden = result.hidden
+        stat.is_precompiled_letter = result.is_precompiled_letter
         stats.append(stat)
 
     month = get_london_month_from_utc_column(Notification.created_at)
@@ -534,7 +534,7 @@ def dao_fetch_monthly_historical_usage_by_template_for_service(service_id, year)
     if fy_start < datetime.now() < fy_end:
         today_results = db.session.query(
             Notification.template_id,
-            Template.hidden,
+            Template.is_precompiled_letter,
             Template.name,
             Template.template_type,
             extract('month', month).label('month'),
@@ -574,7 +574,7 @@ def dao_fetch_monthly_historical_usage_by_template_for_service(service_id, year)
                 new_stat.month = int(today_result.month)
                 new_stat.year = int(today_result.year)
                 new_stat.count = today_result.count
-                new_stat.hidden = today_result.hidden
+                new_stat.is_precompiled_letter = today_result.is_precompiled_letter
                 stats.append(new_stat)
 
     return stats

app/dao/stats_template_usage_by_month_dao.py

@@ -35,9 +35,9 @@ def insert_or_update_stats_for_template(template_id, month, year, count):
 def dao_get_template_usage_stats_by_service(service_id, year):
     return db.session.query(
         StatsTemplateUsageByMonth.template_id,
-        Template.hidden,
         Template.name,
         Template.template_type,
+        Template.is_precompiled_letter,
         StatsTemplateUsageByMonth.month,
         StatsTemplateUsageByMonth.year,
         StatsTemplateUsageByMonth.count

app/dao/templates_dao.py

@@ -5,7 +5,11 @@ from sqlalchemy import asc, desc
 from sqlalchemy.sql.expression import bindparam
 
 from app import db
-from app.models import (Template, TemplateHistory, TemplateRedacted)
+from app.models import (
+    Template,
+    TemplateHistory,
+    TemplateRedacted
+)
 from app.dao.dao_utils import (
     transactional,
     version_class
@@ -135,7 +139,7 @@ def dao_get_templates_for_cache(cache):
     query = db.session.query(Template.id.label('template_id'),
                              Template.template_type,
                              Template.name,
-                             Template.hidden,
+                             Template.is_precompiled_letter,
                              cache_subq.c.count.label('count')
                              ).join(cache_subq,
                                     Template.id == cache_subq.c.template_id

app/letters/utils.py

@@ -5,7 +5,6 @@ from flask import current_app
 
 from notifications_utils.s3 import s3upload
 
-from app.models import LETTER_TYPE
 from app.variables import Retention
 
 
@@ -77,11 +76,3 @@ def get_letter_pdf(notification):
         file_content = obj.get()["Body"].read()
 
     return file_content
-
-
-def is_precompiled_letter(template):
-    return (
-        template.template_type == LETTER_TYPE and
-        template.hidden and
-        template.name == current_app.config['PRECOMPILED_TEMPLATE_NAME']
-    )

app/models.py

@@ -6,6 +6,7 @@ from flask import url_for, current_app
 
 from sqlalchemy.ext.declarative import declared_attr
 from sqlalchemy.ext.associationproxy import association_proxy
+from sqlalchemy.ext.hybrid import hybrid_property
 from sqlalchemy.dialects.postgresql import (
     UUID,
     JSON
@@ -641,6 +642,9 @@ class TemplateProcessTypes(db.Model):
     name = db.Column(db.String(255), primary_key=True)
 
 
+PRECOMPILED_TEMPLATE_NAME = 'Pre-compiled PDF'
+
+
 class TemplateBase(db.Model):
     __abstract__ = True
 
@@ -718,6 +722,14 @@ class TemplateBase(db.Model):
         else:
             return None
 
+    @hybrid_property
+    def is_precompiled_letter(self):
+        return self.hidden and self.name == PRECOMPILED_TEMPLATE_NAME and self.template_type == LETTER_TYPE
+
+    @is_precompiled_letter.setter
+    def is_precompiled_letter(self, value):
+        pass
+
     def _as_utils_template(self):
         if self.template_type == EMAIL_TYPE:
             return PlainTextEmailTemplate(

app/schemas.py

@@ -24,7 +24,6 @@ from notifications_utils.recipients import (
 
 from app import ma
 from app import models
-from app.letters.utils import is_precompiled_letter
 from app.models import ServicePermission
 from app.dao.permissions_dao import permission_dao
 from app.utils import get_template_instance
@@ -315,7 +314,7 @@ class BaseTemplateSchema(BaseSchema):
         return template.get_reply_to_text()
 
     def get_precompiled_letter(self, template):
-        return is_precompiled_letter(template)
+        return template.is_precompiled_letter
 
     class Meta:
         model = models.Template

app/service/rest.py

@@ -68,7 +68,7 @@ from app.errors import (
     InvalidRequest,
     register_errors
 )
-from app.models import Service, EmailBranding
+from app.models import Service, EmailBranding, LETTER_TYPE, PRECOMPILED_TEMPLATE_NAME
 from app.schema_validation import validate
 from app.service import statistics
 from app.service.service_senders_schema import (
@@ -538,9 +538,9 @@ def get_monthly_template_usage(service_id):
                     'year': i.year,
                     'count': i.count,
                     'precompiled_letter': (
-                        i.template_type == 'letter' and
+                        i.template_type == LETTER_TYPE and
                         i.hidden and
-                        i.name == current_app.config['PRECOMPILED_TEMPLATE_NAME']
+                        i.name == PRECOMPILED_TEMPLATE_NAME
                     )
                 }
             )

app/template/rest.py

@@ -21,7 +21,7 @@ from app.dao.templates_dao import (
     dao_get_template_by_id)
 from notifications_utils.template import SMSMessageTemplate
 from app.dao.services_dao import dao_fetch_service_by_id
-from app.letters.utils import get_letter_pdf, is_precompiled_letter
+from app.letters.utils import get_letter_pdf
 from app.models import SMS_TYPE
 from app.notifications.validators import service_has_permission, check_reply_to
 from app.schemas import (template_schema, template_history_schema)
@@ -199,7 +199,7 @@ def preview_letter_template_by_notification_id(service_id, notification_id, file
 
     template = dao_get_template_by_id(notification.template_id)
 
-    if is_precompiled_letter(template):
+    if template.is_precompiled_letter:
 
         try:
 

app/template_statistics/rest.py

@@ -14,7 +14,6 @@ from app.dao.templates_dao import (
     dao_get_template_by_id_and_service_id
 )
 
-from app.letters.utils import is_precompiled_letter
 from app.schemas import notification_with_template_schema
 from app.utils import cache_key_for_service_template_counter
 from app.errors import register_errors, InvalidRequest
@@ -49,7 +48,7 @@ def get_template_statistics_for_service_by_day(service_id):
             'template_id': str(data.template_id),
             'template_name': data.name,
             'template_type': data.template_type,
-            'precompiled_letter': is_precompiled_letter(data)
+            'precompiled_letter': data.is_precompiled_letter
         }
 
     return jsonify(data=[serialize(row) for row in stats])

tests/app/dao/test_stats_template_usage_by_month_dao.py

@@ -3,7 +3,7 @@ from app.dao.stats_template_usage_by_month_dao import (
     insert_or_update_stats_for_template,
     dao_get_template_usage_stats_by_service
 )
-from app.models import StatsTemplateUsageByMonth
+from app.models import StatsTemplateUsageByMonth, LETTER_TYPE, PRECOMPILED_TEMPLATE_NAME
 
 from tests.app.db import create_service, create_template
 
@@ -74,6 +74,36 @@ def test_dao_get_template_usage_stats_by_service(sample_service):
     assert len(result) == 1
 
 
+def test_dao_get_template_usage_stats_by_service_for_precompiled_letters(sample_service):
+
+    letter_template = create_template(service=sample_service, template_type=LETTER_TYPE)
+
+    precompiled_letter_template = create_template(
+        service=sample_service, template_name=PRECOMPILED_TEMPLATE_NAME, hidden=True, template_type=LETTER_TYPE)
+
+    db.session.add(StatsTemplateUsageByMonth(
+        template_id=letter_template.id,
+        month=5,
+        year=2017,
+        count=10
+    ))
+
+    db.session.add(StatsTemplateUsageByMonth(
+        template_id=precompiled_letter_template.id,
+        month=4,
+        year=2017,
+        count=20
+    ))
+
+    result = dao_get_template_usage_stats_by_service(sample_service.id, 2017)
+
+    assert len(result) == 2
+    assert [
+        (letter_template.id, 'letter Template Name', 'letter', False, 5, 2017, 10),
+        (precompiled_letter_template.id, PRECOMPILED_TEMPLATE_NAME, 'letter', True, 4, 2017, 20)
+    ] == result
+
+
 def test_dao_get_template_usage_stats_by_service_specific_year(sample_service):
 
     email_template = create_template(service=sample_service, template_type="email")

tests/app/dao/test_templates_dao.py

@@ -13,7 +13,12 @@ from app.dao.templates_dao import (
     dao_get_templates_for_cache,
     dao_redact_template, dao_update_template_reply_to
 )
-from app.models import Template, TemplateHistory, TemplateRedacted
+from app.models import (
+    Template,
+    TemplateHistory,
+    TemplateRedacted,
+    PRECOMPILED_TEMPLATE_NAME
+)
 
 from tests.app.conftest import sample_template as create_sample_template
 from tests.app.db import create_template, create_letter_contact
@@ -489,8 +494,7 @@ def test_get_templates_by_ids_successful(notify_db, notify_db_session):
         notify_db_session,
         template_name='Sample Template 2',
         template_type="sms",
-        content="Template content",
+        content="Template content"
-        hidden=True
     )
     create_sample_template(
         notify_db,
@@ -505,7 +509,32 @@ def test_get_templates_by_ids_successful(notify_db, notify_db_session):
     templates = dao_get_templates_for_cache(cache)
     assert len(templates) == 2
     assert [(template_1.id, template_1.template_type, template_1.name, False, 2),
-            (template_2.id, template_2.template_type, template_2.name, True, 3)] == templates
+            (template_2.id, template_2.template_type, template_2.name, False, 3)] == templates
+
+
+def test_get_letter_templates_by_ids_successful(notify_db, notify_db_session):
+    template_1 = create_sample_template(
+        notify_db,
+        notify_db_session,
+        template_name=PRECOMPILED_TEMPLATE_NAME,
+        template_type="letter",
+        content="Template content",
+        hidden=True
+    )
+    template_2 = create_sample_template(
+        notify_db,
+        notify_db_session,
+        template_name='Sample Template 2',
+        template_type="letter",
+        content="Template content"
+    )
+    sample_cache_dict = {str.encode(str(template_1.id)): str.encode('2'),
+                         str.encode(str(template_2.id)): str.encode('3')}
+    cache = [[k, v] for k, v in sample_cache_dict.items()]
+    templates = dao_get_templates_for_cache(cache)
+    assert len(templates) == 2
+    assert [(template_1.id, template_1.template_type, template_1.name, True, 2),
+            (template_2.id, template_2.template_type, template_2.name, False, 3)] == templates
 
 
 def test_get_templates_by_ids_successful_for_one_cache_item(notify_db, notify_db_session):

tests/app/letters/test_letter_utils.py

@@ -1,7 +1,6 @@
 import pytest
-from flask import current_app
 
-from app.letters.utils import get_bucket_prefix_for_notification, is_precompiled_letter
+from app.letters.utils import get_bucket_prefix_for_notification
 
 
 def test_get_bucket_prefix_for_notification_valid_notification(sample_notification):
@@ -17,23 +16,3 @@ def test_get_bucket_prefix_for_notification_valid_notification(sample_notificati
 def test_get_bucket_prefix_for_notification_invalid_notification():
     with pytest.raises(AttributeError):
         get_bucket_prefix_for_notification(None)
-
-
-def test_is_precompiled_letter_false(sample_letter_template):
-    assert not is_precompiled_letter(sample_letter_template)
-
-
-def test_is_precompiled_letter_true(sample_letter_template):
-    sample_letter_template.hidden = True
-    sample_letter_template.name = current_app.config['PRECOMPILED_TEMPLATE_NAME']
-    assert is_precompiled_letter(sample_letter_template)
-
-
-def test_is_precompiled_letter_hidden_true_not_name(sample_letter_template):
-    sample_letter_template.hidden = True
-    assert not is_precompiled_letter(sample_letter_template)
-
-
-def test_is_precompiled_letter_name_correct_not_hidden(sample_letter_template):
-    sample_letter_template.name = current_app.config['PRECOMPILED_TEMPLATE_NAME']
-    assert not is_precompiled_letter(sample_letter_template)

tests/app/service/test_rest.py

@@ -25,7 +25,8 @@ from app.models import (
     User,
     DVLA_ORG_LAND_REGISTRY,
     KEY_TYPE_NORMAL, KEY_TYPE_TEAM, KEY_TYPE_TEST,
-    EMAIL_TYPE, SMS_TYPE, LETTER_TYPE, INTERNATIONAL_SMS_TYPE, INBOUND_SMS_TYPE
+    EMAIL_TYPE, SMS_TYPE, LETTER_TYPE, INTERNATIONAL_SMS_TYPE, INBOUND_SMS_TYPE,
+    PRECOMPILED_TEMPLATE_NAME
 )
 from tests import create_authorization_header
 from tests.app.conftest import (
@@ -1834,7 +1835,7 @@ def test_get_template_usage_by_month_returns_two_templates(
     template_one = create_template(
         sample_service,
         template_type=LETTER_TYPE,
-        template_name=current_app.config['PRECOMPILED_TEMPLATE_NAME'],
+        template_name=PRECOMPILED_TEMPLATE_NAME,
         hidden=True
     )
 
@@ -1894,7 +1895,7 @@ def test_get_template_usage_by_month_returns_two_templates(
     assert resp_json[0]["month"] == 4
     assert resp_json[0]["year"] == 2017
     assert resp_json[0]["count"] == 1
-    assert resp_json[0]["precompiled_letter"] is True
+    assert resp_json[0]["is_precompiled_letter"] is True
 
     assert resp_json[1]["template_id"] == str(sample_template.id)
     assert resp_json[1]["name"] == sample_template.name
@@ -1902,7 +1903,7 @@ def test_get_template_usage_by_month_returns_two_templates(
     assert resp_json[1]["month"] == 4
     assert resp_json[1]["year"] == 2017
     assert resp_json[1]["count"] == 3
-    assert resp_json[1]["precompiled_letter"] is False
+    assert resp_json[1]["is_precompiled_letter"] is False
 
     assert resp_json[2]["template_id"] == str(sample_template.id)
     assert resp_json[2]["name"] == sample_template.name
@@ -1910,7 +1911,7 @@ def test_get_template_usage_by_month_returns_two_templates(
     assert resp_json[2]["month"] == 11
     assert resp_json[2]["year"] == 2017
     assert resp_json[2]["count"] == 1
-    assert resp_json[2]["precompiled_letter"] is False
+    assert resp_json[2]["is_precompiled_letter"] is False
 
 
 def test_search_for_notification_by_to_field(client, notify_db, notify_db_session):
@@ -2155,7 +2156,7 @@ def test_get_notification_for_service_includes_precompiled_letter(admin_request,
     )
 
     assert resp['id'] == str(sample_notification.id)
-    assert resp['template']['precompiled_letter'] is False
+    assert resp['template']['is_precompiled_letter'] is False
 
 
 def test_get_all_notifications_for_service_includes_template_redacted(admin_request, sample_service):
@@ -2202,10 +2203,10 @@ def test_get_all_notifications_for_service_includes_template_hidden(admin_reques
     )
 
     assert resp['notifications'][0]['id'] == str(precompiled_noti.id)
-    assert resp['notifications'][0]['template']['precompiled_letter'] is True
+    assert resp['notifications'][0]['template']['is_precompiled_letter'] is True
 
     assert resp['notifications'][1]['id'] == str(letter_noti.id)
-    assert resp['notifications'][1]['template']['precompiled_letter'] is False
+    assert resp['notifications'][1]['template']['is_precompiled_letter'] is False
 
 
 def test_search_for_notification_by_to_field_returns_personlisation(

tests/app/test_model.py

@@ -18,7 +18,8 @@ from app.models import (
     NOTIFICATION_STATUS_LETTER_ACCEPTED,
     NOTIFICATION_STATUS_LETTER_RECEIVED,
     NOTIFICATION_STATUS_TYPES_FAILED,
-    NOTIFICATION_TECHNICAL_FAILURE
+    NOTIFICATION_TECHNICAL_FAILURE,
+    PRECOMPILED_TEMPLATE_NAME
 )
 from tests.app.conftest import (
     sample_template as create_sample_template,
@@ -319,3 +320,23 @@ def test_letter_notification_postcode_can_be_null_for_precompiled_letters(client
     assert json['line_1'] == 'test'
     assert json['line_2'] == 'London'
     assert json['postcode'] is None
+
+
+def test_is_precompiled_letter_false(sample_letter_template):
+    assert not sample_letter_template.is_precompiled_letter
+
+
+def test_is_precompiled_letter_true(sample_letter_template):
+    sample_letter_template.hidden = True
+    sample_letter_template.name = PRECOMPILED_TEMPLATE_NAME
+    assert sample_letter_template.is_precompiled_letter
+
+
+def test_is_precompiled_letter_hidden_true_not_name(sample_letter_template):
+    sample_letter_template.hidden = True
+    assert not sample_letter_template.is_precompiled_letter
+
+
+def test_is_precompiled_letter_name_correct_not_hidden(sample_letter_template):
+    sample_letter_template.name = PRECOMPILED_TEMPLATE_NAME
+    assert not sample_letter_template.is_precompiled_letter