scss-lint 0.29.0 → 0.30.0

data/spec/scss_lint/linter/selector_format_spec.rb

@@ -269,4 +269,333 @@ describe SCSSLint::Linter::SelectorFormat do
       it { should_not report_lint }
     end
   end
+
+  context 'when using a unique `id_convention`' do
+    let(:linter_config) { { 'id_convention' => 'snake_case' } }
+
+    context 'and actual id is correct' do
+      let(:css) { <<-CSS }
+        .hyphenated-lowercase {}
+        #snake_case {}
+      CSS
+
+      it { should_not report_lint }
+    end
+
+    context 'and actual id is incorrect' do
+      let(:css) { <<-CSS }
+        .hyphenated-lowercase {}
+        #hyphenated-lowercase {}
+      CSS
+
+      it { should report_lint line: 2 }
+    end
+
+    context 'and something else uses the `id_convention`' do
+      let(:css) { <<-CSS }
+        .snake_case {}
+        #hyphenated-lowercase {}
+      CSS
+
+      it { should report_lint line: 1 }
+    end
+  end
+
+  context 'when using a unique `class_convention`' do
+    let(:linter_config) { { 'class_convention' => 'camel_case' } }
+
+    context 'and actual class is correct' do
+      let(:css) { <<-CSS }
+        .camelCase {}
+        #hyphenated-lowercase {}
+      CSS
+
+      it { should_not report_lint }
+    end
+
+    context 'and actual class is incorrect' do
+      let(:css) { <<-CSS }
+        .hyphenated-lowercase {}
+        #hyphenated-lowercase {}
+      CSS
+
+      it { should report_lint line: 1 }
+    end
+
+    context 'and something else uses the `class_convention`' do
+      let(:css) { <<-CSS }
+        .hyphenated-lowercase {}
+        #camelCase {}
+      CSS
+
+      it { should report_lint line: 2 }
+    end
+  end
+
+  context 'when using a unique `placeholder_convention`' do
+    let(:linter_config) { { 'placeholder_convention' => 'snake_case' } }
+
+    context 'and actual placeholder is correct' do
+      let(:css) { <<-CSS }
+        .hyphenated-lowercase {}
+        %snake_case {}
+      CSS
+
+      it { should_not report_lint }
+    end
+
+    context 'and actual placeholder is incorrect' do
+      let(:css) { <<-CSS }
+        .hyphenated-lowercase {}
+        %hyphenated-lowercase {}
+      CSS
+
+      it { should report_lint line: 2 }
+    end
+
+    context 'and something else uses the `placeholder_convention`' do
+      let(:css) { <<-CSS }
+        .snake_case {}
+        %snake_case {}
+      CSS
+
+      it { should report_lint line: 1 }
+    end
+  end
+
+  context 'when using a unique `element_convention`' do
+    let(:linter_config) do
+      {
+        'convention' => 'camel_case',
+        'element_convention' => 'hyphenated-lowercase'
+      }
+    end
+
+    context 'and actual element is correct' do
+      let(:css) { <<-CSS }
+        hyphenated-lowercase {}
+        #camelCase {}
+      CSS
+
+      it { should_not report_lint }
+    end
+
+    context 'and actual element is incorrect' do
+      let(:css) { <<-CSS }
+        camelCase {}
+        #camelCase {}
+      CSS
+
+      it { should report_lint line: 1 }
+    end
+
+    context 'and something else uses the `element_convention`' do
+      let(:css) { <<-CSS }
+        hyphenated-lowercase {}
+        #hyphenated-lowercase {}
+      CSS
+
+      it { should report_lint line: 2 }
+    end
+  end
+
+  context 'when using a unique `pseudo_convention`' do
+    let(:linter_config) do
+      {
+        'convention' => 'camel_case',
+        'pseudo_convention' => 'hyphenated-lowercase'
+      }
+    end
+
+    context 'and actual pseudo is correct' do
+      let(:css) { <<-CSS }
+        :hyphenated-lowercase {}
+        #camelCase {}
+      CSS
+
+      it { should_not report_lint }
+    end
+
+    context 'and actual pseudo is incorrect' do
+      let(:css) { <<-CSS }
+        :camelCase {}
+        #camelCase {}
+      CSS
+
+      it { should report_lint line: 1 }
+    end
+
+    context 'and something else uses the `pseudo_convention`' do
+      let(:css) { <<-CSS }
+        :hyphenated-lowercase {}
+        #hyphenated-lowercase {}
+      CSS
+
+      it { should report_lint line: 2 }
+    end
+  end
+
+  context 'when using a unique `attribute_convention`' do
+    let(:linter_config) do
+      {
+        'convention' => 'camel_case',
+        'attribute_convention' => 'hyphenated-lowercase'
+      }
+    end
+
+    context 'and actual attribute is correct' do
+      let(:css) { <<-CSS }
+        [hyphenated-lowercase] {}
+        #camelCase {}
+      CSS
+
+      it { should_not report_lint }
+    end
+
+    context 'and actual attribute is incorrect' do
+      let(:css) { <<-CSS }
+        [camelCase] {}
+        #camelCase {}
+      CSS
+
+      it { should report_lint line: 1 }
+    end
+
+    context 'and something else uses the `attribute_convention`' do
+      let(:css) { <<-CSS }
+        [hyphenated-lowercase] {}
+        #hyphenated-lowercase {}
+      CSS
+
+      it { should report_lint line: 2 }
+    end
+  end
+
+  context 'when using a blend of unique conventions' do
+    let(:linter_config) do
+      {
+        'convention' => 'camel_case',
+        'element_convention' => 'hyphenated-lowercase',
+        'attribute_convention' => 'snake_case',
+        'class_convention' => /[a-z]+\-\-[a-z]+/
+      }
+    end
+
+    context 'and everything is correct' do
+      let(:css) { <<-CSS }
+        #camelCase {}
+        hyphenated-lowercase {}
+        [snake_case] {}
+        .foo--bar {}
+      CSS
+
+      it { should_not report_lint }
+    end
+
+    context 'some things are not correct' do
+      let(:css) { <<-CSS }
+        #camelCase {}
+        camelCase {}
+        [snake_case] {}
+        .fooBar {}
+      CSS
+
+      it { should report_lint line: 2 }
+      it { should report_lint line: 4 }
+    end
+
+    context 'other things are not correct' do
+      let(:css) { <<-CSS }
+        #snake_case {}
+        hyphenated-lowercase {}
+        [camelCase] {}
+        .foo--bar {}
+      CSS
+
+      it { should report_lint line: 1 }
+      it { should report_lint line: 3 }
+    end
+  end
+
+  context 'when the BEM convention is specified' do
+    let(:linter_config) { { 'convention' => 'BEM' } }
+
+    context 'when a name contains no underscores or hyphens' do
+      let(:css) { '.block {}' }
+
+      it { should_not report_lint }
+    end
+
+    context 'when a name contains single hyphen' do
+      let(:css) { '.b-block {}' }
+
+      it { should_not report_lint }
+    end
+
+    context 'when a name contains multiple hyphens' do
+      let(:css) { '.b-block-name {}' }
+
+      it { should_not report_lint }
+    end
+
+    context 'when a name contains multiple hyphens in a row' do
+      let(:css) { '.b-block--modifier {}' }
+
+      it { should report_lint }
+    end
+
+    context 'when a name contains a single underscore' do
+      let(:css) { '.block_modifier {}' }
+
+      it { should report_lint }
+    end
+
+    context 'when a block has name-value modifier' do
+      let(:css) { '.block_modifier_value {}' }
+
+      it { should_not report_lint }
+    end
+
+    context 'when a block has name-value modifier with lots of hyphens' do
+      let(:css) { '.b-block-name_modifier-name-here_value-name-here {}' }
+
+      it { should_not report_lint }
+    end
+
+    context 'when a name has double underscores' do
+      let(:css) { '.b-block__element {}' }
+
+      it { should_not report_lint }
+    end
+
+    context 'when element goes after block with modifier' do
+      let(:css) { '.block_modifier_value__element {}' }
+
+      it { should report_lint }
+    end
+
+    context 'when element has modifier' do
+      let(:css) { '.block__element_modifier_value {}' }
+
+      it { should_not report_lint }
+    end
+
+    context 'when element has not paired modifier' do
+      let(:css) { '.block__element_modifier {}' }
+
+      it { should report_lint }
+    end
+
+    context 'when element has hypenated modifier' do
+      let(:css) { '.block__element--modifier {}' }
+
+      it { should report_lint }
+    end
+
+    context 'when element has hypenated paired modifier' do
+      let(:css) { '.block__element--modifier_value {}' }
+
+      it { should report_lint }
+    end
+  end
 end

data/spec/scss_lint/linter/space_after_property_colon_spec.rb

@@ -55,6 +55,20 @@ describe SCSSLint::Linter::SpaceAfterPropertyColon do
 
       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(:css) { "[class~='\#{$test}'] { width: 100%; }" }
+
+        it { should_not report_lint }
+      end
+
+      context 'and property name is not followed by a space' do
+        let(:css) { "[class~='\#{$test}'] { width:100%; }" }
+
+        it { should report_lint }
+      end
+    end
   end
 
   context 'when no spaces are allowed' do

data/spec/scss_lint/linter/space_after_property_name_spec.rb

@@ -20,4 +20,18 @@ describe SCSSLint::Linter::SpaceAfterPropertyName do
 
     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(:css) { "[class~='\#{$test}'] { width: 100%; }" }
+
+      it { should_not report_lint }
+    end
+
+    context 'and property name is not followed by a space' do
+      let(:css) { "[class~='\#{$test}'] { width : 100%; }" }
+
+      it { should report_lint }
+    end
+  end
 end

data/spec/scss_lint/linter/space_before_brace_spec.rb

@@ -11,6 +11,13 @@ describe SCSSLint::Linter::SpaceBeforeBrace do
       CSS
 
       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
@@ -34,6 +41,26 @@ describe SCSSLint::Linter::SpaceBeforeBrace do
 
       it { should report_lint line: 2 }
     end
+
+    context 'when brace is preceded by a new line' do
+      let(:css) { <<-CSS }
+        .parent
+        {
+          @at-root .child
+          {
+          }
+        }
+      CSS
+
+      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
@@ -44,6 +71,12 @@ describe SCSSLint::Linter::SpaceBeforeBrace do
       CSS
 
       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
@@ -63,6 +96,22 @@ describe SCSSLint::Linter::SpaceBeforeBrace do
 
       it { should report_lint line: 1 }
     end
+
+    context 'when brace is preceded by a new line' do
+      let(:css) { <<-CSS }
+        @each $item in $list
+        {
+        }
+      CSS
+
+      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
@@ -73,6 +122,12 @@ describe SCSSLint::Linter::SpaceBeforeBrace do
       CSS
 
       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
@@ -92,6 +147,22 @@ describe SCSSLint::Linter::SpaceBeforeBrace do
 
       it { should report_lint line: 1 }
     end
+
+    context 'when brace is preceded by a new line' do
+      let(:css) { <<-CSS }
+        @for $i from $start to $end
+        {
+        }
+      CSS
+
+      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
@@ -102,6 +173,12 @@ describe SCSSLint::Linter::SpaceBeforeBrace do
       CSS
 
       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
@@ -121,6 +198,22 @@ describe SCSSLint::Linter::SpaceBeforeBrace do
 
       it { should report_lint line: 1 }
     end
+
+    context 'when brace is preceded by a new line' do
+      let(:css) { <<-CSS }
+        @while $condition
+        {
+        }
+      CSS
+
+      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
@@ -131,6 +224,12 @@ describe SCSSLint::Linter::SpaceBeforeBrace do
       CSS
 
       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
@@ -169,6 +268,22 @@ describe SCSSLint::Linter::SpaceBeforeBrace do
 
       it { should report_lint line: 3 }
     end
+
+    context 'when brace is preceded by a new line' do
+      let(:css) { <<-CSS }
+        p
+        {
+        }
+      CSS
+
+      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
@@ -180,6 +295,12 @@ describe SCSSLint::Linter::SpaceBeforeBrace do
         CSS
 
         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
@@ -199,6 +320,22 @@ describe SCSSLint::Linter::SpaceBeforeBrace do
 
         it { should report_lint line: 1 }
       end
+
+      context 'when brace is preceded by a new line' do
+        let(:css) { <<-CSS }
+          @function func($arg, $arg2)
+          {
+          }
+        CSS
+
+        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
@@ -209,6 +346,12 @@ describe SCSSLint::Linter::SpaceBeforeBrace do
         CSS
 
         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
@@ -228,6 +371,22 @@ describe SCSSLint::Linter::SpaceBeforeBrace do
 
         it { should report_lint line: 1 }
       end
+
+      context 'when brace is preceded by a new line' do
+        let(:css) { <<-CSS }
+          @function func()
+          {
+          }
+        CSS
+
+        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
 
@@ -240,6 +399,12 @@ describe SCSSLint::Linter::SpaceBeforeBrace do
         CSS
 
         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
@@ -259,6 +424,22 @@ describe SCSSLint::Linter::SpaceBeforeBrace do
 
         it { should report_lint line: 1 }
       end
+
+      context 'when brace is preceded by a new line' do
+        let(:css) { <<-CSS }
+          @mixin mixin($arg, $arg2)
+          {
+          }
+        CSS
+
+        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
@@ -269,6 +450,12 @@ describe SCSSLint::Linter::SpaceBeforeBrace do
         CSS
 
         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
@@ -288,6 +475,22 @@ describe SCSSLint::Linter::SpaceBeforeBrace do
 
         it { should report_lint line: 1 }
       end
+
+      context 'when brace is preceded by a new line' do
+        let(:css) { <<-CSS }
+          @mixin mixin
+          {
+          }
+        CSS
+
+        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
 
@@ -300,6 +503,12 @@ describe SCSSLint::Linter::SpaceBeforeBrace do
         CSS
 
         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
@@ -319,6 +528,22 @@ describe SCSSLint::Linter::SpaceBeforeBrace do
 
         it { should report_lint line: 1 }
       end
+
+      context 'when brace is preceded by a new line' do
+        let(:css) { <<-CSS }
+          @include mixin(arg, arg2)
+          {
+          }
+        CSS
+
+        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
@@ -329,6 +554,12 @@ describe SCSSLint::Linter::SpaceBeforeBrace do
         CSS
 
         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
@@ -348,6 +579,22 @@ describe SCSSLint::Linter::SpaceBeforeBrace do
 
         it { should report_lint line: 1 }
       end
+
+      context 'when brace is preceded by a new line' do
+        let(:css) { <<-CSS }
+          @include mixin
+          {
+          }
+        CSS
+
+        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
 
@@ -358,6 +605,12 @@ describe SCSSLint::Linter::SpaceBeforeBrace do
       CSS
 
       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
@@ -366,39 +619,90 @@ describe SCSSLint::Linter::SpaceBeforeBrace do
       CSS
 
       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
-    let(:css) { <<-CSS }
-      a {
-        content: "{";
-      }
-    CSS
+    context 'and the `style` option is `space`' do
+      let(:css) { <<-CSS }
+        a {
+          content: "{";
+        }
+      CSS
 
-    it { should_not report_lint }
+      it { should_not report_lint }
+    end
+
+    context 'and the `style` option is `new_line`' do
+      let(:linter_config) { { 'style' => 'new_line' } }
+
+      let(:css) { <<-CSS }
+        a
+        {
+          content: "{";
+        }
+      CSS
+
+      it { should_not report_lint }
+    end
   end
 
   context 'when using #{} interpolation' do
-    let(:css) { <<-CSS }
-      @mixin test-mixin($class, $prop, $pixels) {
-        .\#{$class} {
-          \#{$prop}: \#{$pixels}px;
+
+    context 'and the `style` option is `space`' do
+      let(:css) { <<-CSS }
+        @mixin test-mixin($class, $prop, $pixels) {
+          .\#{$class} {
+            \#{$prop}: \#{$pixels}px;
+          }
         }
-      }
-    CSS
+      CSS
 
-    it { should_not report_lint }
+      it { should_not report_lint }
+    end
+
+    context 'and the `style` option is `new_line`' do
+      let(:linter_config) { { 'style' => 'new_line' } }
+
+      let(:css) { <<-CSS }
+        @mixin test-mixin($class, $prop, $pixels)
+        {
+          .\#{$class}
+          {
+            \#{$prop}: \#{$pixels}px;
+          }
+        }
+      CSS
+
+      it { should_not report_lint }
+    end
   end
 
   context 'when using braces in comments' do
     let(:css) { '// ({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) { { 'allow_single_line_padding' => allow_single_line_padding } }
+    let(:linter_config) do
+      {
+        'allow_single_line_padding' => allow_single_line_padding,
+        'style' => style
+      }
+    end
 
     let(:css) { <<-CSS }
       p{ }
@@ -407,22 +711,42 @@ describe SCSSLint::Linter::SpaceBeforeBrace do
       p           { &:before{ } }
     CSS
 
-    context 'and the `allow_extra_spaces` option is true' do
+    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_extra_spaces` option is false' do
+    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
 
@@ -433,6 +757,19 @@ describe SCSSLint::Linter::SpaceBeforeBrace do
     CSS
 
     it { should_not report_lint }
+
+    context 'and the `style` option is `new_line`' do
+      let(:linter_config) { { 'style' => 'new_line' } }
+
+      let(:css) { <<-CSS }
+        @if $token == '{'
+        {
+        }
+      CSS
+
+      it { should_not report_lint }
+    end
+
   end
 
   context 'when curly brace is on own line' do
@@ -442,6 +779,53 @@ describe SCSSLint::Linter::SpaceBeforeBrace do
       }
     CSS
 
-    it { should_not report_lint }
+    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(:css) { <<-CSS }
+        .class {
+        }
+      CSS
+
+      it { should report_lint line: 1 }
+    end
+
+    context 'and the curly brace is preceded by multiple spaces' do
+      let(:css) { <<-CSS }
+        .class    {
+        }
+      CSS
+
+      it { should report_lint line: 1 }
+    end
+
+    context 'and there are multiple levels of nesting' do
+      let(:css) { <<-CSS }
+        ul
+        {
+          li
+          {
+            span
+            {
+              a
+              {
+              }
+            }
+          }
+        }
+      CSS
+
+      it { should_not report_lint }
+    end
   end
 end