Please don't top-post.
> On 29/11/2016 13:59, Jérémy Lal wrote:
>> 2016-11-29 13:24 GMT+01:00 Paolo Greppi <paolo.gre...@libpf.com>:
>>> Since chai is in Debian I have re-enabled the original tests in
>>> node-cliui, there is one failing and 23 passing:
>>> 1) cliui layoutDSL ignores ansi escape codes when measuring padding:
>>> AssertionError: expected [ Array(5) ] to deeply equal [ Array(5) ]
>>> + expected - actual
>>> " |",
>>> " __| __| | | _ \\",
>>> " | | | | __/",
>>> " \\__| _| \\__,_| \\___|",
>>> - ""
>>> + " "
>>> at Context.<anonymous>
>>> at callFn (/usr/lib/nodejs/mocha/lib/runnable.js:223:21)
>>> at Test.Runnable.run (/usr/lib/nodejs/mocha/lib/runnable.js:216:7)
>>> at Runner.runTest (/usr/lib/nodejs/mocha/lib/runner.js:373:10)
>>> at /usr/lib/nodejs/mocha/lib/runner.js:451:12
>>> at next (/usr/lib/nodejs/mocha/lib/runner.js:298:14)
>>> at /usr/lib/nodejs/mocha/lib/runner.js:308:7
>>> at next (/usr/lib/nodejs/mocha/lib/runner.js:246:23)
>>> at Immediate._onImmediate (/usr/lib/nodejs/mocha/lib/runner.js:275:5)
>>> at processImmediate [as _immediateCallback] (timers.js:383:17)
>>> Has anybody an idea why ?
>> (please push commits so that we can sbuild easily).
>> It works locally, but not in a clean sbuild chroot.
>> No idea why. Something with how much console features are supported.
>> I'm interested to find out why !
>>> Or should I just skip the failing test using the mocha skip
>>> chain-function (http://mochajs.org/#inclusive-tests) ?
>> Good idea - it's better than a patch !
2016-12-02 10:02 GMT+01:00 Paolo Greppi <paolo.gre...@libpf.com>:
> I have investigated and found that the same fix for node-chalk
> fixes the issue here.
> No need to path to skip tests.
> This package is basically ready for release with the tests enabled.
> Would Jérémy sponsor its upload ?
Sure, please fix
I: node-cliui source: quilt-patch-missing-description 00-enable_chalk.diff