From 524b8911e13264eac2395b6a7ab8c4bdfab68b23 Mon Sep 17 00:00:00 2001
From: Aura Alba <aura.alba@edunext.co>
Date: Wed, 11 Oct 2023 09:53:28 -0500
Subject: [PATCH] feat: Add instructors component

---
 .../AddInstructors/_test_/index.test.jsx      |  45 ++++++++
 .../AddInstructors/_test_/reducer.test.jsx    |  53 +++++++++
 .../Instructors/AddInstructors/index.jsx      | 107 ++++++++++++++++++
 .../Instructors/AddInstructors/reducer.jsx    |  30 +++++
 .../Instructors/InstructorsFilters/index.jsx  |   2 +-
 .../Instructors/InstructorsPage/index.jsx     |   3 +-
 src/features/Instructors/actionTypes.js       |   4 +
 src/features/Instructors/data/api.js          |   8 ++
 8 files changed, 250 insertions(+), 2 deletions(-)
 create mode 100644 src/features/Instructors/AddInstructors/_test_/index.test.jsx
 create mode 100644 src/features/Instructors/AddInstructors/_test_/reducer.test.jsx
 create mode 100644 src/features/Instructors/AddInstructors/index.jsx
 create mode 100644 src/features/Instructors/AddInstructors/reducer.jsx

diff --git a/src/features/Instructors/AddInstructors/_test_/index.test.jsx b/src/features/Instructors/AddInstructors/_test_/index.test.jsx
new file mode 100644
index 00000000..3cace2a7
--- /dev/null
+++ b/src/features/Instructors/AddInstructors/_test_/index.test.jsx
@@ -0,0 +1,45 @@
+import React from 'react';
+import axios from 'axios';
+import { render, fireEvent, act } from '@testing-library/react';
+import AddInstructors from 'features/Instructors/AddInstructors';
+import '@testing-library/jest-dom/extend-expect';
+
+jest.mock('axios');
+
+jest.mock('@edx/frontend-platform/logging', () => ({
+  logError: jest.fn(),
+}));
+
+const mockResponse = {
+  data: [
+    {
+      classId: 'CCX1',
+      className: 'CCX 1',
+      masterCourseName: 'Master1',
+    },
+    {
+      classId: 'CCX2',
+      className: 'CCX 2',
+      masterCourseName: 'Master2',
+    },
+  ],
+};
+
+describe('Add instructor component', () => {
+  test('Render and load modal', async () => {
+    axios.get.mockResolvedValue(mockResponse);
+
+    const { getByPlaceholderText, getByText } = render(
+      <AddInstructors />,
+    );
+    const button = getByText('Add Instructor');
+    await act(async () => {
+      fireEvent.click(button);
+    });
+
+    const instructorInfoInput = getByPlaceholderText('Enter Username or Email of the instructor');
+    const ccxSelect = getByText('Select Class Name');
+    expect(instructorInfoInput).toBeInTheDocument();
+    expect(ccxSelect).toBeInTheDocument();
+  });
+});
diff --git a/src/features/Instructors/AddInstructors/_test_/reducer.test.jsx b/src/features/Instructors/AddInstructors/_test_/reducer.test.jsx
new file mode 100644
index 00000000..c9fda83e
--- /dev/null
+++ b/src/features/Instructors/AddInstructors/_test_/reducer.test.jsx
@@ -0,0 +1,53 @@
+import {
+  FETCH_CCX_LIST_FAILURE,
+  FETCH_CCX_LIST_REQUEST,
+  FETCH_CCX_LIST_SUCCESS,
+} from 'features/Instructors/actionTypes';
+import { RequestStatus } from 'features/constants';
+import reducer from 'features/Instructors/InstructorsPage/reducer';
+
+describe('Instructor page reducers', () => {
+  const initialState = {
+    data: [],
+    error: null,
+  };
+
+  test('should handle FETCH_CCX_LIST_REQUEST', () => {
+    const state = {
+      ...initialState,
+      status: RequestStatus.LOADING,
+    };
+    const action = {
+      type: FETCH_CCX_LIST_REQUEST,
+    };
+    expect(reducer(state, action)).toEqual(state);
+  });
+
+  test('should handle FETCH_CCX_LIST_SUCCESS', () => {
+    const state = {
+      ...initialState,
+      status: RequestStatus.SUCCESS,
+      count: 0,
+    };
+    const action = {
+      type: FETCH_CCX_LIST_SUCCESS,
+      payload: {
+        data: [],
+      },
+    };
+    expect(reducer(state, action)).toEqual(state);
+  });
+
+  test('should handle FETCH_CCX_LIST_FAILURE', () => {
+    const state = {
+      ...initialState,
+      status: RequestStatus.ERROR,
+      error: '',
+    };
+    const action = {
+      type: FETCH_CCX_LIST_FAILURE,
+      payload: '',
+    };
+    expect(reducer(state, action)).toEqual(state);
+  });
+});
diff --git a/src/features/Instructors/AddInstructors/index.jsx b/src/features/Instructors/AddInstructors/index.jsx
new file mode 100644
index 00000000..939ed3e9
--- /dev/null
+++ b/src/features/Instructors/AddInstructors/index.jsx
@@ -0,0 +1,107 @@
+import React, { useState, useReducer, useEffect } from 'react';
+import {
+  Button, FormGroup, ModalDialog, useToggle, Form,
+} from '@edx/paragon';
+import { getCCXList } from 'features/Instructors/data/api';
+import reducer from 'features/Instructors/AddInstructors/reducer';
+import { logError } from '@edx/frontend-platform/logging';
+import { camelCaseObject } from '@edx/frontend-platform';
+
+import { FETCH_CCX_LIST_FAILURE, FETCH_CCX_LIST_REQUEST, FETCH_CCX_LIST_SUCCESS } from 'features/Instructors/actionTypes';
+import { RequestStatus } from 'features/constants';
+
+const initialState = {
+  data: [],
+  status: RequestStatus.SUCCESS,
+  error: null,
+};
+
+const addInstructorState = {
+  instructorInfo: '',
+  ccxId: '',
+  ccxName: '',
+};
+
+const AddInstructors = () => {
+  const [state, dispatch] = useReducer(reducer, initialState);
+  const [isOpen, open, close] = useToggle(false);
+  const [addInstructorInfo, setAddInstructorInfo] = useState(addInstructorState);
+
+  const fetchData = async () => {
+    dispatch({ type: FETCH_CCX_LIST_REQUEST });
+
+    try {
+      const response = camelCaseObject(await getCCXList());
+      dispatch({ type: FETCH_CCX_LIST_SUCCESS, payload: response.data });
+    } catch (error) {
+      dispatch({ type: FETCH_CCX_LIST_FAILURE, payload: error });
+      logError(error);
+    }
+  };
+
+  const handleInstructorInput = (e) => {
+    setAddInstructorInfo({
+      ...addInstructorInfo,
+      instructorInfo: e.target.value.trim(),
+    });
+  };
+
+  const handleCcxSelect = (e) => {
+    const select = e.target;
+    setAddInstructorInfo({
+      ...addInstructorInfo,
+      ccxId: select.value,
+      ccxName: select.options[select.selectedIndex].text,
+    });
+  };
+
+  useEffect(() => {
+    fetchData();
+  }, []);
+
+  return (
+    <>
+      <Button variant="primary" onClick={open} size="sm">
+        Add Instructor
+      </Button>
+      <ModalDialog
+        title="Add Instructor"
+        isOpen={isOpen}
+        onClose={close}
+        hasCloseButton
+      >
+        <ModalDialog.Header>
+          <ModalDialog.Title>
+            Add Instructor
+          </ModalDialog.Title>
+        </ModalDialog.Header>
+        <ModalDialog.Body>
+          <FormGroup>
+            <Form.Control
+              as="select"
+              floatingLabel="Select Class Name"
+              className="my-4 mr-0"
+              onChange={handleCcxSelect}
+              id="selectCcx"
+            >
+              {state.data.map((ccx) => <option value={ccx.classId}>{ccx.className}</option>)}
+            </Form.Control>
+            <Form.Control
+              type="text"
+              placeholder="Enter Username or Email of the instructor"
+              floatingLabel="Username or Email"
+              value={addInstructorInfo.instructorInfo}
+              onChange={handleInstructorInput}
+              className="my-4 mr-0"
+            />
+            <div className="d-flex justify-content-end">
+              <Button>Add</Button>
+            </div>
+          </FormGroup>
+        </ModalDialog.Body>
+      </ModalDialog>
+    </>
+  );
+};
+
+export default AddInstructors;
diff --git a/src/features/Instructors/AddInstructors/reducer.jsx b/src/features/Instructors/AddInstructors/reducer.jsx
new file mode 100644
index 00000000..2e3b604b
--- /dev/null
+++ b/src/features/Instructors/AddInstructors/reducer.jsx
@@ -0,0 +1,30 @@
+import {
+  FETCH_CCX_LIST_FAILURE,
+  FETCH_CCX_LIST_REQUEST,
+  FETCH_CCX_LIST_SUCCESS,
+} from 'features/Instructors/actionTypes';
+import { RequestStatus } from 'features/constants';
+
+const reducer = (state, action) => {
+  switch (action.type) {
+    case FETCH_CCX_LIST_REQUEST:
+      return { ...state, status: RequestStatus.LOADING };
+    case FETCH_CCX_LIST_SUCCESS: {
+      return {
+        ...state,
+        status: RequestStatus.SUCCESS,
+        data: action.payload,
+      };
+    }
+    case FETCH_CCX_LIST_FAILURE:
+      return {
+        ...state,
+        status: RequestStatus.ERROR,
+        error: action.payload,
+      };
+    default:
+      return state;
+  }
+};
+
+export default reducer;
diff --git a/src/features/Instructors/InstructorsFilters/index.jsx b/src/features/Instructors/InstructorsFilters/index.jsx
index 8227f0a2..4284f1c5 100644
--- a/src/features/Instructors/InstructorsFilters/index.jsx
+++ b/src/features/Instructors/InstructorsFilters/index.jsx
@@ -31,7 +31,7 @@ const InstructorsFilters = ({ fetchData, resetPagination }) => {
   };
 
   return (
-    <DropdownButton title="Filters" variant="outline-primary">
+    <DropdownButton title="Filters" variant="outline-primary" size="sm" className="mr-3">
       <Form className="row justify-content-center px-3 py-2">
         <Form.Group as={Col} className="mb-0">
           <Form.Control
diff --git a/src/features/Instructors/InstructorsPage/index.jsx b/src/features/Instructors/InstructorsPage/index.jsx
index 87b40ede..ddedcd36 100644
--- a/src/features/Instructors/InstructorsPage/index.jsx
+++ b/src/features/Instructors/InstructorsPage/index.jsx
@@ -8,6 +8,7 @@ import {
 } from '@edx/paragon';
 import InstructorsTable from 'features/Instructors/InstructorsTable';
 import InstructorsFilters from 'features/Instructors/InstructorsFilters';
+import AddInstructors from 'features/Instructors/AddInstructors';
 
 import { getInstructorData } from 'features/Instructors/data/api';
 import {
@@ -62,8 +63,8 @@ const InstructorsPage = () => {
       <h2>Instructors</h2>
       <div className="d-flex justify-content-end">
         <InstructorsFilters fetchData={fetchData} resetPagination={resetPagination} />
+        <AddInstructors />
       </div>
-
       <InstructorsTable
         data={state.data}
         count={state.count}
diff --git a/src/features/Instructors/actionTypes.js b/src/features/Instructors/actionTypes.js
index 0ddfea42..ca77d0d8 100644
--- a/src/features/Instructors/actionTypes.js
+++ b/src/features/Instructors/actionTypes.js
@@ -2,3 +2,7 @@ export const FETCH_INSTRUCTOR_DATA_REQUEST = 'FETCH_INSTRUCTOR_DATA_REQUEST';
 export const FETCH_INSTRUCTOR_DATA_SUCCESS = 'FETCH_INSTRUCTOR_DATA_SUCCESS';
 export const FETCH_INSTRUCTOR_DATA_FAILURE = 'FETCH_INSTRUCTOR_DATA_FAILURE';
 export const UPDATE_CURRENT_PAGE = 'UPDATE_CURRENT_PAGE';
+
+export const FETCH_CCX_LIST_REQUEST = 'FETCH_CCX_LIST_REQUEST';
+export const FETCH_CCX_LIST_SUCCESS = 'FETCH_CCX_LIST_SUCCESS';
+export const FETCH_CCX_LIST_FAILURE = 'FETCH_CCX_LIST_FAILURE';
diff --git a/src/features/Instructors/data/api.js b/src/features/Instructors/data/api.js
index 9cee8fb5..c87e5bb0 100644
--- a/src/features/Instructors/data/api.js
+++ b/src/features/Instructors/data/api.js
@@ -14,6 +14,14 @@ function getInstructorData(page, filters) {
   );
 }
 
+function getCCXList() {
+  const apiV2BaseUrl = getConfig().COURSE_OPERATIONS_API_V2_BASE_URL;
+  return getAuthenticatedHttpClient().get(
+    `${apiV2BaseUrl}/classes?limit=false`,
+  );
+}
+
 export {
   getInstructorData,
+  getCCXList,
 };