models: Use non-null slugs for 'Check.name'

The schema for 'Check' defines 'Check.name' as a 'CharField'. This is
less than ideal as names with spaces and special characters can't be
represented cleanly in URLs etc. We should use 'SlugField' instead.

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>
Reviewed-by: Andy Doan <andy.doan@linaro.org>
Closes: #33

diff --git a/patchwork/migrations/0013_slug_check_context.py b/patchwork/migrations/0013_slug_check_context.py
new file mode 100644 (file)
index 0000000..17e33aa
--- /dev/null
+++ b/patchwork/migrations/0013_slug_check_context.py
@@ -0,0 +1,19 @@
+# -*- coding: utf-8 -*-
+from __future__ import unicode_literals
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('patchwork', '0012_add_coverletter_model'),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name='check',
+            name='context',
+            field=models.SlugField(default=b'default', help_text=b'A label to discern check from checks of other testing systems.', max_length=255),
+        ),
+    ]

diff --git a/patchwork/models.py b/patchwork/models.py
index 6209527d0bb7d4f4f0f55a513301e223cae9b06d..0ddb4098579ae8484729763e185b6940c07ba507 100644 (file)
--- a/patchwork/models.py
+++ b/patchwork/models.py
@@ -581,12 +581,12 @@ class Check(models.Model):
         help_text='The state of the check.')
     target_url = models.URLField(
         blank=True, null=True,
-        help_text='The target URL to associate with this check. This should'
-        ' be specific to the patch.')
+        help_text='The target URL to associate with this check. This should '
+        'be specific to the patch.')
     description = models.TextField(
         blank=True, null=True, help_text='A brief description of the check.')
-    context = models.CharField(
-        max_length=255, default='default', blank=True, null=True,
+    context = models.SlugField(
+        max_length=255, default='default',
         help_text='A label to discern check from checks of other testing '
         'systems.')