Skip to content

Commit

Permalink
Issue #SB-16090 merge: Merge pull request #221 from Ajoymaity/release…
Browse files Browse the repository at this point in the history 
…-2.7.0

Issue #SB-16090 test: Unit test for storage and file service
  • Loading branch information
@AmiableAnil
AmiableAnil authored Jan 3, 2020
2 parents 8de1f6e + 1f521ae commit 43fef90
Show file tree
Hide file tree
Showing 6 changed files with 326 additions and 74 deletions.
101 changes: 52 additions & 49 deletions src/content/handlers/get-child-contents-handler.spec.ts
View file
Original file line number Diff line number Diff line change
@@ -1,20 +1,23 @@
import {ChildContentsHandler} from './get-child-contents-handler';
import {DbService} from '../../db';
import {GetContentDetailsHandler} from './get-content-details-handler';
import {ContentEntry} from '../db/schema';
import {ContentMapper} from '../util/content-mapper';
import {of} from 'rxjs';
import {ChildContent, HierarchyInfo} from '..';
import { ChildContentsHandler } from './get-child-contents-handler';
import { DbService } from '../../db';
import { GetContentDetailsHandler } from './get-content-details-handler';
import { ContentEntry } from '../db/schema';
import { ContentMapper } from '../util/content-mapper';
import { of } from 'rxjs';
import { ChildContent, HierarchyInfo } from '..';
import { FileService } from '../../util/file/def/file-service';

describe('ChildContentsHandler', () => {
let childContentHandler: ChildContentsHandler;
const mockDbService: Partial<DbService> = {};
const mockGetContentDetailsHandler: Partial<GetContentDetailsHandler> = {};
const mockFileService: Partial<FileService> = {};

beforeAll(() => {
childContentHandler = new ChildContentsHandler(
mockDbService as DbService,
mockGetContentDetailsHandler as GetContentDetailsHandler
mockGetContentDetailsHandler as GetContentDetailsHandler,
mockFileService as FileService
);
});

Expand Down Expand Up @@ -47,7 +50,7 @@ describe('ChildContentsHandler', () => {
};
const childContentsMap = new Map();
childContentsMap.set('IDENTIFIER', 'd0_id');
ContentMapper.mapContentDBEntryToContent = jest.fn(() => of([]));
ContentMapper.mapContentDBEntryToContent = jest.fn(() => ({hierarchyInfo: {data: ''}}));
const data = JSON.parse(request[ContentEntry.COLUMN_NAME_LOCAL_DATA]);
mockDbService.execute = jest.fn(() => of([]));
// act
Expand All @@ -60,34 +63,6 @@ describe('ChildContentsHandler', () => {
});
});

// it('should not be fetch content', async(done) => {
// // arrange
// const request: ContentEntry.SchemaMap = {
// identifier: 'IDENTIFIER',
// server_data: 'SERVER_DATA',
// local_data: '{"children": [{"ALL": 0}, "do_234", "do_345"]}',
// mime_type: 'MIME_TYPE',
// manifest_version: 'MAINFEST_VERSION',
// content_type: 'CONTENT_TYPE'
// };
// const currentLevel = -1;
// const level = 2;
// ContentMapper.mapContentDBEntryToContent = jest.fn(() => of([]));
// mockDbService.execute = jest.fn(() => of([]));
// const data = JSON.parse(request[ContentEntry.COLUMN_NAME_LOCAL_DATA]);
// console.log(data.children[0].ALL);
// // JSON.parse = jest.fn().mockImplementationOnce(() => {
// // return request[ContentEntry.COLUMN_NAME_LOCAL_DATA];
// // });
// // act
// await childContentHandler.fetchChildrenOfContent(request, currentLevel, level).then(() => {
// // expect(JSON.parse(request[ContentEntry.COLUMN_NAME_LOCAL_DATA])).toHaveBeenCalled();
// expect(data.children[0].ALL).toEqual(0);
// done();
// });
// // assert
// });

it('should parent child relation', (done) => {
// arrange
const request: ContentEntry.SchemaMap = {
Expand All @@ -114,10 +89,10 @@ describe('ChildContentsHandler', () => {
identifier: 'SAMPLE_IDENTIFIER_1',
contentType: 'SAMPLE_CONTENT_TYPE_1'
},
{
identifier: 'SAMPLE_IDENTIFIER_2',
contentType: 'SAMPLE_CONTENT_TYPE_2'
}];
{
identifier: 'SAMPLE_IDENTIFIER_2',
contentType: 'SAMPLE_CONTENT_TYPE_2'
}];
const identifier = 'IDENTIFIER';
mockGetContentDetailsHandler.fetchFromDB = jest.fn(() => of([]));
// act
Expand All @@ -133,10 +108,10 @@ describe('ChildContentsHandler', () => {
identifier: 'SAMPLE_IDENTIFIER_1',
contentType: 'SAMPLE_CONTENT_TYPE_1'
},
{
identifier: 'SAMPLE_IDENTIFIER_2',
contentType: 'SAMPLE_CONTENT_TYPE_2'
}];
{
identifier: 'SAMPLE_IDENTIFIER_2',
contentType: 'SAMPLE_CONTENT_TYPE_2'
}];
const currentIdentifier = 'SAMPLE_CURRENT_IDENTIFIER';
const contentKeyList = ['SAMPLE_IDENTIFIER_1', 'SAMPLE_IDENTIFIER_2',
'SAMPLE_IDENTIFIER_1/SAMPLE_IDENTIFIER_2/SAMPLE_CURRENT_IDENTIFIER'];
Expand All @@ -152,10 +127,10 @@ describe('ChildContentsHandler', () => {
identifier: 'SAMPLE_IDENTIFIER_1',
contentType: 'SAMPLE_CONTENT_TYPE_1'
},
{
identifier: 'SAMPLE_IDENTIFIER_2',
contentType: 'SAMPLE_CONTENT_TYPE_2'
}];
{
identifier: 'SAMPLE_IDENTIFIER_2',
contentType: 'SAMPLE_CONTENT_TYPE_2'
}];
const currentIdentifier = 'SAMPLE_CURRENT_IDENTIFIER';
const contentKeyList = ['SAMPLE_IDENTIFIER_1', 'SAMPLE_IDENTIFIER_2',
'SAMPLE_IDENTIFIER_1/SAMPLE_IDENTIFIER_2/SAMPLE_CURRENT_IDENTIFIER'];
Expand All @@ -164,4 +139,32 @@ describe('ChildContentsHandler', () => {
// assert
done();
});

it('should get ChildIdentifiers From Manifest', (done) => {
// arrange
mockFileService.readAsText = jest.fn(() => {
});
const readAsText = (mockFileService.readAsText as jest.Mock)
.mockResolvedValue(JSON.stringify({ archive: { items: [{ identifier: 'pass' }, { identifier: 'pass-2' }] } }));
readAsText().then((value) => {
return value;
});
// act
childContentHandler.getChildIdentifiersFromManifest('textbook_unit_1').then(() => {
// assert
expect(mockFileService.readAsText).toHaveBeenCalledWith('file:///textbook_unit_1', 'manifest.json');
done();
});
});

it('should get ChildIdentifiers From Manifest for catch part', async(done) => {
// arrange
mockFileService.readAsText = jest.fn(() => Promise.reject('textbook'));
// act
childContentHandler.getChildIdentifiersFromManifest('textbook_unit_1').then(() => {
// assert
expect(mockFileService.readAsText).toHaveBeenCalledWith('file:///textbook_unit_1', 'manifest.json');
done();
});
});
});
47 changes: 47 additions & 0 deletions src/content/handlers/get-content-heirarchy-handler.spec.ts
View file
Original file line number Diff line number Diff line change
@@ -0,0 +1,47 @@
import { GetContentHeirarchyHandler } from './get-content-heirarchy-handler';
import { ApiService } from '../..';
import { ContentServiceConfig, ContentDetailRequest } from '..';
import { of } from 'rxjs';

describe('GetContentHeirarchyHandler', () => {
let getContentHeirarchyHandler: GetContentHeirarchyHandler;
const mockApiService: Partial<ApiService> = {};
const mockContentServiceConfig: Partial<ContentServiceConfig> = {};

beforeAll(() => {
getContentHeirarchyHandler = new GetContentHeirarchyHandler(
mockApiService as ApiService,
mockContentServiceConfig as ContentServiceConfig
);
});

beforeEach(() => {
jest.clearAllMocks();
});

it('should be create a instance of getContentHeirarchyHandler', () => {
expect(getContentHeirarchyHandler).toBeTruthy();
});

it('shpuld handle mapContentFromContentHeirarchyData by invoked handle()', (done) => {
// arrange
const request: ContentDetailRequest = {
contentId: 'do_123'
};
mockApiService.fetch = jest.fn(() => of({
body: {
result: {
content: {
children: ['child-1', 'child-2']
}
}
}
}));
// act
getContentHeirarchyHandler.handle(request).subscribe(() => {
expect(mockApiService.fetch).toHaveBeenCalled();
done();
});
// assert
});
});
31 changes: 17 additions & 14 deletions src/content/impl/content-service-impl.spec.ts
View file
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,7 @@ describe('ContentServiceImpl', () => {
};

beforeAll(() => {
container.bind<ContentService>(InjectionTokens.CONTENT_SERVICE).to(ContentServiceImpl);
container.bind<ContentService>(InjectionTokens.CONTENT_SERVICE).to(ContentServiceImpl).inTransientScope();
container.bind<SdkConfig>(InjectionTokens.SDK_CONFIG).toConstantValue(mockSdkConfig as SdkConfig);
container.bind<ApiService>(InjectionTokens.API_SERVICE).toConstantValue(mockApiService as ApiService);
container.bind<DbService>(InjectionTokens.DB_SERVICE).toConstantValue(mockDbService as DbService);
Expand Down Expand Up @@ -553,6 +553,12 @@ describe('ContentServiceImpl', () => {
});
it('should be find child content', (done) => {
// arrange
(ChildContentsHandler as any as jest.Mock<ChildContentsHandler>).mockImplementation(() => {
return {
fetchChildrenOfContent: jest.fn(() => Promise.resolve({}))
};
});
contentService = container.get(InjectionTokens.CONTENT_SERVICE);
const hierarInfoData: HierarchyInfo[] = [{
identifier: 'd0_123',
contentType: 'content_type'
Expand All @@ -567,19 +573,15 @@ describe('ContentServiceImpl', () => {
JSON.parse = jest.fn().mockImplementationOnce(() => {
return mockDbService.read;
});
(ChildContentsHandler as any as jest.Mock<ChildContentsHandler>).mockImplementation(() => {
return {
fetchChildrenOfContent: jest.fn(() => Promise.resolve({}))
};
});

ArrayUtil.joinPreservingQuotes = jest.fn(() => of([]));
mockDbService.execute = jest.fn(() => of([]));
// act
contentService.getChildContents(request).subscribe(() => {
// assert
expect(mockDbService.read).toHaveBeenCalled();
expect(ArrayUtil.joinPreservingQuotes).toHaveBeenCalled();
expect(mockDbService.execute).toBeCalled();
// expect(ArrayUtil.joinPreservingQuotes).toHaveBeenCalled();
// expect(mockDbService.execute).toBeCalled();
done();
});
// assert
Expand Down Expand Up @@ -611,16 +613,17 @@ describe('ContentServiceImpl', () => {
});
it('should used for search content', (done) => {
// arrange
const request: ContentSearchCriteria = {
limit: 1,
offset: 2
};
(SearchContentHandler as any as jest.Mock<SearchContentHandler>).mockImplementation(() => {
return {
getSearchContentRequest: jest.fn(() => of('')),
mapSearchResponse: jest.fn(() => of(''))
getSearchContentRequest: jest.fn(() => ({filter: {}})),
mapSearchResponse: jest.fn(() => ({id: 'sid'}))
};
});
const request: ContentSearchCriteria = {
limit: 1,
offset: 2
};
contentService = container.get(InjectionTokens.CONTENT_SERVICE);

mockSharedPreferences.getString = jest.fn(() => of([]));
spyOn(mockApiService, 'fetch').and.returnValue(of({
Expand Down
Loading

0 comments on commit 43fef90

Please sign in to comment.