scss-lint 0.25.1 → 0.26.0

data/spec/scss_lint/linter/border_zero_spec.rb

@@ -0,0 +1,84 @@
+require 'spec_helper'
+
+describe SCSSLint::Linter::BorderZero do
+  context 'when a rule is empty' do
+    let(:css) { <<-CSS }
+      p {
+      }
+    CSS
+
+    it { should_not report_lint }
+  end
+
+  context 'when a property' do
+    context 'contains a normal border' do
+      let(:css) { <<-CSS }
+        p {
+          border: 1px solid #000;
+        }
+      CSS
+
+      it { should_not report_lint }
+    end
+
+    context 'has a border of 0' do
+      let(:css) { <<-CSS }
+        p {
+          border: 0;
+        }
+      CSS
+
+      it { should_not report_lint }
+    end
+
+    context 'has a border of none' do
+      let(:css) { <<-CSS }
+        p {
+          border: none;
+        }
+      CSS
+
+      it { should report_lint line: 2 }
+    end
+
+    context 'has a border-top of none' do
+      let(:css) { <<-CSS }
+        p {
+          border-top: none;
+        }
+      CSS
+
+      it { should report_lint line: 2 }
+    end
+
+    context 'has a border-right of none' do
+      let(:css) { <<-CSS }
+        p {
+          border-right: none;
+        }
+      CSS
+
+      it { should report_lint line: 2 }
+    end
+
+    context 'has a border-bottom of none' do
+      let(:css) { <<-CSS }
+        p {
+          border-bottom: none;
+        }
+      CSS
+
+      it { should report_lint line: 2 }
+    end
+
+    context 'has a border-left of none' do
+      let(:css) { <<-CSS }
+        p {
+          border-left: none;
+        }
+      CSS
+
+      it { should report_lint line: 2 }
+    end
+  end
+end

data/spec/scss_lint/linter/capitalization_in_selector_spec.rb

@@ -0,0 +1,71 @@
+require 'spec_helper'
+
+describe SCSSLint::Linter::CapitalizationInSelector do
+  context 'when selector is all lowercase' do
+    let(:css) { <<-CSS }
+      span {
+      }
+    CSS
+
+    it { should_not report_lint }
+  end
+
+  context 'when selector is lowercase with non-alphabetic characters' do
+    let(:css) { <<-CSS }
+      .foo-bar {
+      }
+    CSS
+
+    it { should_not report_lint }
+  end
+
+  context 'when selector is camelCase' do
+    let(:css) { <<-CSS }
+      .fooBar {
+      }
+    CSS
+
+    it { should report_lint line: 1 }
+  end
+
+  context 'when selector is UPPER CASE' do
+    let(:css) { <<-CSS }
+      SPAN {
+      }
+    CSS
+
+    it { should report_lint line: 1 }
+  end
+
+  context 'when attribute selector has attribute containing uppercase letters' do
+    let(:css) { <<-CSS }
+      [dataText] {
+      }
+    CSS
+
+    it { should report_lint line: 1 }
+  end
+
+  context 'when attribute selector has value containing uppercase letters' do
+    let(:css) { <<-CSS }
+      [data-text=someText] {
+      }
+    CSS
+
+    it { should_not report_lint }
+  end
+
+  context 'when a selector name is whitelisted' do
+    let(:linter_config) { { 'ignored_names' => %w[Foo] } }
+    let(:css) { '.Foo {}' }
+
+    it { should_not report_lint }
+  end
+
+  context 'when a certain type of selector is ignored' do
+    let(:linter_config) { { 'ignored_types' => %w[class] } }
+    let(:css) { '.Foo {}' }
+
+    it { should_not report_lint }
+  end
+end

data/spec/scss_lint/linter/color_keyword_spec.rb

@@ -0,0 +1,83 @@
+require 'spec_helper'
+
+describe SCSSLint::Linter::ColorKeyword do
+  context 'when a color is specified as a hex' do
+    let(:css) { <<-CSS }
+      p {
+        color: #fff;
+      }
+    CSS
+
+    it { should_not report_lint }
+  end
+
+  context 'when a color is specified as a keyword' do
+    let(:css) { <<-CSS }
+      p {
+        color: white;
+      }
+    CSS
+
+    it { should report_lint line: 2 }
+  end
+
+  context 'when a color keyword exists in a shorthand property' do
+    let(:css) { <<-CSS }
+      p {
+        border: 1px solid black;
+      }
+    CSS
+
+    it { should report_lint line: 2 }
+  end
+
+  context 'when a property contains a color keyword as a string' do
+    let(:css) { <<-CSS }
+      p {
+        content: 'white';
+      }
+    CSS
+
+    it { should_not report_lint }
+  end
+
+  context 'when a function call contains a color keyword' do
+    let(:css) { <<-CSS }
+      p {
+        color: function(red);
+      }
+    CSS
+
+    it { should report_lint line: 2 }
+  end
+
+  context 'when a mixin include contains a color keyword' do
+    let(:css) { <<-CSS }
+      p {
+        @include some-mixin(red);
+      }
+    CSS
+
+    it { should report_lint line: 2 }
+  end
+
+  context 'when the "transparent" color keyword is used' do
+    let(:css) { <<-CSS }
+      p {
+        @include mixin(transparent);
+      }
+    CSS
+
+    it { should_not report_lint }
+  end
+
+  context 'when color keyword appears in a string identifier' do
+    let(:css) { <<-CSS }
+      p {
+        content: content-with-blue-in-name;
+      }
+    CSS
+
+    it { should_not report_lint }
+  end
+end

data/spec/scss_lint/linter/comment_spec.rb

@@ -0,0 +1,55 @@
+require 'spec_helper'
+
+describe SCSSLint::Linter::Comment do
+  context 'when no comments exist' do
+    let(:css) { <<-CSS }
+      p {
+        margin: 0;
+      }
+    CSS
+
+    it { should_not report_lint }
+  end
+
+  context 'when comment is a single line comment' do
+    let(:css) { '// Single line comment' }
+
+    it { should_not report_lint }
+  end
+
+  context 'when comment is a single line comment at the end of a line' do
+    let(:css) { <<-CSS }
+      p {
+        margin: 0; // Comment at end of line
+      }
+    CSS
+
+    it { should_not report_lint }
+  end
+
+  context 'when comment is a multi-line comment' do
+    let(:css) { <<-CSS }
+      h1 {
+        color: #eee;
+      }
+      /*
+       * This is a multi-line comment that should report a lint
+       */
+      p {
+        color: #DDD;
+      }
+    CSS
+
+    it { should report_lint line: 4 }
+  end
+
+  context 'when multi-line-style comment is a at the end of a line' do
+    let(:css) { <<-CSS }
+      h1 {
+        color: #eee; /* This is a comment */
+      }
+    CSS
+
+    it { should report_lint line: 2 }
+  end
+end

data/spec/scss_lint/linter/compass/property_with_mixin_spec.rb

@@ -0,0 +1,55 @@
+require 'spec_helper'
+
+describe SCSSLint::Linter::Compass::PropertyWithMixin do
+  context 'when a rule has a property with an equivalent Compass mixin' do
+    let(:css) { <<-CSS }
+      p {
+        opacity: .5;
+      }
+    CSS
+
+    it { should report_lint line: 2 }
+  end
+
+  context 'when a rule includes a Compass property mixin' do
+    let(:css) { <<-CSS }
+      p {
+        @include opacity(.5);
+      }
+    CSS
+
+    it { should_not report_lint }
+  end
+
+  context 'when a rule does not have a property with a corresponding Compass mixin' do
+    let(:css) { <<-CSS }
+      p {
+        margin: 0;
+      }
+    CSS
+
+    it { should_not report_lint }
+  end
+
+  context 'when a rule includes display: inline-block' do
+    let(:css) { <<-CSS }
+      p {
+        display: inline-block;
+      }
+    CSS
+
+    it { should report_lint line: 2 }
+  end
+
+  context 'when properties are ignored' do
+    let(:linter_config) { { 'ignore' => %w[inline-block] } }
+
+    let(:css) { <<-CSS }
+      p {
+        display: inline-block;
+      }
+    CSS
+
+    it { should_not report_lint }
+  end
+end

data/spec/scss_lint/linter/debug_statement_spec.rb

@@ -0,0 +1,21 @@
+require 'spec_helper'
+
+describe SCSSLint::Linter::DebugStatement do
+  context 'when no debug statements are present' do
+    let(:css) { <<-CSS }
+      p {
+        color: #fff;
+      }
+    CSS
+
+    it { should_not report_lint }
+  end
+
+  context 'when a debug statement is present' do
+    let(:css) { <<-CSS }
+      @debug 'This is a debug statement';
+    CSS
+
+    it { should report_lint line: 1 }
+  end
+end

data/spec/scss_lint/linter/declaration_order_spec.rb

@@ -0,0 +1,94 @@
+require 'spec_helper'
+
+describe SCSSLint::Linter::DeclarationOrder do
+  context 'when rule is empty' do
+    let(:css) { <<-CSS }
+      p {
+      }
+    CSS
+
+    it { should_not report_lint }
+  end
+
+  context 'when rule contains only properties' do
+    let(:css) { <<-CSS }
+      p {
+        background: #000;
+        margin: 5px;
+      }
+    CSS
+
+    it { should_not report_lint }
+  end
+
+  context 'when rule contains only mixins' do
+    let(:css) { <<-CSS }
+      p {
+        @include border-radius(5px);
+        @include box-shadow(5px);
+      }
+    CSS
+
+    it { should_not report_lint }
+  end
+
+  context 'when rule contains no mixins or properties' do
+    let(:css) { <<-CSS }
+      p {
+        a {
+          color: #f00;
+        }
+      }
+    CSS
+
+    it { should_not report_lint }
+  end
+
+  context 'when rule contains properties after nested rules' do
+    let(:css) { <<-CSS }
+      p {
+        a {
+          color: #f00;
+        }
+        color: #f00;
+        margin: 5px;
+      }
+    CSS
+
+    it { should report_lint }
+  end
+
+  context 'when @extend appears before any properties or rules' do
+    let(:css) { <<-CSS }
+      .warn {
+        font-weight: bold;
+      }
+      .error {
+        @extend .warn;
+        color: #f00;
+        a {
+          color: #ccc;
+        }
+      }
+    CSS
+
+    it { should_not report_lint }
+  end
+
+  context 'when @extend appears after a property' do
+    let(:css) { <<-CSS }
+      .warn {
+        font-weight: bold;
+      }
+      .error {
+        color: #f00;
+        @extend .warn;
+        a {
+          color: #ccc;
+        }
+      }
+    CSS
+
+    it { should report_lint }
+  end
+end