diff --git a/src/pages/trainer/training/edit/actions/fetchTrainingContent.ts b/src/pages/trainer/training/edit/actions/fetchTrainingContent.ts index 1eeb89c..d738ef4 100644 --- a/src/pages/trainer/training/edit/actions/fetchTrainingContent.ts +++ b/src/pages/trainer/training/edit/actions/fetchTrainingContent.ts @@ -3,7 +3,7 @@ import {trainerApi} from '../../../../../rest-api'; export const fetchTrainingContentStarted = (trainingId: number) => { return (dispatcher) => { - const promise = trainerApi.getTrainingConentByTrainingId(trainingId); + const promise = trainerApi.getTrainingContentByTrainingId(trainingId); promise.then( (data) => dispatcher(fetchTrainingContentCompleted(data)), diff --git a/src/pages/trainer/training/edit/actions/spec/fetchTrainingContent.spec.ts b/src/pages/trainer/training/edit/actions/spec/fetchTrainingContent.spec.ts index c1aa4bc..dca4a61 100644 --- a/src/pages/trainer/training/edit/actions/spec/fetchTrainingContent.spec.ts +++ b/src/pages/trainer/training/edit/actions/spec/fetchTrainingContent.spec.ts @@ -52,7 +52,7 @@ describe('fetchTrainingContentStarted', () => { }); }); - it('should calls to getTrainingConentByTrainingId and expect payload', sinon.test((done) => { + it('should calls to getTrainingContentByTrainingId and expect payload', sinon.test((done) => { // Arrange const sinon: sinon.SinonStatic = this; @@ -60,7 +60,7 @@ describe('fetchTrainingContentStarted', () => { const expectedContent = 'Test content'; const store = mockStore([]); - const getTrainingConentByTrainingIdStub = sinon.stub(trainerApi, 'getTrainingConentByTrainingId', () => { + const getTrainingContentByTrainingIdStub = sinon.stub(trainerApi, 'getTrainingContentByTrainingId', () => { return { then: (callback) => { callback(expectedContent); @@ -72,7 +72,7 @@ describe('fetchTrainingContentStarted', () => { store.dispatch(fetchTrainingContentStarted(trainingId)) .then(() => { // Assert - expect(getTrainingConentByTrainingIdStub.calledWith(trainingId)).to.be.true; + expect(getTrainingContentByTrainingIdStub.calledWith(trainingId)).to.be.true; expect(store.getActions()[0].payload).to.equal(expectedContent); done(); diff --git a/src/rest-api/trainer/spec/trainerApi.spec.ts b/src/rest-api/trainer/spec/trainerApi.spec.ts index 89edfd8..ce4af74 100644 --- a/src/rest-api/trainer/spec/trainerApi.spec.ts +++ b/src/rest-api/trainer/spec/trainerApi.spec.ts @@ -1,6 +1,8 @@ import {expect} from 'chai'; import * as trainerMockData from '../trainerMockData'; +import * as exerciseEvaluationMockData from '../exerciseEvaluationMockedData'; import {EditTrainingSummary} from '../../../model/trainer/editTrainingSummary'; +import { ExerciseEvaluation } from '../../../model/trainer/deliveryEvaluation'; import {trainerApi} from '../trainerApi'; describe('TrainerApi', () => { @@ -9,7 +11,7 @@ describe('TrainerApi', () => { expect(trainerApi).not.to.be.undefined; }); - describe('getTrainingConentByTrainingId', () => { + describe('getTrainingContentByTrainingId', () => { it('returns expected content by id equals 1', sinon.test((done) => { // Arrange const sinon: sinon.SinonStatic = this; @@ -26,7 +28,7 @@ describe('TrainerApi', () => { ); // Act - const promise = trainerApi.getTrainingConentByTrainingId(id); + const promise = trainerApi.getTrainingContentByTrainingId(id); promise.then((content) => { // Assert @@ -54,7 +56,7 @@ describe('TrainerApi', () => { ); // Act - const promise = trainerApi.getTrainingConentByTrainingId(id); + const promise = trainerApi.getTrainingContentByTrainingId(id); promise.then((content) => { // Assert @@ -63,4 +65,30 @@ describe('TrainerApi', () => { }); }).bind(this)); }); + + describe('getExerciseEvaluationById', () => { + it('returns expected Exercise Evaluation by id equals 1', sinon.test((done) => { + // Arrange + const sinon: sinon.SinonStatic = this; + const id = 123; + const expectedMockEvaluation: ExerciseEvaluation = { + id: 123, + name: 'Exercise delivery - Module UX + Design', + studentDelivery:null + }; + const exerciseEvaluationMockDataStub = sinon.stub(exerciseEvaluationMockData, 'exerciseEvaluationMockedData', + [expectedMockEvaluation], + ); + + // Act + const promise = trainerApi.getExerciseEvaluationById(id); + + promise.then((content) => { + // Assert + expect(content.name).to.equal(expectedMockEvaluation.name); + done(); + }); + }).bind(this)); + + }); }); diff --git a/src/rest-api/trainer/trainerApi.ts b/src/rest-api/trainer/trainerApi.ts index 8f4b999..e2e085d 100644 --- a/src/rest-api/trainer/trainerApi.ts +++ b/src/rest-api/trainer/trainerApi.ts @@ -1,7 +1,9 @@ import {trainingContentMockData} from './trainerMockData'; +import {exerciseEvaluationMockedData} from './exerciseEvaluationMockedData'; +import { ExerciseEvaluation } from '../../model/trainer/deliveryEvaluation'; class TrainerApi { - public getTrainingConentByTrainingId(id: number): Promise { + public getTrainingContentByTrainingId(id: number): Promise { let content: string; const trainingContent = trainingContentMockData.filter((trainingMockData) => { @@ -14,6 +16,23 @@ class TrainerApi { return Promise.resolve(content); } + public getExerciseEvaluationById(id: number): Promise { + let evaluation: ExerciseEvaluation; + + const evaluationData = exerciseEvaluationMockedData.find((evaluationItem) => { + return evaluationItem.id === id; + }); + + if (evaluationData) { + evaluation = evaluationData; + } + + return Promise.resolve(evaluationData); + } + + public saveExerciseEvaluation(exerciseEvaluation:ExerciseEvaluation){ + return Promise.resolve(exerciseEvaluation); + } } export const trainerApi = new TrainerApi();