Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature/counters #107

Merged
merged 5 commits into from Jun 14, 2018
Merged
Changes from 1 commit
Commits
File filter
Filter file types
Jump to
Jump to file
Failed to load files.

Always

Just for now

Fixes for counters
  • Loading branch information
zarembsky authored and IAmThePan committed Jun 14, 2018
commit ad579edfb9b070234d48d19065dc2af802ba0891
@@ -13,6 +13,7 @@

import {
GET_SUMMARY_DATA,
GET_CLIQZ_MODULE_DATA,
UPDATE_TRACKER_COUNTS,
UPDATE_GHOSTERY_PAUSED,
UPDATE_SITE_POLICY,
@@ -21,18 +22,14 @@ import {
import { sendMessageInPromise } from '../utils/msg';

/**
* Fetch summary data from background
* @deprecated in favor or PanelActions.getPanelData()
* Fetch cliqz modules data from background
* @return {Object} dispatch
*/
export function getSummaryData(tabId) {
export function getCliqzModuleData(tabId) {
return function (dispatch) {
return sendMessageInPromise('getPanelData', {
tabId,
view: 'summary',
}).then((data) => {
sendMessageInPromise('getCliqzModuleData').then((data) => {
dispatch({
type: GET_SUMMARY_DATA,
type: GET_CLIQZ_MODULE_DATA,
data,
});
});
@@ -38,19 +38,6 @@ msg.sendMessageInPromise = jest.fn(messageType => new Promise((resolve, reject)
}));

describe('app/panel/actions/SummaryActions.js', () => {
test('getSummaryData action should return correctly', () => {
const initialState = {};
const store = mockStore(initialState);

const data = testData;
const expectedPayload = { data, type: GET_SUMMARY_DATA };

return store.dispatch(summaryActions.getSummaryData()).then(() => {
const actions = store.getActions();
expect(actions).toEqual([expectedPayload]);
});
});

test('updateTrackerCounts action should return correctly', () => {
const initialState = {};
const store = mockStore(initialState);
@@ -66,6 +66,12 @@ class Summary extends React.Component {
this.setTrackerLatency(this.props);
this.updateSiteNotScanned(this.props);
}
/**
* Lifecycle event
*/
componentDidMount() {
this.props.actions.getCliqzModuleData();
}

/**
* Lifecycle event
@@ -13,6 +13,7 @@

// panel
export const GET_PANEL_DATA = 'GET_PANEL_DATA';
export const GET_CLIQZ_MODULE_DATA = 'GET_CLIQZ_MODULE_DATA';
export const SHOW_NOTIFICATION = 'SHOW_NOTIFICATION';
export const TOGGLE_CLIQZ_FEATURE = 'TOGGLE_CLIQZ_FEATURE';
export const CLOSE_NOTIFICATION = 'CLOSE_NOTIFICATION';
@@ -41,14 +41,6 @@ describe('app/panel/reducers/summary.js', () => {
expect(summaryReducer(undefined, {})).toEqual(initialState);
});

test('reducer correctly handles GET_SUMMARY_DATA', () => {
const data = { test: true };
const action = { data, type: GET_SUMMARY_DATA };
const initState = Immutable({});

expect(summaryReducer(initState, action)).toEqual(data);
});

test('reducer correctly handles UPDATE_GHOSTERY_PAUSED', () => {
const data = { time: null, ghosteryPaused: true };
const action = { data, type: UPDATE_GHOSTERY_PAUSED };
@@ -15,6 +15,7 @@

import {
GET_SUMMARY_DATA,
GET_CLIQZ_MODULE_DATA,
UPDATE_GHOSTERY_PAUSED,
UPDATE_SITE_POLICY,
UPDATE_TRACKER_COUNTS
@@ -49,6 +50,10 @@ export default (state = initialState, action) => {
case GET_SUMMARY_DATA: {
return Object.assign({}, state, action.data);
}
case GET_CLIQZ_MODULE_DATA: {
console.log("GET_CLIQZ_MODULE_DATA CALLED", action.data);
return Object.assign({}, state, { adBlock: action.data.adblock, antiTracking: action.data.antitracking });
}
case UPDATE_GHOSTERY_PAUSED: {
return Object.assign({}, state, { paused_blocking: action.data.ghosteryPaused, paused_blocking_timeout: action.data.time });
}
ProTip! Use n and p to navigate between commits in a pull request.