Jobs: Make URL field required, per @bwaliszewski (#1525)

ewdurbin · web-flow · commit fd0a4d074216 · 2019-11-01T12:15:00.000-04:00
* Jobs: Make URL field required, per @bwaliszewski * Jobs: add required parameter to test for jobs submission form
diff --git a/jobs/migrations/0020_auto_20191101_1601.py b/jobs/migrations/0020_auto_20191101_1601.py
@@ -0,0 +1,18 @@
+# Generated by Django 2.0.13 on 2019-11-01 16:01
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('jobs', '0019_job_submitted_by'),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name='job',
+            name='url',
+            field=models.URLField(null=True, verbose_name='URL'),
+        ),
+    ]
diff --git a/jobs/models.py b/jobs/models.py
@@ -109,7 +109,7 @@ class Job(ContentManageable):
     url = models.URLField(
         verbose_name='URL',
         null=True,
-        blank=True)
+        blank=False)
 
     submitted_by = models.ForeignKey(
         User,
diff --git a/jobs/tests/test_views.py b/jobs/tests/test_views.py
@@ -301,7 +301,8 @@ def test_job_create(self):
             'country': 'USA',
             'description': 'Lorem ipsum dolor sit amet',
             'requirements': 'Some requirements',
-            'email': 'hr@company.com'
+            'email': 'hr@company.com',
+            'url': 'https://jobs.company.com',
         }
 
         # Check that anonymous posting is not allowed. See #852.

Original file line number	Diff line number	Diff line change
`@@ -301,7 +301,8 @@ def test_job_create(self):`
`301`	`301`	`'country': 'USA',`
`302`	`302`	`'description': 'Lorem ipsum dolor sit amet',`
`303`	`303`	`'requirements': 'Some requirements',`
`304`		`- 'email': 'hr@company.com'`
	`304`	`+ 'email': 'hr@company.com',`
	`305`	`+ 'url': 'https://jobs.company.com',`
`305`	`306`	`}`
`306`	`307`
`307`	`308`	`# Check that anonymous posting is not allowed. See #852.`