Skip to content

Commit

Permalink
Added graph complexity check on page load
Browse files Browse the repository at this point in the history
  • Loading branch information
jpellizzari committed Nov 10, 2016
1 parent ed26575 commit ee0bb8e
Show file tree
Hide file tree
Showing 3 changed files with 48 additions and 3 deletions.
36 changes: 34 additions & 2 deletions client/app/scripts/reducers/__tests__/root-test.js
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
const is = require('immutable').is;

import {is, fromJS} from 'immutable';
// Root reducer test suite using Jasmine matchers

describe('RootReducer', () => {
Expand Down Expand Up @@ -44,6 +43,34 @@ describe('RootReducer', () => {
}
};

const topologies = [{
hide_if_empty: true,
name: 'Processes',
rank: 1,
sub_topologies: [],
url: '/api/topology/processes',
fullName: 'Processes',
id: 'processes',
options: [
{
defaultValue: 'hide',
id: 'unconnected',
options: [
{
label: 'Unconnected nodes hidden',
value: 'hide'
}
]
}
],
stats: {
edge_count: 319,
filtered_nodes: 214,
node_count: 320,
nonpseudo_node_count: 320
}
}];

// actions

const ChangeTopologyOptionAction = {
Expand Down Expand Up @@ -468,4 +495,9 @@ describe('RootReducer', () => {
nextState = reducer(nextState, { type: ActionTypes.CLICK_BACKGROUND });
expect(nextState.get('showingHelp')).toBe(false);
});
it('switches to grid mode when complexity is high', () => {
let nextState = initialState.set('currentTopology', fromJS(topologies[0]));
nextState = reducer(nextState, {type: ActionTypes.SET_RECEIVED_NODES_DELTA});
expect(nextState.get('gridMode')).toBe(true);
});
});
10 changes: 9 additions & 1 deletion client/app/scripts/reducers/root.js
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,8 @@ import { EDGE_ID_SEPARATOR } from '../constants/naming';
import { applyPinnedSearches, updateNodeMatches } from '../utils/search-utils';
import { getNetworkNodes, getAvailableNetworks } from '../utils/network-view-utils';
import { findTopologyById, getAdjacentNodes, setTopologyUrlsById, updateTopologyIds,
filterHiddenTopologies, addTopologyFullname, getDefaultTopology } from '../utils/topology-utils';
filterHiddenTopologies, addTopologyFullname, getDefaultTopology, graphExceedsComplexityThresh
} from '../utils/topology-utils';

const log = debug('scope:app-store');
const error = debug('scope:error');
Expand Down Expand Up @@ -501,6 +502,13 @@ export function rootReducer(state = initialState, action) {
}

case ActionTypes.SET_RECEIVED_NODES_DELTA: {
// Turn on the table view if the graph is too complex
if (!state.get('nodesLoaded')) {
const topoStats = state.get('currentTopology').get('stats');
state = graphExceedsComplexityThresh(topoStats)
? state.set('gridMode', true)
: state;
}
return state.set('nodesLoaded', true);
}

Expand Down
5 changes: 5 additions & 0 deletions client/app/scripts/utils/topology-utils.js
Original file line number Diff line number Diff line change
Expand Up @@ -175,3 +175,8 @@ export function getCurrentTopologyUrl(state) {
export function isNodeMatchingQuery(node, query) {
return node.get('label').includes(query) || node.get('subLabel').includes(query);
}

export function graphExceedsComplexityThresh(stats) {
// Check to see if complexity is high. Used to trigger table view on page load.
return (stats.get('node_count') + (2 * stats.get('edge_count'))) > 500;
}

0 comments on commit ee0bb8e

Please sign in to comment.