add an additional tag so we can route zendesk tickets better for white label sites

add tests

fix

use org

common/djangoapps/student/tests/test_microsite.py

@@ -10,6 +10,8 @@ from django.contrib.auth.models import User
 
 FAKE_MICROSITE = {
     "SITE_NAME": "openedx.localhost",
+    "university": "fakeuniversity",
+    "course_org_filter": "fakeorg",
     "REGISTRATION_EXTRA_FIELDS": {
         "address1": "required",
         "city": "required",

common/djangoapps/util/tests/test_submit_feedback.py

@@ -11,6 +11,8 @@ from zendesk import ZendeskError
 import json
 import mock
 
+from student.tests.test_microsite import fake_microsite_get_value
+
 
 @mock.patch.dict("django.conf.settings.FEATURES", {"ENABLE_FEEDBACK_SUBMISSION": True})
 @override_settings(ZENDESK_URL="dummy", ZENDESK_USER="dummy", ZENDESK_API_KEY="dummy")
@@ -51,7 +53,7 @@ class SubmitFeedbackTest(TestCase):
             HTTP_REFERER="test_referer",
             HTTP_USER_AGENT="test_user_agent",
             REMOTE_ADDR="1.2.3.4",
-            SERVER_NAME="test_server"
+            SERVER_NAME="test_server",
         )
         req.user = user
         return views.submit_feedback(req)
@@ -189,6 +191,49 @@ class SubmitFeedbackTest(TestCase):
         self.assertEqual(zendesk_mock_instance.mock_calls, expected_zendesk_calls)
         self._assert_datadog_called(datadog_mock, with_tags=True)
 
+    @mock.patch("microsite_configuration.microsite.get_value", fake_microsite_get_value)
+    def test_valid_request_anon_user_microsite(self, zendesk_mock_class, datadog_mock):
+        """
+        Test a valid request from an anonymous user to a mocked out microsite
+
+        The response should have a 200 (success) status code, and a ticket with
+        the given information should have been submitted via the Zendesk API with the additional
+        tag that will come from microsite configuration
+        """
+        zendesk_mock_instance = zendesk_mock_class.return_value
+        zendesk_mock_instance.create_ticket.return_value = 42
+        self._test_success(self._anon_user, self._anon_fields)
+        expected_zendesk_calls = [
+            mock.call.create_ticket(
+                {
+                    "ticket": {
+                        "requester": {"name": "Test User", "email": "test@edx.org"},
+                        "subject": "a subject",
+                        "comment": {"body": "some details"},
+                        "tags": ["test_course", "test_issue", "LMS", "whitelabel_fakeorg"]
+                    }
+                }
+            ),
+            mock.call.update_ticket(
+                42,
+                {
+                    "ticket": {
+                        "comment": {
+                            "public": False,
+                            "body":
+                            "Additional information:\n\n"
+                            "Client IP: 1.2.3.4\n"
+                            "Host: test_server\n"
+                            "Page: test_referer\n"
+                            "Browser: test_user_agent"
+                        }
+                    }
+                }
+            )
+        ]
+        self.assertEqual(zendesk_mock_instance.mock_calls, expected_zendesk_calls)
+        self._assert_datadog_called(datadog_mock, with_tags=True)
+
     def test_bad_request_auth_user_no_subject(self, zendesk_mock_class, datadog_mock):
         """Test a request from an authenticated user not specifying `subject`."""
         self._test_bad_request_omit_field(self._auth_user, self._auth_fields, "subject", zendesk_mock_class, datadog_mock)

common/djangoapps/util/views.py

@@ -11,6 +11,7 @@ from django.http import (Http404, HttpResponse, HttpResponseNotAllowed,
 from dogapi import dog_stats_api
 from edxmako.shortcuts import render_to_response
 import zendesk
+from microsite_configuration import microsite
 
 import calc
 import track.views
@@ -100,6 +101,13 @@ def _record_feedback_in_zendesk(realname, email, subject, details, tags, additio
 
     # Tag all issues with LMS to distinguish channel in Zendesk; requested by student support team
     zendesk_tags = list(tags.values()) + ["LMS"]
+
+    # Per edX support, we would like to be able to route white label feedback items
+    # via tagging
+    white_label_org = microsite.get_value('course_org_filter')
+    if white_label_org:
+        zendesk_tags = zendesk_tags + ["whitelabel_{org}".format(org=white_label_org)]
+
     new_ticket = {
         "ticket": {
             "requester": {"name": realname, "email": email},