Page MenuHomePhabricator
Differential D7932 Diff 26843 lib/reducers/report-store-reducer.test.js

Changeset View

Standalone View

View Options

lib/reducers/report-store-reducer.test.js

Show First 20 Lines • Show All 202 Lines • ▼ Show 20 Linesdescribe('updateReportsEnabledActionType test', () => {
}); });
}); });
describe('queueReportsActionType test', () => { describe('queueReportsActionType test', () => {
test('should handle adding enabled report', () => { test('should handle adding enabled report', () => {
const action = { const action = {
type: 'QUEUE_REPORTS', type: 'QUEUE_REPORTS',
payload: { payload: {
reports: [mockErrorReport], reports: [
{ ...mockErrorReport, id: 'new-1' },
{ ...mockInconsistencyReport, id: 'new-2' },
],
}, },
}; };
const reportStore = { const reportStore = {
queuedReports: [mockErrorReport], queuedReports: [mockErrorReport],
enabledReports: { enabledReports: {
crashReports: true, crashReports: true,
inconsistencyReports: false, inconsistencyReports: false,
mediaReports: false, mediaReports: false,
}, },
}; };
const { reportStore: result } = reduceReportStore(reportStore, action, [ const { reportStore: result } = reduceReportStore(reportStore, action, [
mockErrorReport, { ...mockErrorReport, id: 'new-3' },
mockInconsistencyReport, { ...mockInconsistencyReport, id: 'new-4' },
kamilAuthorUnsubmitted
Done
Inline Actions

now logic is based on ids - so modifying the test to use different ids for different objects

kamil: now logic is based on `id`s - so modifying the test to use different ids for different objects
]); ]);
expect(result.queuedReports).toHaveLength(3); expect(result.queuedReports).toHaveLength(3);
const enabledReports = result.queuedReports.filter( const enabledReports = result.queuedReports.filter(
report => report.type === reportTypes.ERROR, report => report.type === reportTypes.ERROR,
); );
const notEnabledReportsExist = result.queuedReports.some( const notEnabledReportsExist = result.queuedReports.some(
report => report.type !== reportTypes.ERROR, report => report.type !== reportTypes.ERROR,
); );
▲ Show 20 Lines • Show All 70 Lines • Show Last 20 Lines