feat: productize course review experiment

openedx · Jul 25, 2023 · e936725 · e936725
1 parent 5d50e9d
commit e936725
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 36 deletions.
diff --git a/src/components/course/course-header/CourseHeader.jsx b/src/components/course/course-header/CourseHeader.jsx
@@ -1,4 +1,4 @@
-import React, { useContext, useEffect, useMemo } from 'react';
+import React, { useContext, useMemo } from 'react';
 import classNames from 'classnames';
 import {
   Breadcrumb,
@@ -8,8 +8,6 @@ import {
 } from '@edx/paragon';
 import { Link } from 'react-router-dom';
 import { AppContext } from '@edx/frontend-platform/react';
-import { getConfig } from '@edx/frontend-platform/config';
-import { sendEnterpriseTrackEvent } from '@edx/frontend-enterprise-utils';
 
 import { CourseContext } from '../CourseContextProvider';
 import CourseSkills from '../CourseSkills';
@@ -25,7 +23,6 @@ import LicenseRequestedAlert from '../LicenseRequestedAlert';
 import SubsidyRequestButton from '../SubsidyRequestButton';
 import CourseReview from '../CourseReview';
 
-import { isExperimentVariant } from '../../../utils/optimizely';
 import CoursePreview from './CoursePreview';
 
 const CourseHeader = () => {
@@ -34,7 +31,6 @@ const CourseHeader = () => {
     state: {
       course,
       catalog,
-      courseReviews,
     },
     isPolicyRedemptionEnabled,
   } = useContext(CourseContext);
@@ -44,31 +40,6 @@ const CourseHeader = () => {
     () => getDefaultProgram(course.programs),
     [course],
   );
-  const config = getConfig();
-  const isExperimentVariationA = isExperimentVariant(
-    config.EXPERIMENT_5_ID,
-    config.EXPERIMENT_5_VARIANT_1_ID,
-  );
-  const hasSufficientReviewCount = courseReviews?.reviewsCount >= 5;
-  useEffect(() => {
-    if (hasSufficientReviewCount && isExperimentVariationA) {
-      sendEnterpriseTrackEvent(
-        enterpriseConfig.uuid,
-        'edx.ui.enterprise.learner_portal.course.viewedWithReviewsVariation',
-        {
-          course_key: course.key,
-        },
-      );
-    } else {
-      sendEnterpriseTrackEvent(
-        enterpriseConfig.uuid,
-        'edx.ui.enterprise.learner_portal.course.viewedWithoutReviewsVariation',
-        {
-          course_key: course.key,
-        },
-      );
-    }
-  }, [hasSufficientReviewCount, isExperimentVariationA, enterpriseConfig.uuid, course.key]);
 
   return (
     <div className="course-header">
@@ -137,7 +108,7 @@ const CourseHeader = () => {
           <Col xs={12} lg={12}>
             {catalog.containsContentItems ? (
               <>
-                {hasSufficientReviewCount && isExperimentVariationA && <CourseReview />}
+                <CourseReview />
                 {defaultProgram && (
                   <p className="font-weight-bold mt-3 mb-0">
                     This course is part of a {formatProgramType(defaultProgram.type)}.

diff --git a/src/components/course/course-header/tests/CourseHeader.test.jsx b/src/components/course/course-header/tests/CourseHeader.test.jsx
@@ -13,7 +13,6 @@ import CourseHeader from '../CourseHeader';
 import { COURSE_PACING_MAP } from '../../data/constants';
 import { TEST_OWNER } from '../../tests/data/constants';
 import { CourseEnrollmentsContext } from '../../../dashboard/main-content/course-enrollments/CourseEnrollmentsContextProvider';
-import * as optimizelyUtils from '../../../../utils/optimizely';
 
 jest.mock('react-router-dom', () => ({
   useLocation: jest.fn(),
@@ -162,7 +161,6 @@ describe('<CourseHeader />', () => {
   });
 
   test('renders course reviews section', () => {
-    jest.spyOn(optimizelyUtils, 'isExperimentVariant').mockImplementation(() => true);
     render(
       <CourseHeaderWrapper
         initialAppState={initialAppState}
@@ -177,7 +175,6 @@ describe('<CourseHeader />', () => {
   });
 
   test('renders course reviews section and change the review information content', () => {
-    jest.spyOn(optimizelyUtils, 'isExperimentVariant').mockImplementation(() => true);
     render(
       <CourseHeaderWrapper
         initialAppState={initialAppState}

diff --git a/src/index.jsx b/src/index.jsx
@@ -49,8 +49,6 @@ initialize({
         GETSMARTER_PRIVACY_POLICY_URL: process.env.GETSMARTER_PRIVACY_POLICY_URL || null,
         GETSMARTER_LEARNER_DASHBOARD_URL: process.env.GETSMARTER_LEARNER_DASHBOARD_URL || null,
         FEATURE_CONTENT_HIGHLIGHTS: process.env.FEATURE_CONTENT_HIGHLIGHTS || null,
-        EXPERIMENT_5_ID: process.env.EXPERIMENT_5_ID || null,
-        EXPERIMENT_5_VARIANT_1_ID: process.env.EXPERIMENT_5_VARIANT_1_ID || null,
         FEATURE_ENABLE_EMET_REDEMPTION: process.env.FEATURE_ENABLE_EMET_REDEMPTION || null,
         ENTERPRISE_SUBSIDY_BASE_URL: process.env.ENTERPRISE_SUBSIDY_BASE_URL || null,
       });