added document_filename field to submissions model

Removed course module migration

lms/djangoapps/api_manager/migrations/0011_auto__add_coursemodulecompletion.py

@@ -7,17 +7,7 @@ from django.db import models
 
 class Migration(SchemaMigration):
 
-    def forwards(self, orm):
-        # Adding model 'CourseModuleCompletion'
-        db.create_table('api_manager_coursemodulecompletion', (
-            ('id', self.gf('django.db.models.fields.AutoField')(primary_key=True)),
-            ('created', self.gf('model_utils.fields.AutoCreatedField')(default=datetime.datetime.now)),
-            ('modified', self.gf('model_utils.fields.AutoLastModifiedField')(default=datetime.datetime.now)),
-            ('user', self.gf('django.db.models.fields.related.ForeignKey')(related_name='course_completions', to=orm['auth.User'])),
-            ('course_id', self.gf('django.db.models.fields.CharField')(max_length=255, db_index=True)),
-            ('content_id', self.gf('django.db.models.fields.CharField')(max_length=255, db_index=True)),
-        ))
-        db.send_create_signal('api_manager', ['CourseModuleCompletion'])
+    def forwards(self, orm):      
 
         # Adding M2M table for field groups on 'Organization'
         db.create_table('api_manager_organization_groups', (
@@ -29,9 +19,7 @@ class Migration(SchemaMigration):
 
 
     def backwards(self, orm):
-        # Deleting model 'CourseModuleCompletion'
-        db.delete_table('api_manager_coursemodulecompletion')
-
+      
         # Removing M2M table for field groups on 'Organization'
         db.delete_table('api_manager_organization_groups')
 

lms/djangoapps/projects/models.py

@@ -58,6 +58,7 @@ class WorkgroupSubmission(TimeStampedModel):
     document_id = models.CharField(max_length=255)
     document_url = models.CharField(max_length=255)
     document_mime_type = models.CharField(max_length=255)
+    document_filename = models.CharField(max_length=255, blank=True, null=True)
 
 
 class WorkgroupSubmissionReview(TimeStampedModel):

lms/djangoapps/projects/serializers.py

@@ -68,7 +68,8 @@ class WorkgroupSubmissionSerializer(serializers.HyperlinkedModelSerializer):
         """ Meta class for defining additional serializer characteristics """
         model = WorkgroupSubmission
         fields = (
-            'id', 'url', 'created', 'modified', 'document_id', 'document_url', 'document_mime_type',
+            'id', 'url', 'created', 'modified', 'document_id', 'document_url',
+            'document_mime_type', 'document_filename',
             'user', 'workgroup', 'reviews'
         )
 

lms/djangoapps/projects/tests/test_workgroup_submissions.py

@@ -47,6 +47,7 @@ class SubmissionsApiTests(TestCase):
         self.test_document_id = "Document12345.pdf"
         self.test_document_url = "http://test-s3.amazonaws.com/bucketname"
         self.test_document_mime_type = "application/pdf"
+        self.test_document_filename = "Test PDF Document"
 
         self.test_user = User.objects.create(
             email="test@edx.org",
@@ -133,6 +134,7 @@ class SubmissionsApiTests(TestCase):
             'document_id': self.test_document_id,
             'document_url': self.test_document_url,
             'document_mime_type': self.test_document_mime_type,
+            'document_filename': self.test_document_filename,
         }
         response = self.do_post(self.test_submissions_uri, submission_data)
         self.assertEqual(response.status_code, 400)
@@ -143,6 +145,7 @@ class SubmissionsApiTests(TestCase):
             'document_id': self.test_document_id,
             'document_url': self.test_document_url,
             'document_mime_type': self.test_document_mime_type,
+            'document_filename': self.test_document_filename,
         }
         response = self.do_post(self.test_submissions_uri, submission_data)
         self.assertEqual(response.status_code, 400)
@@ -154,6 +157,7 @@ class SubmissionsApiTests(TestCase):
             'document_id': self.test_document_id,
             'document_url': self.test_document_url,
             'document_mime_type': self.test_document_mime_type,
+            'document_filename': self.test_document_filename,
         }
         response = self.do_post(self.test_submissions_uri, submission_data)
         self.assertEqual(response.status_code, 201)
@@ -172,6 +176,7 @@ class SubmissionsApiTests(TestCase):
         self.assertEqual(response.data['document_id'], self.test_document_id)
         self.assertEqual(response.data['document_url'], self.test_document_url)
         self.assertEqual(response.data['document_mime_type'], self.test_document_mime_type)
+        self.assertEqual(response.data['document_filename'], self.test_document_filename)
         self.assertIsNotNone(response.data['reviews'])
         self.assertIsNotNone(response.data['created'])
         self.assertIsNotNone(response.data['modified'])
@@ -188,6 +193,7 @@ class SubmissionsApiTests(TestCase):
             'document_id': self.test_document_id,
             'document_url': self.test_document_url,
             'document_mime_type': self.test_document_mime_type,
+            'document_filename': self.test_document_filename,
         }
         response = self.do_post(self.test_submissions_uri, submission_data)
         self.assertEqual(response.status_code, 201)