stylelint-scss selector-no-union-class-name breaks entire stylelint process

[jantimon]

Reproduction css example:

@mixin demo() {
   @content;
}

.a-selector {

  @include demo() { 
      button:hover & {
  	background:purple
     }
  }

}

If you run this code against the selector-no-union-class-name test it will crash the entire process.

I did some research and found out that the reason is that the parent of button:hover & { is @include demo() { and therefore there is no selector property for the parent in this line:

https://github.com/kristerkari/stylelint-scss/blob/147b44cf10725fbf7b08a33885a6f7b7b8df4bb9/src/rules/selector-no-union-class-name/index.js#L38

The good news is that the selector is in the parent of the parent:

So probably we could just iterate over the parents until we find the closest parent with a selector` property.
If none can be found we just skip the test.

What do you think?

[kristerkari]

Thanks! I'll have a look at it

[kristerkari]

@jantimon I made a PR with a fix: #346

[jantimon]

Added a small improvement over #346 in #347