chooser_upload should be called with a GET request

Author: teixas

wagtailmedia/tests/test_views.py

@@ -690,22 +690,6 @@ class TestMediaChooserUploadView(TestCase, WagtailTestUtils):
     def setUp(self):
         self.login()
 
-    def test_simple_audio(self):
-        response = self.client.get(reverse(
-            'wagtailmedia:chooser_upload', args=('audio', )))
-        self.assertEqual(response.status_code, 200)
-        self.assertTemplateUsed(response, 'wagtailmedia/chooser/chooser.html')
-        response_json = json.loads(response.content.decode())
-        self.assertEqual(response_json['step'], 'chooser')
-
-    def test_simple_video(self):
-        response = self.client.get(reverse(
-            'wagtailmedia:chooser_upload', args=('video', )))
-        self.assertEqual(response.status_code, 200)
-        self.assertTemplateUsed(response, 'wagtailmedia/chooser/chooser.html')
-        response_json = json.loads(response.content.decode())
-        self.assertEqual(response_json['step'], 'chooser')
-
     def test_upload_audio(self):
         response = self.client.post(
             reverse('wagtailmedia:chooser_upload', args=('audio', )), {

wagtailmedia/views/chooser.py

@@ -140,10 +140,6 @@ def chooser_upload(request, media_type):
                 json_data={'step': 'media_chosen',
                            'result': get_media_json(media)}
             )
-    else:
-        media = Media(uploaded_by_user=request.user, type=media_type)
-        form = MediaForm(
-            instance=media, user=request.user, prefix='media-chooser-upload')
 
     media_files = permission_policy.instances_user_has_any_permission_for(
         request.user, ['change', 'delete']