@@ -335,7 +348,7 @@ export default class Gradebook extends React.Component {
(
prevPage: state.grades.prevPage,
nextPage: state.grades.nextPage,
assignmnetTypes: state.assignmentTypes.results,
+ areGradesFrozen: state.assignmentTypes.areGradesFrozen,
showSpinner: state.grades.showSpinner,
}
);
diff --git a/src/data/actions/assignmentTypes.js b/src/data/actions/assignmentTypes.js
index 2ac795e..d6638b6 100644
--- a/src/data/actions/assignmentTypes.js
+++ b/src/data/actions/assignmentTypes.js
@@ -2,12 +2,14 @@ import {
STARTED_FETCHING_ASSIGNMENT_TYPES,
GOT_ASSIGNMENT_TYPES,
ERROR_FETCHING_ASSIGNMENT_TYPES,
+ GOT_ARE_GRADES_FROZEN,
} from '../constants/actionTypes/assignmentTypes';
import LmsApiService from '../services/LmsApiService';
const startedFetchingAssignmentTypes = () => ({ type: STARTED_FETCHING_ASSIGNMENT_TYPES });
const errorFetchingAssignmentTypes = () => ({ type: ERROR_FETCHING_ASSIGNMENT_TYPES });
const gotAssignmentTypes = assignmentTypes => ({ type: GOT_ASSIGNMENT_TYPES, assignmentTypes });
+const gotGradesFrozen = areGradesFrozen => ({ type: GOT_ARE_GRADES_FROZEN, areGradesFrozen });
const fetchAssignmentTypes = courseId => (
(dispatch) => {
@@ -16,6 +18,7 @@ const fetchAssignmentTypes = courseId => (
.then(response => response.data)
.then((data) => {
dispatch(gotAssignmentTypes(Object.keys(data.assignment_types)));
+ dispatch(gotGradesFrozen(data.grades_frozen));
})
.catch(() => {
dispatch(errorFetchingAssignmentTypes());
diff --git a/src/data/actions/assignmentTypes.test.js b/src/data/actions/assignmentTypes.test.js
index b13bd55..e4df35b 100644
--- a/src/data/actions/assignmentTypes.test.js
+++ b/src/data/actions/assignmentTypes.test.js
@@ -9,6 +9,7 @@ import {
STARTED_FETCHING_ASSIGNMENT_TYPES,
GOT_ASSIGNMENT_TYPES,
ERROR_FETCHING_ASSIGNMENT_TYPES,
+ GOT_ARE_GRADES_FROZEN,
} from '../constants/actionTypes/assignmentTypes';
const mockStore = configureMockStore([thunk]);
@@ -21,29 +22,30 @@ describe('actions', () => {
describe('fetchAssignmentTypes', () => {
const courseId = 'course-v1:edX+DemoX+Demo_Course';
-
- it('dispatches success action after fetching fetchAssignmentTypes', () => {
- const responseData = {
- assignment_types: {
- Exam: {
- drop_count: 0,
- min_count: 1,
- short_label: 'Exam',
- type: 'Exam',
- weight: 0.25,
- },
- Homework: {
- drop_count: 1,
- min_count: 3,
- short_label: 'Ex',
- type: 'Homework',
- weight: 0.75,
- },
+ const responseData = {
+ assignment_types: {
+ Exam: {
+ drop_count: 0,
+ min_count: 1,
+ short_label: 'Exam',
+ type: 'Exam',
+ weight: 0.25,
},
- };
+ Homework: {
+ drop_count: 1,
+ min_count: 3,
+ short_label: 'Ex',
+ type: 'Homework',
+ weight: 0.75,
+ },
+ },
+ grades_frozen: false,
+ };
+ it('dispatches success action after fetching fetchAssignmentTypes', () => {
const expectedActions = [
{ type: STARTED_FETCHING_ASSIGNMENT_TYPES },
{ type: GOT_ASSIGNMENT_TYPES, assignmentTypes: Object.keys(responseData.assignment_types) },
+ { type: GOT_ARE_GRADES_FROZEN, areGradesFrozen: responseData.grades_frozen },
];
const store = mockStore();
@@ -69,5 +71,21 @@ describe('actions', () => {
expect(store.getActions()).toEqual(expectedActions);
});
});
+
+ it('dispatches frozen grade action with True value after fetching', () => {
+ const expectedActions = [
+ { type: STARTED_FETCHING_ASSIGNMENT_TYPES },
+ { type: GOT_ASSIGNMENT_TYPES, assignmentTypes: Object.keys(responseData.assignment_types) },
+ { type: GOT_ARE_GRADES_FROZEN, areGradesFrozen: true },
+ ];
+ const store = mockStore();
+ responseData.grades_frozen = true;
+ axiosMock.onGet(`${configuration.LMS_BASE_URL}/api/grades/v1/gradebook/${courseId}/grading-info?graded_only=true`)
+ .replyOnce(200, JSON.stringify(responseData));
+
+ return store.dispatch(fetchAssignmentTypes(courseId)).then(() => {
+ expect(store.getActions()).toEqual(expectedActions);
+ });
+ });
});
});
diff --git a/src/data/constants/actionTypes/assignmentTypes.js b/src/data/constants/actionTypes/assignmentTypes.js
index 2acf57c..ba81c5e 100644
--- a/src/data/constants/actionTypes/assignmentTypes.js
+++ b/src/data/constants/actionTypes/assignmentTypes.js
@@ -1,10 +1,12 @@
const STARTED_FETCHING_ASSIGNMENT_TYPES = 'STARTED_FETCHING_ASSIGNMENT_TYPES';
const GOT_ASSIGNMENT_TYPES = 'GOT_ASSIGNMENT_TYPES';
const ERROR_FETCHING_ASSIGNMENT_TYPES = 'ERROR_FETCHING_ASSIGNMENT_TYPES';
+const GOT_ARE_GRADES_FROZEN = 'GOT_ARE_GRADES_FROZEN';
export {
STARTED_FETCHING_ASSIGNMENT_TYPES,
GOT_ASSIGNMENT_TYPES,
ERROR_FETCHING_ASSIGNMENT_TYPES,
+ GOT_ARE_GRADES_FROZEN,
};
diff --git a/src/data/reducers/assignmentTypes.js b/src/data/reducers/assignmentTypes.js
index 13c5490..daca7da 100644
--- a/src/data/reducers/assignmentTypes.js
+++ b/src/data/reducers/assignmentTypes.js
@@ -2,6 +2,7 @@ import {
STARTED_FETCHING_ASSIGNMENT_TYPES,
ERROR_FETCHING_ASSIGNMENT_TYPES,
GOT_ASSIGNMENT_TYPES,
+ GOT_ARE_GRADES_FROZEN,
} from '../constants/actionTypes/assignmentTypes';
const initialState = {
@@ -31,6 +32,13 @@ const assignmentTypes = (state = initialState, action) => {
finishedFetching: true,
errorFetching: true,
};
+ case GOT_ARE_GRADES_FROZEN:
+ return {
+ ...state,
+ areGradesFrozen: action.areGradesFrozen,
+ errorFetching: false,
+ finishedFetching: true,
+ };
default:
return state;
}
diff --git a/src/data/reducers/assignmentTypes.test.js b/src/data/reducers/assignmentTypes.test.js
index 37a2aeb..7224d17 100644
--- a/src/data/reducers/assignmentTypes.test.js
+++ b/src/data/reducers/assignmentTypes.test.js
@@ -3,6 +3,7 @@ import {
STARTED_FETCHING_ASSIGNMENT_TYPES,
ERROR_FETCHING_ASSIGNMENT_TYPES,
GOT_ASSIGNMENT_TYPES,
+ GOT_ARE_GRADES_FROZEN,
} from '../constants/actionTypes/assignmentTypes';
const initialState = {
@@ -51,4 +52,17 @@ describe('assignmentTypes reducer', () => {
type: ERROR_FETCHING_ASSIGNMENT_TYPES,
})).toEqual(expected);
});
+
+ it('updates areGradesFrozen success state', () => {
+ const expected = {
+ ...initialState,
+ errorFetching: false,
+ finishedFetching: true,
+ areGradesFrozen: true,
+ };
+ expect(assignmentTypes(undefined, {
+ type: GOT_ARE_GRADES_FROZEN,
+ areGradesFrozen: true,
+ })).toEqual(expected);
+ });
});