Skip to content
This repository has been archived by the owner on Oct 15, 2020. It is now read-only.

Commit

Permalink
test: fixing test breaks
Browse files Browse the repository at this point in the history
* Marked async-wrap test flaky
* Fixed inspector test which tries to enable Profiler

PR-URL: #277
Reviewed-By: Kunal Pathak <kunal.pathak@microsoft.com>
  • Loading branch information
kfarnung committed Jun 5, 2017
1 parent 99c3ef5 commit 7528ca6
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 6 deletions.
18 changes: 12 additions & 6 deletions test/inspector/test-inspector-debug-brk.js
Original file line number Diff line number Diff line change
Expand Up @@ -25,14 +25,20 @@ function testBreakpointOnStart(session) {
{ 'method': 'Debugger.setPauseOnExceptions',
'params': {'state': 'none'} },
{ 'method': 'Debugger.setAsyncCallStackDepth',
'params': {'maxDepth': 0} },
{ 'method': 'Profiler.enable' },
{ 'method': 'Profiler.setSamplingInterval',
'params': {'interval': 100} },
'params': {'maxDepth': 0} }
];

if (process.jsEngine !== 'chakracore') {
commands.push(
{ 'method': 'Profiler.enable' },
{ 'method': 'Profiler.setSamplingInterval',
'params': {'interval': 100} });
}

commands.push(
{ 'method': 'Debugger.setBlackboxPatterns',
'params': {'patterns': []} },
{ 'method': 'Runtime.runIfWaitingForDebugger' }
];
{ 'method': 'Runtime.runIfWaitingForDebugger' });

session
.sendInspectorCommands(commands)
Expand Down
1 change: 1 addition & 0 deletions test/parallel/parallel.status
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,7 @@ test-fs-watch-encoding : FAIL, PASS
[$jsEngine==chakracore]
test-assert-checktag : PASS,FLAKY
test-async-wrap-getasyncid : PASS,FLAKY
test-async-wrap-promise-after-enabled : PASS,FLAKY
test-buffer-bindingobj-no-zerofill : PASS,FLAKY
test-buffer-sharedarraybuffer : PASS,FLAKY
test-child-process-promisified : PASS,FLAKY
Expand Down

0 comments on commit 7528ca6

Please sign in to comment.