scss_lint 0.38.0

data/spec/scss_lint/linter/duplicate_property_spec.rb

@@ -0,0 +1,189 @@
+require 'spec_helper'
+
+describe SCSSLint::Linter::DuplicateProperty do
+  context 'when rule set is empty' do
+    let(:scss) { <<-SCSS }
+      p {
+      }
+    SCSS
+
+    it { should_not report_lint }
+  end
+
+  context 'when rule set contains no duplicates' do
+    let(:scss) { <<-SCSS }
+      p {
+        margin: 0;
+        padding: 0;
+      }
+    SCSS
+
+    it { should_not report_lint }
+  end
+
+  context 'when rule set contains duplicates' do
+    let(:scss) { <<-SCSS }
+      p {
+        margin: 0;
+        padding: 0;
+        margin: 1em;
+      }
+    SCSS
+
+    it { should report_lint line: 4 }
+  end
+
+  context 'when rule set contains duplicates but only on vendor-prefixed property values' do
+    let(:scss) { <<-SCSS }
+      p {
+        background: -moz-linear-gradient(center top , #fff, #000);
+        background: -ms-linear-gradient(center top , #fff, #000);
+        background: -o-linear-gradient(center top , #fff, #000);
+        background: -webkit-gradient(linear, left top, left bottom, from(#fff), to(#000));
+        background: linear-gradient(center top , #fff, #000);
+        margin: 1em;
+      }
+    SCSS
+
+    it { should_not report_lint }
+  end
+
+  context 'when placeholder contains duplicates but only on vendor-prefixed values' do
+    let(:scss) { <<-SCSS }
+      %cursor-grabbing {
+        cursor: -moz-grabbing;
+        cursor: -webkit-grabbing;
+        cursor: grabbing;
+      }
+    SCSS
+
+    it { should_not report_lint }
+  end
+
+  context 'when placeholder contains exact duplicates besides for vendor-prefixed values' do
+    let(:scss) { <<-SCSS }
+      %cursor-grabbing {
+        cursor: grabbing;
+        cursor: grabbing;
+      }
+    SCSS
+
+    it { should report_lint line: 3 }
+  end
+
+  context 'when mixin contains duplicates but only on vendor-prefixed values' do
+    let(:scss) { <<-SCSS }
+      @mixin cursor-grabbing($num) {
+        cursor: -moz-grabbing;
+        cursor: -webkit-grabbing;
+        cursor: grabbing;
+      }
+    SCSS
+
+    it { should_not report_lint }
+  end
+
+  context 'when mixin contains duplicates besides for vendor-prefixed values' do
+    let(:scss) { <<-SCSS }
+      @mixin cursor-grabbing($num) {
+        cursor: grabbing;
+        cursor: grabbing;
+      }
+    SCSS
+
+    it { should report_lint line: 3 }
+  end
+
+  context 'when rule set contains exact duplicates besides for vendor-prefixed property values' do
+    let(:scss) { <<-SCSS }
+      p {
+        background: -moz-linear-gradient(center top , #fff, #000);
+        background: -ms-linear-gradient(center top , #fff, #000);
+        background: -o-linear-gradient(center top , #fff, #000);
+        background: -webkit-gradient(linear, left top, left bottom, from(#fff), to(#000));
+        background: linear-gradient(center top , #fff, #000);
+        background: linear-gradient(center top , #fff, #000);
+        margin: 1em;
+      }
+    SCSS
+
+    it { should report_lint line: 7 }
+  end
+
+  context 'when rule set contains non-exact duplicates besides for vendor-prefixed values' do
+    let(:scss) { <<-SCSS }
+      p {
+        background: -moz-linear-gradient(center top , #fff, #000);
+        background: -ms-linear-gradient(center top , #fff, #000);
+        background: -o-linear-gradient(center top , #fff, #000);
+        background: -webkit-gradient(linear, left top, left bottom, from(#fff), to(#000));
+        background: linear-gradient(center top , #fff, #000);
+        background: linear-gradient-b(center top , #fff, #000);
+        margin: 1em;
+      }
+    SCSS
+
+    it { should report_lint line: 7 }
+  end
+
+  context 'when rule set contains multiple duplicates' do
+    let(:scss) { <<-SCSS }
+      p {
+        margin: 0;
+        padding: 0;
+        margin: 1em;
+        margin: 2em;
+      }
+    SCSS
+
+    it { should report_lint line: 4 }
+    it { should report_lint line: 5 }
+  end
+
+  context 'when rule set contains duplicate properties with interpolation' do
+    let(:scss) { <<-SCSS }
+      p {
+        $direction: 'right';
+        margin-\#{direction}: 0;
+        $direction: 'left';
+        margin-\#{direction}: 0;
+      }
+    SCSS
+
+    it { should_not report_lint }
+  end
+
+  context 'when property contains a variable' do
+    let(:scss) { <<-SCSS }
+      p {
+        color: $some-color;
+      }
+    SCSS
+
+    it { should_not report_lint }
+  end
+
+  context 'when property contains a duplicate variable' do
+    let(:scss) { <<-SCSS }
+      p {
+        color: $some-color;
+        color: $some-color;
+      }
+    SCSS
+
+    it { should report_lint line: 3 }
+  end
+
+  context 'when property contains a duplicate value in a nested rule set' do
+    let(:scss) { <<-SCSS }
+      .outer {
+        .inner {
+          color: $some-color;
+          color: $some-color;
+        }
+      }
+    SCSS
+
+    it { should report_lint line: 4 }
+  end
+end

data/spec/scss_lint/linter/else_placement_spec.rb

@@ -0,0 +1,106 @@
+require 'spec_helper'
+
+describe SCSSLint::Linter::ElsePlacement do
+  context 'when @if contains no accompanying @else' do
+    let(:scss) { <<-SCSS }
+      @if $condition {
+        $var: 1;
+      }
+    SCSS
+
+    it { should_not report_lint }
+  end
+
+  context 'when @else is on different line' do
+    let(:scss) { <<-SCSS }
+      @if $condition {
+        $var: 1;
+      }
+      @else {
+        $var: 0;
+      }
+    SCSS
+
+    it { should report_lint line: 4 }
+  end
+
+  context 'when @else is on the same line as previous curly' do
+    let(:scss) { <<-SCSS }
+      @if $condition {
+        $var: 1;
+      } @else {
+        $var: 0;
+      }
+    SCSS
+
+    it { should_not report_lint }
+  end
+
+  context 'when `@else if` is on different line' do
+    let(:scss) { <<-SCSS }
+      @if $condition {
+        $var: 1;
+      }
+      @else if $other_condition {
+        $var: 2;
+      }
+      @else {
+        $var: 0;
+      }
+    SCSS
+
+    it { should report_lint line: 4 }
+    it { should report_lint line: 7 }
+  end
+
+  context 'when `@else if` is on the same line as previous curly' do
+    let(:scss) { <<-SCSS }
+      @if $condition {
+        $var: 1;
+      } @else if $other_condition {
+        $var: 2;
+      } @else {
+        $var: 0;
+      }
+    SCSS
+
+    it { should_not report_lint }
+  end
+
+  context 'when @else is on same line as @if' do
+    let(:scss) { <<-SCSS }
+      @if $condition { $var: 1; } @else { $var: 0; }
+    SCSS
+
+    it { should_not report_lint }
+  end
+
+  context 'when placement of @else on a new line is preferred' do
+    let(:linter_config) { { 'style' => 'new_line' } }
+
+    context 'when @else is on a new line' do
+      let(:scss) { <<-SCSS }
+        @if $condition {
+          $var: 1;
+        }
+        @else {
+          $var: 0;
+        }
+      SCSS
+
+      it { should_not report_lint }
+    end
+
+    context 'when @else is on the same line as previous curly brace' do
+      let(:scss) { <<-SCSS }
+        @if $condition {
+          $var: 1;
+        } @else {
+          $var: 0;
+        }
+      SCSS
+
+      it { should report_lint line: 3 }
+    end
+  end
+end

data/spec/scss_lint/linter/empty_line_between_blocks_spec.rb

@@ -0,0 +1,276 @@
+require 'spec_helper'
+
+describe SCSSLint::Linter::EmptyLineBetweenBlocks do
+  context 'when there are multiple rule sets' do
+    context 'with blank lines between them' do
+      let(:scss) { <<-SCSS }
+        a {
+        }
+
+        b {
+        }
+
+        p {
+        }
+      SCSS
+
+      it { should_not report_lint }
+    end
+
+    context 'with no blank line between them' do
+      let(:scss) { <<-SCSS }
+        a {
+        }
+        b {
+        }
+        p {
+        }
+      SCSS
+
+      it { should report_lint line: 2 }
+      it { should report_lint line: 4 }
+      it { should_not report_lint line: 6 }
+    end
+  end
+
+  context 'when a rule set contains nested rule sets' do
+    context 'and the nested rule sets have no blank line between them' do
+      let(:scss) { <<-SCSS }
+        p {
+          a {
+          }
+          b {
+          }
+        }
+      SCSS
+
+      it { should report_lint line: 3 }
+      it { should_not report_lint line: 5 }
+    end
+
+    context 'and the nested rule sets have a blank line between them' do
+      let(:scss) { <<-SCSS }
+        p {
+          a {
+          }
+
+          b {
+          }
+        }
+      SCSS
+
+      it { should_not report_lint }
+    end
+
+    context 'and the nested rule set has a property preceding it' do
+      let(:scss) { <<-SCSS }
+        p {
+          margin: 0;
+          a {
+          }
+        }
+      SCSS
+
+      it { should report_lint line: 3 }
+    end
+
+    context 'and the nested rule set has a property and empty line preceding it' do
+      let(:scss) { <<-SCSS }
+        p {
+          margin: 0;
+
+          a {
+          }
+        }
+      SCSS
+
+      it { should_not report_lint }
+    end
+  end
+
+  context 'when mixins are defined' do
+    context 'and there is no blank line between them' do
+      let(:scss) { <<-SCSS }
+        @mixin mixin1() {
+        }
+        @mixin mixin2() {
+        }
+      SCSS
+
+      it { should report_lint line: 2 }
+    end
+
+    context 'and there is a blank line between them' do
+      let(:scss) { <<-SCSS }
+        @mixin mixin1() {
+        }
+
+        @mixin mixin2() {
+        }
+      SCSS
+
+      it { should_not report_lint }
+    end
+  end
+
+  context 'when mixins are included without content' do
+    let(:scss) { <<-SCSS }
+      p {
+        @include mixin1();
+        property: blah;
+        @include mixin2(4);
+      }
+    SCSS
+
+    it { should_not report_lint }
+  end
+
+  context 'when mixins are included with content' do
+    context 'and there is no blank line between them' do
+      let(:scss) { <<-SCSS }
+        @include mixin1() {
+          property: value;
+        }
+        @include mixin2() {
+          property: value;
+        }
+      SCSS
+
+      it { should report_lint line: 3 }
+    end
+
+    context 'and there is a blank line between them' do
+      let(:scss) { <<-SCSS }
+        @include mixin1() {
+          property: value;
+        }
+
+        @include mixin2() {
+          property: value;
+        }
+      SCSS
+
+      it { should_not report_lint }
+    end
+  end
+
+  context 'when functions are defined' do
+    context 'and there is no blank line between them' do
+      let(:scss) { <<-SCSS }
+        @function func1() {
+        }
+        @function func2() {
+        }
+      SCSS
+
+      it { should report_lint line: 2 }
+    end
+
+    context 'and there is a blank line between them' do
+      let(:scss) { <<-SCSS }
+        @function func1() {
+        }
+
+        @function func2() {
+        }
+      SCSS
+
+      it { should_not report_lint }
+    end
+  end
+
+  context 'when a rule set is preceded by a comment' do
+    let(:scss) { <<-SCSS }
+      a {
+      }
+
+      // This is a comment
+      p {
+      }
+    SCSS
+
+    it { should_not report_lint }
+  end
+
+  context 'when a rule set is immediately followed by a comment' do
+    let(:scss) { <<-SCSS }
+      a {
+      } // A comment
+
+      a {
+      } /* Another comment */
+    SCSS
+
+    it { should_not report_lint }
+  end
+
+  context 'when rule set is followed by a comment on the next line' do
+    let(:scss) { <<-SCSS }
+      a {
+      }
+      // A trailing comment (often a control comment)
+
+      b {
+      }
+    SCSS
+
+    it { should_not report_lint }
+  end
+
+  context 'when there are multiple placeholder rule sets' do
+    context 'with blank lines between them' do
+      let(:scss) { <<-SCSS }
+        %a {
+        }
+
+        %b {
+        }
+
+        %c {
+        }
+      SCSS
+
+      it { should_not report_lint }
+    end
+
+    context 'with no blank line between them' do
+      let(:scss) { <<-SCSS }
+        %a {
+        }
+        %b {
+        }
+        %c {
+        }
+      SCSS
+
+      it { should report_lint line: 2 }
+      it { should report_lint line: 4 }
+      it { should_not report_lint line: 6 }
+    end
+  end
+
+  context 'when blocks occupy a single line' do
+    let(:linter_config) { { 'ignore_single_line_blocks' => ignore_single_line_blocks } }
+
+    let(:scss) { <<-SCSS }
+      .icon-up { &:before { content: '^'; } }
+      .icon-right { &:before { content: '>'; } }
+      @include some-mixin { content: '<'; }
+      @include some-other-mixin { content: 'v'; }
+    SCSS
+
+    context 'and the `ignore_single_line_blocks` option is true' do
+      let(:ignore_single_line_blocks) { true }
+
+      it { should_not report_lint }
+    end
+
+    context 'and the `ignore_single_line_blocks` option is false' do
+      let(:ignore_single_line_blocks) { false }
+
+      it { should report_lint line: 1 }
+      it { should report_lint line: 2 }
+      it { should report_lint line: 3 }
+    end
+  end
+end