Quantcast
Channel: Node.js Tools for Visual Studio
Viewing all articles
Browse latest Browse all 4630

Commented Issue: NodeTests.dll failures [760]

$
0
0
In recent runs, these 2 tests are failing on both machines (see rolling test results for details):

```
NodejsTests.SourceMapTests.TestMappingLineAndColumn
NodejsTests.Debugger.DebuggerTests.TestInterpreterArguments
```

Comments: `TestInterpreterArguments`: First test fails due to following changes: https://nodejstools.codeplex.com/SourceControl/changeset/05836efb1207b4b723fd1b5c15a00987cda67615#Nodejs/Product/Nodejs/Debugger/ExceptionHandler.cs Since we add to break never all exceptions from list it should be fixed like this: ``` c# TestDebuggerSteps( "PassedArgs.js", new[] { new TestStep(action: TestAction.ResumeThread, expectedEntryPointHit: 0), new TestStep(action: TestAction.ResumeProcess, expectedExceptionRaised: new ExceptionInfo("Error", "Error: Invalid args", 3)), new TestStep(action: TestAction.ResumeProcess, expectedExitCode: 8), }, exceptionTreatments: CollectExceptionTreatments(ExceptionHitTreatment.BreakAlways, "Error") ); ``` Note: after that change we've a duplicate values in the array: ``` c# ... "Error(ENOENT)", "Error(MODULE_NOT_FOUND)", "SyntaxError" ``` Second test fails due to ivalid order of command line arguments: ``` >node 42 --debug-brk=5858 PassedArgs.js module.js:340 throw err; ^ Error: Cannot find module '...\42' at Function.Module._resolveFilename (module.js:338:15) at Function.Module._load (module.js:280:25) at Function.Module.runMain (module.js:497:10) at startup (node.js:119:16) at node.js:902:3 ``` It was changed in the following commit: https://nodejstools.codeplex.com/SourceControl/changeset/e02a298e1f78d99af0e4496137c90619a0408ef2#Nodejs/Product/Nodejs/Debugger/NodeDebugger.cs

Viewing all articles
Browse latest Browse all 4630

Trending Articles



<script src="https://jsc.adskeeper.com/r/s/rssing.com.1596347.js" async> </script>