From d249b5e4aa999df3cc16f9a623b3dd65640d64d3 Mon Sep 17 00:00:00 2001
From: Taras Lytvynenko <69678257+Inferato@users.noreply.github.com>
Date: Thu, 16 Nov 2023 12:04:04 +0200
Subject: [PATCH] fix: Logical operations moved into a separate variable

---
 .../bulk-email-recipient/BulkEmailRecipient.jsx          | 4 ++--
 .../__factories__/bulkEmailFormCourseMode.factory.js     | 9 +++++++++
 2 files changed, 11 insertions(+), 2 deletions(-)

diff --git a/src/components/bulk-email-tool/bulk-email-form/bulk-email-recipient/BulkEmailRecipient.jsx b/src/components/bulk-email-tool/bulk-email-form/bulk-email-recipient/BulkEmailRecipient.jsx
index 6f6d163..1acb9b5 100644
--- a/src/components/bulk-email-tool/bulk-email-form/bulk-email-recipient/BulkEmailRecipient.jsx
+++ b/src/components/bulk-email-tool/bulk-email-form/bulk-email-recipient/BulkEmailRecipient.jsx
@@ -20,6 +20,7 @@ export default function BulkEmailRecipient(props) {
     additionalCohorts,
     courseModes,
   } = props;
+  const isCourseModes = courseModes && courseModes.length > 1;
   return (
     <Form.Group>
       <Form.Label>
@@ -57,8 +58,7 @@ export default function BulkEmailRecipient(props) {
         </Form.Checkbox>
         {
           // additional modes
-          courseModes
-          && courseModes.length > 1
+          isCourseModes
           && courseModes.map((courseMode) => (
             <Form.Checkbox
               key={`track:${courseMode.slug}`}
diff --git a/src/components/bulk-email-tool/bulk-email-form/data/__factories__/bulkEmailFormCourseMode.factory.js b/src/components/bulk-email-tool/bulk-email-form/data/__factories__/bulkEmailFormCourseMode.factory.js
index 6564dd1..14874fa 100644
--- a/src/components/bulk-email-tool/bulk-email-form/data/__factories__/bulkEmailFormCourseMode.factory.js
+++ b/src/components/bulk-email-tool/bulk-email-form/data/__factories__/bulkEmailFormCourseMode.factory.js
@@ -1,5 +1,14 @@
+/**
+ * Creates a factory function for generating course modes.
+ *
+ * @returns {Object[]} An array of course mode objects generated by different factories.
+ */
 import { Factory } from 'rosie'; // eslint-disable-line import/no-extraneous-dependencies
 
+/**
+ * Generates an array of course mode objects using Rosie Factory.
+ * @returns {Array<Object>} An array of course mode objects with attributes 'slug' and 'name'.
+ */
 const courseModeFactory = () => {
   const AuditModeFactory = Factory.define('AuditModeFactory')
     .attr('slug', 'audit')