added testcases to check email send once

CenterForOpenScience · Jan 15, 2025 · 3c93931 · 3c93931
1 parent d007d63
commit 3c93931
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 1 deletion.
diff --git a/admin/base/settings/defaults.py b/admin/base/settings/defaults.py
@@ -40,7 +40,9 @@
 CSRF_COOKIE_HTTPONLY = False
 
 ALLOWED_HOSTS = [
-    '.osf.io'
+    '.osf.io',
+    'localhost',
+    '127.0.0.1'
 ]
 
 AUTH_PASSWORD_VALIDATORS = [

diff --git a/api_tests/requests/views/test_node_request_institutional_access.py b/api_tests/requests/views/test_node_request_institutional_access.py
@@ -194,6 +194,14 @@ def test_institutional_admin_unauth_institution(self, app, project, institution_
         assert res.status_code == 403
         assert 'Institutional request access is not enabled.' in res.json['errors'][0]['detail']
 
+    @mock.patch('website.mails.send_mail')
+    def test_email_send_to_all_admins_once_on_institutional_request(self, mock_mail, app, project, url, create_payload, institutional_admin):
+        project.is_public = True
+        project.save()
+        res = app.post_json_api(url, create_payload, auth=institutional_admin.auth)
+        assert res.status_code == 201
+        assert mock_mail.call_count == 1
+
     @mock.patch('api.requests.serializers.send_mail')
     def test_email_not_sent_without_recipient(self, mock_mail, app, project, institutional_admin, url,
                                                  create_payload, institution):

diff --git a/osf/features.yaml b/osf/features.yaml
@@ -192,6 +192,7 @@ flags:
   - flag_name: INSTITUTIONAL_DASHBOARD_2024
     name: institutional_dashboard_2024
     note: whether to surface older or updated (in 2024) institutional metrics
+    everyone: false
 
 switches:
   - flag_name: DISABLE_ENGAGEMENT_EMAILS