Extend factories create_filegroup to also create files in a group

And test request view with a filegroup; this should be the default
now - every file has to be in at least the default grop.

Author: rebkwok

tests/factories.py

@@ -47,7 +47,11 @@ def write_request_file(request, path, contents=""):
     path.write_text(contents)
 
 
-def create_filegroup(release_request, group_name):
+def create_filegroup(release_request, group_name, filepaths=None):
+    for filepath in filepaths or []:
+        api.add_file_to_request(
+            release_request, filepath, User(1, release_request.author), group_name
+        )
     return api._get_or_create_filegroupmetadata(release_request.id, group_name)
 
 

tests/integration/test_views.py

@@ -307,13 +307,29 @@ def test_request_view_with_directory(client_with_permission, ui_options):
 
 def test_request_view_with_file(client_with_permission, ui_options):
     release_request = factories.create_release_request("workspace")
-    factories.write_request_file(release_request, "file.txt", "foobar")
+    factories.write_workspace_file("workspace", "file.txt", "foobar")
+    factories.create_filegroup(
+        release_request,
+        group_name="default_group",
+        filepaths=["file.txt"]
+    )
+
     response = client_with_permission.get(
         f"/requests/view/{release_request.id}/file.txt"
     )
+    assert "default_group" in response.rendered_content
     assert "foobar" in response.rendered_content
 
 
+def test_request_view_with_submitted_request(client_with_permission, ui_options):
+    release_request = factories.create_release_request("workspace", status=Status.SUBMITTED)
+    response = client_with_permission.get(
+        f"/requests/view/{release_request.id}", follow=True
+    )
+    assert "Reject Request" in response.rendered_content
+    assert "Release Files" in response.rendered_content
+
+
 def test_request_view_with_404(client_with_permission):
     release_request = factories.create_release_request("workspace")
     response = client_with_permission.get(