can't generate less file with mixin multiple parameters about sc5-styleguide

[cyberixae]

I was able to reproduce this. After adding a two parameter mixin some tests fail and hang.

[cyberixae]

Preventing the tests from hanging is not trivial. The tests hang because the styleguide itself never finishes running. We currently run the styleguide in the before phase of Mocha tests and the before phase does not support timeouts. Therefore if we wish to prevent the tests from hanging we would need to create a promise in the before phase and resolve the promise as a part of executing the first actual test.

[cyberixae]

I have tracked the reason of style guide hanging down to the gonzales.srcToAST call in kss-splitter.js The call never returns and therefore causes style guide to hang. We need to find out whether or not upgrading Gonzales will fix this problem.

[cyberixae]

The problem can be reproduced by adding the following less notation to test/projects/less-project/source/styles/style.less Not sure what kind of chapter numbering would be good for language specific tests. Of course while we are at it we could also add multi-parameter mixin tests for other style languages.

// Section with multiparameter less mixin
//
// Markup:
// <div class="mixin-call">Section markup</div>
//
// Styleguide 5.0

.border-mixin(@width; @color) {
  border: @width solid @color;
}

.mixin-call {
  .border-mixin(1px; @color-blue);
}

[cyberixae]

Upgrading Gonzales did not seem to fix the problem.

[cognivator]

+1
We are a mixin-heavy LESS shop. Loving the tool, save for this issue, and a couple others (issue #546, issue #510)