scss_lint 0.38.0

data/spec/scss_lint/linter/space_after_property_colon_spec.rb

@@ -0,0 +1,373 @@
+require 'spec_helper'
+
+describe SCSSLint::Linter::SpaceAfterPropertyColon do
+  let(:linter_config) { { 'style' => style } }
+
+  context 'when one space is preferred' do
+    let(:style) { 'one_space' }
+
+    context 'when the colon after a property is not followed by space' do
+      let(:scss) { <<-SCSS }
+        p {
+          margin:0;
+        }
+      SCSS
+
+      it { should report_lint line: 2 }
+    end
+
+    context 'when colon after property is not followed by space and the semicolon is missing' do
+      let(:scss) { <<-SCSS }
+        p {
+          color:#eee
+        }
+      SCSS
+
+      it { should report_lint line: 2 }
+    end
+
+    context 'when the colon after a property is followed by a space' do
+      let(:scss) { <<-SCSS }
+        p {
+          margin: 0;
+        }
+      SCSS
+
+      it { should_not report_lint }
+    end
+
+    context 'when the colon after a property is surrounded by spaces' do
+      let(:scss) { <<-SCSS }
+        p {
+          margin : bold;
+        }
+      SCSS
+
+      it { should_not report_lint }
+    end
+
+    context 'when the colon after a property is followed by multiple spaces' do
+      let(:scss) { <<-SCSS }
+        p {
+          margin:  bold;
+        }
+      SCSS
+
+      it { should report_lint line: 2 }
+    end
+
+    context 'when interpolation within single quotes is followed by inline property' do
+      context 'and property name is followed by a space' do
+        let(:scss) { "[class~='\#{$test}'] { width: 100%; }" }
+
+        it { should_not report_lint }
+      end
+
+      context 'and property name is not followed by a space' do
+        let(:scss) { "[class~='\#{$test}'] { width:100%; }" }
+
+        it { should report_lint }
+      end
+    end
+
+    context 'when there are nested properties with incorrect spacing' do
+      let(:scss) { <<-SCSS }
+        .class {
+          font: {
+            weight :bold;
+          }
+        }
+      SCSS
+
+      it { should report_lint line: 3 }
+    end
+
+    context 'when the colon after a property is followed by a space and a newline' do
+      let(:scss) { <<-SCSS }
+        p {
+          margin:\s
+        0;
+        }
+      SCSS
+
+      it { should report_lint line: 2 }
+    end
+
+    context 'when the colon after a property is followed by a tab' do
+      let(:scss) { <<-SCSS }
+        p {
+          margin:\t0;
+        }
+      SCSS
+
+      it { should report_lint line: 2 }
+    end
+  end
+
+  context 'when no spaces are allowed' do
+    let(:style) { 'no_space' }
+
+    context 'when the colon after a property is not followed by space' do
+      let(:scss) { <<-SCSS }
+        p {
+          margin:0;
+        }
+      SCSS
+
+      it { should_not report_lint }
+    end
+
+    context 'when colon after property is not followed by space and the semicolon is missing' do
+      let(:scss) { <<-SCSS }
+        p {
+          color:#eee
+        }
+      SCSS
+
+      it { should_not report_lint }
+    end
+
+    context 'when the colon after a property is followed by a space' do
+      let(:scss) { <<-SCSS }
+        p {
+          margin: 0;
+        }
+      SCSS
+
+      it { should report_lint line: 2 }
+    end
+
+    context 'when the colon after a property is surrounded by spaces' do
+      let(:scss) { <<-SCSS }
+        p {
+          margin : bold;
+        }
+      SCSS
+
+      it { should report_lint line: 2 }
+    end
+
+    context 'when the colon after a property is followed by multiple spaces' do
+      let(:scss) { <<-SCSS }
+        p {
+          margin:  bold;
+        }
+      SCSS
+
+      it { should report_lint line: 2 }
+    end
+
+    context 'when the colon after a property is followed by a newline' do
+      let(:scss) { <<-SCSS }
+        p {
+          margin:
+        0;
+        }
+      SCSS
+
+      it { should report_lint line: 2 }
+    end
+
+    context 'when the colon after a property is followed by a tab' do
+      let(:scss) { <<-SCSS }
+        p {
+          margin:\t0;
+        }
+      SCSS
+
+      it { should report_lint line: 2 }
+    end
+  end
+
+  context 'when at least one space is preferred' do
+    let(:style) { 'at_least_one_space' }
+
+    context 'when the colon after a property is not followed by space' do
+      let(:scss) { <<-SCSS }
+        p {
+          margin:0;
+        }
+      SCSS
+
+      it { should report_lint line: 2 }
+    end
+
+    context 'when colon after property is not followed by space and the semicolon is missing' do
+      let(:scss) { <<-SCSS }
+        p {
+          color:#eee
+        }
+      SCSS
+
+      it { should report_lint line: 2 }
+    end
+
+    context 'when the colon after a property is followed by a space' do
+      let(:scss) { <<-SCSS }
+        p {
+          margin: 0;
+        }
+      SCSS
+
+      it { should_not report_lint }
+    end
+
+    context 'when the colon after a property is surrounded by spaces' do
+      let(:scss) { <<-SCSS }
+        p {
+          margin : bold;
+        }
+      SCSS
+
+      it { should_not report_lint }
+    end
+
+    context 'when the colon after a property is followed by multiple spaces' do
+      let(:scss) { <<-SCSS }
+        p {
+          margin:  bold;
+        }
+      SCSS
+
+      it { should_not report_lint }
+    end
+
+    context 'when the colon after a property is followed by multiple spaces and a tab' do
+      let(:scss) { <<-SCSS }
+        p {
+          margin:  \tbold;
+        }
+      SCSS
+
+      it { should report_lint line: 2 }
+    end
+  end
+
+  context 'when one space or newline is preferred' do
+    let(:style) { 'one_space_or_newline' }
+
+    context 'when the colon after a property is not followed by space' do
+      let(:scss) { <<-SCSS }
+        p {
+          margin:0;
+        }
+      SCSS
+
+      it { should report_lint line: 2 }
+    end
+
+    context 'when the colon after a property is followed by a space' do
+      let(:scss) { <<-SCSS }
+        p {
+          margin: 0;
+        }
+      SCSS
+
+      it { should_not report_lint }
+    end
+
+    context 'when the colon after a property is followed by a newline and spaces' do
+      let(:scss) { <<-SCSS }
+        p {
+          background-image:
+              url(https://something.crazy.long/with/paths?and=queries)
+        }
+      SCSS
+
+      it { should_not report_lint }
+    end
+
+    context 'when the colon after a property is followed by a newline and no spaces' do
+      let(:scss) { <<-SCSS }
+        p {
+          background-image:
+url(https://something.crazy.long/with/paths?and=queries)
+        }
+      SCSS
+
+      it { should_not report_lint }
+    end
+
+    context 'when the colon after a property is followed by a space and then a newline' do
+      let(:scss) { <<-SCSS }
+        p {
+          background-image:\s
+url(https://something.crazy.long/with/paths?and=queries)
+        }
+      SCSS
+
+      it { should report_lint line: 2 }
+    end
+  end
+
+  context 'when aligned property values are preferred' do
+    let(:style) { 'aligned' }
+
+    context 'when the colon after a single property is not followed by space' do
+      let(:scss) { <<-SCSS }
+        p {
+          margin:0;
+        }
+      SCSS
+
+      it { should_not report_lint }
+    end
+
+    context 'when the colon after a single property is followed by a space' do
+      let(:scss) { <<-SCSS }
+        p {
+          margin: 0;
+        }
+      SCSS
+
+      it { should_not report_lint }
+    end
+
+    context 'when properties are not aligned' do
+      let(:scss) { <<-SCSS }
+        p {
+          content: 'hello';
+          margin: 0;
+          padding: 0;
+        }
+      SCSS
+
+      it { should report_lint line: 2 }
+    end
+
+    context 'when properties aligned but the names are not' do
+      let(:scss) { <<-SCSS }
+        p {
+          content:  'hello';
+           margin:  0;
+          padding:  0;
+        }
+      SCSS
+
+      it { should_not report_lint }
+    end
+
+    context 'when properties aligned but the names with spaces are not' do
+      let(:scss) { <<-SCSS }
+        p {
+          content :  'hello';
+           margin :  0;
+          padding :  0;
+        }
+      SCSS
+
+      it { should_not report_lint }
+    end
+
+    context 'when properties are aligned' do
+      let(:scss) { <<-SCSS }
+        p {
+          margin:  0;
+          padding: 0;
+        }
+      SCSS
+
+      it { should_not report_lint }
+    end
+  end
+end

data/spec/scss_lint/linter/space_after_property_name_spec.rb

@@ -0,0 +1,37 @@
+require 'spec_helper'
+
+describe SCSSLint::Linter::SpaceAfterPropertyName do
+  context 'when a property name is followed by a space' do
+    let(:scss) { <<-SCSS }
+      p {
+        margin : 0;
+      }
+    SCSS
+
+    it { should report_lint line: 2 }
+  end
+
+  context 'when a property name is not followed by a space' do
+    let(:scss) { <<-SCSS }
+      p {
+        margin: 0;
+      }
+    SCSS
+
+    it { should_not report_lint }
+  end
+
+  context 'when interpolation within single quotes is followed by inline property' do
+    context 'and property name is followed by a space' do
+      let(:scss) { "[class~='\#{$test}'] { width: 100%; }" }
+
+      it { should_not report_lint }
+    end
+
+    context 'and property name is not followed by a space' do
+      let(:scss) { "[class~='\#{$test}'] { width : 100%; }" }
+
+      it { should report_lint }
+    end
+  end
+end

data/spec/scss_lint/linter/space_before_brace_spec.rb

@@ -0,0 +1,829 @@
+require 'spec_helper'
+
+describe SCSSLint::Linter::SpaceBeforeBrace do
+  context 'with a @at-root block' do
+    context 'when brace is preceded by a space' do
+      let(:scss) { <<-SCSS }
+        .parent {
+          @at-root .child {
+          }
+        }
+      SCSS
+
+      it { should_not report_lint }
+
+      context 'and the `style` option is set to `new_line`' do
+        let(:linter_config) { { 'style' => 'new_line' } }
+
+        it { should report_lint line: 1 }
+        it { should report_lint line: 2 }
+      end
+    end
+
+    context 'when brace is preceded by multiple spaces' do
+      let(:scss) { <<-SCSS }
+        .parent {
+          @at-root .child  {
+          }
+        }
+      SCSS
+
+      it { should report_lint line: 2 }
+    end
+
+    context 'when brace is not preceded by a space' do
+      let(:scss) { <<-SCSS }
+        .parent {
+          @at-root .child{
+          }
+        }
+      SCSS
+
+      it { should report_lint line: 2 }
+    end
+
+    context 'when brace is preceded by a new line' do
+      let(:scss) { <<-SCSS }
+        .parent
+        {
+          @at-root .child
+          {
+          }
+        }
+      SCSS
+
+      it { should report_lint line: 2 }
+      it { should report_lint line: 4 }
+
+      context 'and the `style` option is `new_line`' do
+        let(:linter_config) { { 'style' => 'new_line' } }
+
+        it { should_not report_lint }
+      end
+    end
+  end
+
+  context 'with an @each block' do
+    context 'when brace is preceded by a space' do
+      let(:scss) { <<-SCSS }
+        @each $item in $list {
+        }
+      SCSS
+
+      it { should_not report_lint }
+
+      context 'and the `style` option is set to `new_line`' do
+        let(:linter_config) { { 'style' => 'new_line' } }
+
+        it { should report_lint line: 1 }
+      end
+    end
+
+    context 'when brace is preceded by multiple spaces' do
+      let(:scss) { <<-SCSS }
+        @each $item in $list  {
+        }
+      SCSS
+
+      it { should report_lint line: 1 }
+    end
+
+    context 'when brace is not preceded by a space' do
+      let(:scss) { <<-SCSS }
+        @each $item in $list{
+        }
+      SCSS
+
+      it { should report_lint line: 1 }
+    end
+
+    context 'when brace is preceded by a new line' do
+      let(:scss) { <<-SCSS }
+        @each $item in $list
+        {
+        }
+      SCSS
+
+      it { should report_lint line: 2 }
+
+      context 'and the `style` option is `new_line`' do
+        let(:linter_config) { { 'style' => 'new_line' } }
+
+        it { should_not report_lint }
+      end
+    end
+  end
+
+  context 'with a @for block' do
+    context 'when brace is preceded by a space' do
+      let(:scss) { <<-SCSS }
+        @for $i from $start to $end {
+        }
+      SCSS
+
+      it { should_not report_lint }
+
+      context 'and the `style` option is set to `new_line`' do
+        let(:linter_config) { { 'style' => 'new_line' } }
+
+        it { should report_lint line: 1 }
+      end
+    end
+
+    context 'when brace is preceded by multiple spaces' do
+      let(:scss) { <<-SCSS }
+        @for $i from $start to $end  {
+        }
+      SCSS
+
+      it { should report_lint line: 1 }
+    end
+
+    context 'when brace is not preceded by a space' do
+      let(:scss) { <<-SCSS }
+        @for $i from $start to $end{
+        }
+      SCSS
+
+      it { should report_lint line: 1 }
+    end
+
+    context 'when brace is preceded by a new line' do
+      let(:scss) { <<-SCSS }
+        @for $i from $start to $end
+        {
+        }
+      SCSS
+
+      it { should report_lint line: 2 }
+
+      context 'and the `style` option is `new_line`' do
+        let(:linter_config) { { 'style' => 'new_line' } }
+
+        it { should_not report_lint }
+      end
+    end
+  end
+
+  context 'with a @while block' do
+    context 'when brace is preceded by a space' do
+      let(:scss) { <<-SCSS }
+        @while $condition {
+        }
+      SCSS
+
+      it { should_not report_lint }
+
+      context 'and the `style` option is set to `new_line`' do
+        let(:linter_config) { { 'style' => 'new_line' } }
+
+        it { should report_lint line: 1 }
+      end
+    end
+
+    context 'when brace is preceded by multiple spaces' do
+      let(:scss) { <<-SCSS }
+        @while $condition  {
+        }
+      SCSS
+
+      it { should report_lint line: 1 }
+    end
+
+    context 'when brace is not preceded by a space' do
+      let(:scss) { <<-SCSS }
+        @while $condition{
+        }
+      SCSS
+
+      it { should report_lint line: 1 }
+    end
+
+    context 'when brace is preceded by a new line' do
+      let(:scss) { <<-SCSS }
+        @while $condition
+        {
+        }
+      SCSS
+
+      it { should report_lint line: 2 }
+
+      context 'and the `style` option is `new_line`' do
+        let(:linter_config) { { 'style' => 'new_line' } }
+
+        it { should_not report_lint }
+      end
+    end
+  end
+
+  context 'with a rule selector' do
+    context 'when brace is preceded by a space' do
+      let(:scss) { <<-SCSS }
+        p {
+        }
+      SCSS
+
+      it { should_not report_lint }
+
+      context 'and the `style` option is set to `new_line`' do
+        let(:linter_config) { { 'style' => 'new_line' } }
+
+        it { should report_lint line: 1 }
+      end
+    end
+
+    context 'when brace is preceded by multiple spaces' do
+      let(:scss) { <<-SCSS }
+        p  {
+        }
+      SCSS
+
+      it { should report_lint line: 1 }
+    end
+
+    context 'when brace is not preceded by a space' do
+      let(:scss) { <<-SCSS }
+        p{
+        }
+      SCSS
+
+      it { should report_lint line: 1 }
+    end
+
+    context 'when brace is in a single line rule set' do
+      let(:scss) { <<-SCSS }
+        .single-line-selector{color: #f00;}
+      SCSS
+
+      it { should report_lint line: 1 }
+    end
+
+    context 'when brace is following a multi-selector rule set' do
+      let(:scss) { <<-SCSS }
+        .selector1,
+        .selector2,
+        .selector3{
+        }
+      SCSS
+
+      it { should report_lint line: 3 }
+    end
+
+    context 'when brace is preceded by a new line' do
+      let(:scss) { <<-SCSS }
+        p
+        {
+        }
+      SCSS
+
+      it { should report_lint line: 2 }
+
+      context 'and the `style` option is `new_line`' do
+        let(:linter_config) { { 'style' => 'new_line' } }
+
+        it { should_not report_lint }
+      end
+    end
+  end
+
+  context 'with a function declaration' do
+    context 'with arguments' do
+      context 'when brace is preceded by a space' do
+        let(:scss) { <<-SCSS }
+          @function func($arg, $arg2) {
+          }
+        SCSS
+
+        it { should_not report_lint }
+
+        context 'and the `style` option is set to `new_line`' do
+          let(:linter_config) { { 'style' => 'new_line' } }
+
+          it { should report_lint line: 1 }
+        end
+      end
+
+      context 'when brace is preceded by multiple spaces' do
+        let(:scss) { <<-SCSS }
+          @function func($arg, $arg2)  {
+          }
+        SCSS
+
+        it { should report_lint line: 1 }
+      end
+
+      context 'when brace is not preceded by a space' do
+        let(:scss) { <<-SCSS }
+          @function func($arg, $arg2){
+          }
+        SCSS
+
+        it { should report_lint line: 1 }
+      end
+
+      context 'when brace is preceded by a new line' do
+        let(:scss) { <<-SCSS }
+          @function func($arg, $arg2)
+          {
+          }
+        SCSS
+
+        it { should report_lint line: 2 }
+
+        context 'and the `style` option is `new_line`' do
+          let(:linter_config) { { 'style' => 'new_line' } }
+
+          it { should_not report_lint }
+        end
+      end
+    end
+
+    context 'without arguments' do
+      context 'when brace is preceded by a space' do
+        let(:scss) { <<-SCSS }
+          @function func() {
+          }
+        SCSS
+
+        it { should_not report_lint }
+
+        context 'and the `style` option is set to `new_line`' do
+          let(:linter_config) { { 'style' => 'new_line' } }
+
+          it { should report_lint line: 1 }
+        end
+      end
+
+      context 'when brace is preceded by multiple spaces' do
+        let(:scss) { <<-SCSS }
+          @function func()  {
+          }
+        SCSS
+
+        it { should report_lint line: 1 }
+      end
+
+      context 'when brace is not preceded by a space' do
+        let(:scss) { <<-SCSS }
+          @function func(){
+          }
+        SCSS
+
+        it { should report_lint line: 1 }
+      end
+
+      context 'when brace is preceded by a new line' do
+        let(:scss) { <<-SCSS }
+          @function func()
+          {
+          }
+        SCSS
+
+        it { should report_lint line: 2 }
+
+        context 'and the `style` option is `new_line`' do
+          let(:linter_config) { { 'style' => 'new_line' } }
+
+          it { should_not report_lint }
+        end
+      end
+    end
+  end
+
+  context 'with a mixin declaration' do
+    context 'with arguments' do
+      context 'when brace is preceded by a space' do
+        let(:scss) { <<-SCSS }
+          @mixin mixin($arg, $arg2) {
+          }
+        SCSS
+
+        it { should_not report_lint }
+
+        context 'and the `style` option is set to `new_line`' do
+          let(:linter_config) { { 'style' => 'new_line' } }
+
+          it { should report_lint line: 1 }
+        end
+      end
+
+      context 'when brace is preceded by multiple spaces' do
+        let(:scss) { <<-SCSS }
+          @mixin mixin($arg, $arg2)  {
+          }
+        SCSS
+
+        it { should report_lint line: 1 }
+      end
+
+      context 'when brace is not preceded by a space' do
+        let(:scss) { <<-SCSS }
+          @mixin mixin($arg, $arg2){
+          }
+        SCSS
+
+        it { should report_lint line: 1 }
+      end
+
+      context 'when brace is preceded by a new line' do
+        let(:scss) { <<-SCSS }
+          @mixin mixin($arg, $arg2)
+          {
+          }
+        SCSS
+
+        it { should report_lint line: 2 }
+
+        context 'and the `style` option is `new_line`' do
+          let(:linter_config) { { 'style' => 'new_line' } }
+
+          it { should_not report_lint }
+        end
+      end
+    end
+
+    context 'without arguments' do
+      context 'when brace is preceded by a space' do
+        let(:scss) { <<-SCSS }
+          @mixin mixin {
+          }
+        SCSS
+
+        it { should_not report_lint }
+
+        context 'and the `style` option is set to `new_line`' do
+          let(:linter_config) { { 'style' => 'new_line' } }
+
+          it { should report_lint line: 1 }
+        end
+      end
+
+      context 'when brace is preceded by multiple spaces' do
+        let(:scss) { <<-SCSS }
+          @mixin mixin  {
+          }
+        SCSS
+
+        it { should report_lint line: 1 }
+      end
+
+      context 'when brace is not preceded by a space' do
+        let(:scss) { <<-SCSS }
+          @mixin mixin{
+          }
+        SCSS
+
+        it { should report_lint line: 1 }
+      end
+
+      context 'when brace is preceded by a new line' do
+        let(:scss) { <<-SCSS }
+          @mixin mixin
+          {
+          }
+        SCSS
+
+        it { should report_lint line: 2 }
+
+        context 'and the `style` option is `new_line`' do
+          let(:linter_config) { { 'style' => 'new_line' } }
+
+          it { should_not report_lint }
+        end
+      end
+    end
+  end
+
+  context 'with a mixin include with braces' do
+    context 'with arguments' do
+      context 'when brace is preceded by a space' do
+        let(:scss) { <<-SCSS }
+          @include mixin(arg, arg2) {
+          }
+        SCSS
+
+        it { should_not report_lint }
+
+        context 'and the `style` option is set to `new_line`' do
+          let(:linter_config) { { 'style' => 'new_line' } }
+
+          it { should report_lint line: 1 }
+        end
+      end
+
+      context 'when brace is preceded by multiple spaces' do
+        let(:scss) { <<-SCSS }
+          @include mixin(arg, arg2)  {
+          }
+        SCSS
+
+        it { should report_lint line: 1 }
+      end
+
+      context 'when brace is not preceded by a space' do
+        let(:scss) { <<-SCSS }
+          @include mixin(arg, arg2){
+          }
+        SCSS
+
+        it { should report_lint line: 1 }
+      end
+
+      context 'when brace is preceded by a new line' do
+        let(:scss) { <<-SCSS }
+          @include mixin(arg, arg2)
+          {
+          }
+        SCSS
+
+        it { should report_lint line: 2 }
+
+        context 'and the `style` option is `new_line`' do
+          let(:linter_config) { { 'style' => 'new_line' } }
+
+          it { should_not report_lint }
+        end
+      end
+    end
+
+    context 'without arguments' do
+      context 'when brace is preceded by a space' do
+        let(:scss) { <<-SCSS }
+          @include mixin {
+          }
+        SCSS
+
+        it { should_not report_lint }
+
+        context 'and the `style` option is set to `new_line`' do
+          let(:linter_config) { { 'style' => 'new_line' } }
+
+          it { should report_lint line: 1 }
+        end
+      end
+
+      context 'when brace is preceded by multiple spaces' do
+        let(:scss) { <<-SCSS }
+          @include mixin  {
+          }
+        SCSS
+
+        it { should report_lint line: 1 }
+      end
+
+      context 'when brace is not preceded by a space' do
+        let(:scss) { <<-SCSS }
+          @include mixin{
+          }
+        SCSS
+
+        it { should report_lint line: 1 }
+      end
+
+      context 'when brace is preceded by a new line' do
+        let(:scss) { <<-SCSS }
+          @include mixin
+          {
+          }
+        SCSS
+
+        it { should report_lint line: 2 }
+
+        context 'and the `style` option is `new_line`' do
+          let(:linter_config) { { 'style' => 'new_line' } }
+
+          it { should_not report_lint }
+        end
+      end
+    end
+  end
+
+  context 'with a mixin include with no braces' do
+    context 'with arguments' do
+      let(:scss) { <<-SCSS }
+        @include mixin(arg, arg2);
+      SCSS
+
+      it { should_not report_lint }
+
+      context 'and the `style` option is set to `new_line`' do
+        let(:linter_config) { { 'style' => 'new_line' } }
+
+        it { should_not report_lint }
+      end
+    end
+
+    context 'without arguments' do
+      let(:scss) { <<-SCSS }
+        @include mixin;
+      SCSS
+
+      it { should_not report_lint }
+
+      context 'and the `style` option is set to `new_line`' do
+        let(:linter_config) { { 'style' => 'new_line' } }
+
+        it { should_not report_lint }
+      end
+    end
+  end
+
+  context 'when curly brace appears in a string' do
+    context 'and the `style` option is `space`' do
+      let(:scss) { <<-SCSS }
+        a {
+          content: "{";
+        }
+      SCSS
+
+      it { should_not report_lint }
+    end
+
+    context 'and the `style` option is `new_line`' do
+      let(:linter_config) { { 'style' => 'new_line' } }
+
+      let(:scss) { <<-SCSS }
+        a
+        {
+          content: "{";
+        }
+      SCSS
+
+      it { should_not report_lint }
+    end
+  end
+
+  context 'when using #{} interpolation' do
+    context 'and the `style` option is `space`' do
+      let(:scss) { <<-SCSS }
+        @mixin test-mixin($class, $prop, $pixels) {
+          .\#{$class} {
+            \#{$prop}: \#{$pixels}px;
+          }
+        }
+      SCSS
+
+      it { should_not report_lint }
+    end
+
+    context 'and the `style` option is `new_line`' do
+      let(:linter_config) { { 'style' => 'new_line' } }
+
+      let(:scss) { <<-SCSS }
+        @mixin test-mixin($class, $prop, $pixels)
+        {
+          .\#{$class}
+          {
+            \#{$prop}: \#{$pixels}px;
+          }
+        }
+      SCSS
+
+      it { should_not report_lint }
+    end
+  end
+
+  context 'when using braces in comments' do
+    let(:scss) { '// ({x})' }
+
+    it { should_not report_lint }
+
+    context 'and the `style` option is set to `new_line`' do
+      let(:linter_config) { { 'style' => 'new_line' } }
+
+      it { should_not report_lint }
+    end
+  end
+
+  context 'when blocks occupy a single line' do
+    let(:linter_config) do
+      {
+        'allow_single_line_padding' => allow_single_line_padding,
+        'style' => style
+      }
+    end
+
+    let(:scss) { <<-SCSS }
+      p{ }
+      p { }
+      p           { &:before { } }
+      p           { &:before{ } }
+    SCSS
+
+    context 'and the `allow_single_line_padding` option is true' do
+      let(:allow_single_line_padding) { true }
+      let(:style) { 'space' }
+
+      it { should report_lint line: 1 }
+      it { should_not report_lint line: 2 }
+      it { should_not report_lint line: 3 }
+      it { should report_lint line: 4 }
+
+      context 'and the `style` option is `new_line`' do
+        let(:style) { 'new_line' }
+
+        it { should report_lint line: 1 }
+        it { should_not report_lint line: 2 }
+        it { should_not report_lint line: 3 }
+        it { should report_lint line: 4 }
+      end
+    end
+
+    context 'and the `allow_single_line_padding` option is false' do
+      let(:allow_single_line_padding) { false }
+      let(:style) { 'space' }
+
+      it { should report_lint line: 1 }
+      it { should_not report_lint line: 2 }
+      it { should report_lint line: 3 }
+      it { should report_lint line: 4 }
+
+      context 'and the `style` option is `new_line`' do
+        let(:style) { 'new_line' }
+
+        it { should report_lint line: 1 }
+        it { should report_lint line: 2 }
+        it { should report_lint line: 3 }
+        it { should report_lint line: 4 }
+      end
+    end
+  end
+
+  context 'when curly brace is in single quotes' do
+    let(:scss) { <<-SCSS }
+      @if $token == '{' {
+      }
+    SCSS
+
+    it { should_not report_lint }
+
+    context 'and the `style` option is `new_line`' do
+      let(:linter_config) { { 'style' => 'new_line' } }
+
+      let(:scss) { <<-SCSS }
+        @if $token == '{'
+        {
+        }
+      SCSS
+
+      it { should_not report_lint }
+    end
+  end
+
+  context 'when curly brace is on own line' do
+    let(:scss) { <<-SCSS }
+      .class
+      {
+      }
+    SCSS
+
+    it { should report_lint line: 2 }
+
+    context 'and the `style` option is `new_line`' do
+      let(:linter_config) { { 'style' => 'new_line' } }
+
+      it { should_not report_lint }
+    end
+  end
+
+  context 'when the `style` option is `new_line`' do
+    let(:linter_config) { { 'style' => 'new_line' } }
+
+    context 'and the curly brace is preceded by a space' do
+      let(:scss) { <<-SCSS }
+        .class {
+        }
+      SCSS
+
+      it { should report_lint line: 1 }
+    end
+
+    context 'and the curly brace is preceded by multiple spaces' do
+      let(:scss) { <<-SCSS }
+        .class    {
+        }
+      SCSS
+
+      it { should report_lint line: 1 }
+    end
+
+    context 'and there are multiple levels of nesting' do
+      let(:scss) { <<-SCSS }
+        ul
+        {
+          li
+          {
+            span
+            {
+              a
+              {
+              }
+            }
+          }
+        }
+      SCSS
+
+      it { should_not report_lint }
+    end
+  end
+end