| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
1 parent f31a611 commit e64ebac
2 files changed
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -23,7 +23,10 @@ function startCLI(args, flags = [], spawnOpts = {}) { | |||
| 23 | 23 | if (this === child.stderr) { | |
| 24 | 24 | stderrOutput += chunk; | |
| 25 | 25 | } | |
| 26 | - outputBuffer.push(chunk); | ||
| 26 | + // TODO(trott): Figure out why the "breakpoints restored." message appears | ||
| 27 | + // in unpredictable places especially on AIX in CI. We shouldn't be | ||
| 28 | + // excluding it, but it gets in the way of the output checking for tests. | ||
| 29 | + outputBuffer.push(chunk.replace(/\n*\d+ breakpoints restored\.\n*/mg, '')); | ||
| 27 | 30 | } | |
| 28 | 31 | ||
| 29 | 32 | function getOutput() { | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -40,16 +40,7 @@ function launchTarget(...args) { | |||
| 40 | 40 | }) | |
| 41 | 41 | .then(() => cli.command('sb("alive.js", 3)')) | |
| 42 | 42 | .then(() => cli.waitFor(/break/)) | |
| 43 | - // TODO: There is a known issue on AIX and some other operating systems | ||
| 44 | - // where the breakpoints aren't properly resolved yet when we reach this | ||
| 45 | - // point. Eventually that should be figured out but for now we don't | ||
| 46 | - // want to fail builds because of it. | ||
| 47 | - // What it should be: | ||
| 48 | - // | ||
| 49 | - // .then(() => cli.waitForPrompt()) | ||
| 50 | - // | ||
| 51 | - // What we're diong for now: | ||
| 52 | - .then(() => cli.waitFor(/>\s+(?:\n1 breakpoints restored\.)?$/)) | ||
| 43 | + .then(() => cli.waitForPrompt()) | ||
| 53 | 44 | .then(() => { | |
| 54 | 45 | assert.match( | |
| 55 | 46 | cli.output, | |
| Back | FazBrowse Home | New Git URL |
0 commit comments