Skip to content

Commit

Permalink
Merge pull request #647 from sergey-shilov/fix/INDY-1251
Browse files Browse the repository at this point in the history
INDY-1251: fix data in notifier plugin tests and unskip them.
  • Loading branch information
ashcherbakov authored Apr 27, 2018
2 parents 29c3c2a + c670811 commit 3b2b92d
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions plenum/test/plugin/test_notifier_plugin_manager.py
Original file line number Diff line number Diff line change
Expand Up @@ -112,7 +112,6 @@ def testPluginManagerSendMessageUponSuspiciousSpike(
assert sent == 3


@pytest.mark.skip(reason='INDY-1251')
def testNodeSendNodeRequestSpike(pluginManagerWithImportedModules, testNode):
def mockProcessRequest(obj, inc=1):
obj.nodeRequestSpikeMonitorData['accum'] += inc
Expand All @@ -134,14 +133,13 @@ def mockProcessRequest(obj, inc=1):
assert testNode.sendNodeRequestSpike() is None
mockProcessRequest(testNode, 2)
assert testNode.sendNodeRequestSpike() is None
mockProcessRequest(testNode, 10)
mockProcessRequest(testNode, 8)
assert testNode.sendNodeRequestSpike() is None
mockProcessRequest(testNode, 100)
sent, found = testNode.sendNodeRequestSpike()
assert sent == 3


@pytest.mark.skip(reason='INDY-1251')
def testMonitorSendClusterThroughputSpike(pluginManagerWithImportedModules,
testNode):
N = 15
Expand All @@ -159,7 +157,7 @@ def testMonitorSendClusterThroughputSpike(pluginManagerWithImportedModules,
assert testNode.monitor.sendClusterThroughputSpike() is None
testNode.monitor.clusterThroughputSpikeMonitorData['accum'] = [2]
assert testNode.monitor.sendClusterThroughputSpike() is None
testNode.monitor.clusterThroughputSpikeMonitorData['accum'] = [4, 6]
testNode.monitor.clusterThroughputSpikeMonitorData['accum'] = [7, 9]
assert testNode.monitor.sendClusterThroughputSpike() is None
testNode.monitor.clusterThroughputSpikeMonitorData['accum'] = [100]
sent, found = testNode.monitor.sendClusterThroughputSpike()
Expand Down

0 comments on commit 3b2b92d

Please sign in to comment.