scss-lint-bliss 1.0.0

data/spec/scss_lint/linter/bliss/module_spec.rb

@@ -0,0 +1,254 @@
+require 'spec_helper'
+
+describe SCSSLint::Linter::Bliss::Module do
+  let(:linter_config) { {
+    'module_name' => 'Foo',
+    'ignored_utility_class_prefixes' => ['is']
+  } }
+
+  context 'when a module class is used from a different module' do
+    let(:scss) { <<-SCSS }
+      .Foo {
+        .Bar {
+          color: blue;
+        }
+      }
+    SCSS
+
+    it { should report_lint line: 2 }
+  end
+
+  context 'when a utility class is used' do
+    let(:scss) { <<-SCSS }
+      .Foo {
+        .bar {
+          color: blue;
+        }
+      }
+    SCSS
+
+    it { should report_lint line: 2 }
+  end
+
+  context 'when a utility class is added to ignored_utility_class_prefixes' do
+    let(:scss) { <<-SCSS }
+      .Foo {
+        .isOk &-bar {
+          color: blue;
+        }
+      }
+    SCSS
+
+    it { should_not report_lint }
+  end
+
+  context 'when an ignored_utility_class_prefixes utility class is used as the last descendant selector' do
+    let(:scss) { <<-SCSS }
+      .Foo {
+        &-bar .isNotOk {
+          color: blue;
+        }
+      }
+    SCSS
+
+    it { should report_lint line:2 }
+  end
+
+  context 'when allow_utility_classes_in_module is enabled and a utility class is used (not as last descendant selector)' do
+    let(:linter_config) { {
+      'module_name' => 'Foo',
+      'allow_utility_direct_styling' => true
+    } }
+    let(:scss) { <<-SCSS }
+      .Foo {
+        .utilOk &-bar {
+          color: blue;
+        }
+      }
+    SCSS
+
+    it { should_not report_lint }
+  end
+
+  context 'when any utility class is used as the last descendant selector' do
+    let(:scss) { <<-SCSS }
+      .Foo {
+        &-bar .any-util {
+          color: blue;
+        }
+      }
+    SCSS
+
+    it { should report_lint line:2 }
+  end
+
+  context 'when allow_utility_direct_styling is enabled and any utility class is used as the last descendant selector' do
+    let(:linter_config) { {
+      'module_name' => 'Foo',
+      'allow_utility_direct_styling' => true
+    } }
+    let(:scss) { <<-SCSS }
+      .Foo {
+        &-bar .any-util {
+          color: blue;
+        }
+      }
+    SCSS
+
+    it { should_not report_lint }
+  end
+
+  context 'when allow_element_selector_in_module is false and element selector is used' do
+    let(:linter_config) { {
+      'module_name' => 'Foo',
+      'allow_element_selector_in_module' => false
+    } }
+    let(:scss) { <<-SCSS }
+      .Foo {
+        span {
+          color: blue;
+        }
+      }
+    SCSS
+
+    it { should report_lint line:2 }
+  end
+
+  context 'when allow_element_selector_in_module is true and element selector is used' do
+    let(:linter_config) { {
+      'module_name' => 'Foo',
+      'allow_element_selector_in_module' => true
+    } }
+    let(:scss) { <<-SCSS }
+      .Foo {
+        span {
+          color: blue;
+        }
+      }
+    SCSS
+
+    it { should_not report_lint }
+  end
+
+  context 'when allow_attribute_selector_in_module is false and attribute selector is used' do
+    let(:linter_config) { {
+      'module_name' => 'Foo',
+      'allow_attribute_selector_in_module' => false
+    } }
+    let(:scss) { <<-SCSS }
+      .Foo {
+        [foo="bar"] {
+          color: blue;
+        }
+      }
+    SCSS
+
+    it { should report_lint line:2 }
+  end
+
+  context 'when allow_attribute_selector_in_module is true and attribute selector is used' do
+    let(:linter_config) { {
+      'module_name' => 'Foo',
+      'allow_attribute_selector_in_module' => true
+    } }
+    let(:scss) { <<-SCSS }
+      .Foo {
+        [foo="bar"] {
+          color: blue;
+        }
+      }
+    SCSS
+
+    it { should_not report_lint }
+  end
+
+  context 'when allow_id_selector_in_module is false and id selector is used' do
+    let(:linter_config) { {
+      'module_name' => 'Foo',
+      'allow_id_selector_in_module' => false
+    } }
+    let(:scss) { <<-SCSS }
+      .Foo {
+        #bar {
+          color: blue;
+        }
+      }
+    SCSS
+
+    it { should report_lint line:2 }
+  end
+
+  context 'when allow_id_selector_in_module is true and id selector is used' do
+    let(:linter_config) { {
+      'module_name' => 'Foo',
+      'allow_id_selector_in_module' => true
+    } }
+    let(:scss) { <<-SCSS }
+      .Foo {
+        #bar {
+          color: blue;
+        }
+      }
+    SCSS
+
+    it { should_not report_lint }
+  end
+
+  context 'when allow_module_margin is false and margin is applied to module' do
+    let(:linter_config) { {
+      'module_name' => 'Foo',
+      'allow_module_margin' => false
+    } }
+    let(:scss) { <<-SCSS }
+      .Foo {
+        margin-top: 10px;
+      }
+    SCSS
+
+    it { should report_lint line:2 }
+  end
+
+  context 'when allow_module_margin is true and margin is applied to module' do
+    let(:linter_config) { {
+      'module_name' => 'Foo',
+      'allow_module_margin' => true
+    } }
+    let(:scss) { <<-SCSS }
+      .Foo {
+        margin-top: 10px;
+      }
+    SCSS
+
+    it { should_not report_lint }
+  end
+
+  context 'when allow_module_width is false and width is applied to module' do
+    let(:linter_config) { {
+      'module_name' => 'Foo',
+      'allow_module_width' => false
+    } }
+    let(:scss) { <<-SCSS }
+      .Foo {
+        width: 40%;
+      }
+    SCSS
+
+    it { should report_lint line:2 }
+  end
+
+  context 'when allow_module_width is true and width is applied to module' do
+    let(:linter_config) { {
+      'module_name' => 'Foo',
+      'allow_module_width' => true
+    } }
+    let(:scss) { <<-SCSS }
+      .Foo {
+        width: 40%;
+      }
+    SCSS
+
+    it { should_not report_lint }
+  end
+
+
+end

data/spec/scss_lint/linter/border_zero_spec.rb

@@ -0,0 +1,118 @@
+require 'spec_helper'
+
+describe SCSSLint::Linter::BorderZero do
+  context 'when a rule is empty' do
+    let(:scss) { <<-SCSS }
+      p {
+      }
+    SCSS
+
+    it { should_not report_lint }
+  end
+
+  context 'when a property' do
+    context 'contains a normal border' do
+      let(:scss) { <<-SCSS }
+        p {
+          border: 1px solid #000;
+        }
+      SCSS
+
+      it { should_not report_lint }
+    end
+
+    context 'has a border of 0' do
+      let(:scss) { <<-SCSS }
+        p {
+          border: 0;
+        }
+      SCSS
+
+      it { should_not report_lint }
+    end
+
+    context 'has a border of none' do
+      let(:scss) { <<-SCSS }
+        p {
+          border: none;
+        }
+      SCSS
+
+      it { should report_lint line: 2 }
+    end
+
+    context 'has a border-top of none' do
+      let(:scss) { <<-SCSS }
+        p {
+          border-top: none;
+        }
+      SCSS
+
+      it { should report_lint line: 2 }
+    end
+
+    context 'has a border-right of none' do
+      let(:scss) { <<-SCSS }
+        p {
+          border-right: none;
+        }
+      SCSS
+
+      it { should report_lint line: 2 }
+    end
+
+    context 'has a border-bottom of none' do
+      let(:scss) { <<-SCSS }
+        p {
+          border-bottom: none;
+        }
+      SCSS
+
+      it { should report_lint line: 2 }
+    end
+
+    context 'has a border-left of none' do
+      let(:scss) { <<-SCSS }
+        p {
+          border-left: none;
+        }
+      SCSS
+
+      it { should report_lint line: 2 }
+    end
+  end
+
+  context 'when a convention of `none` is preferred' do
+    let(:linter_config) { { 'convention' => 'none' } }
+
+    context 'and the border is `none`' do
+      let(:scss) { <<-SCSS }
+        p {
+          border: none;
+        }
+      SCSS
+
+      it { should_not report_lint }
+    end
+
+    context 'and the border is `0`' do
+      let(:scss) { <<-SCSS }
+        p {
+          border: 0;
+        }
+      SCSS
+
+      it { should report_lint }
+    end
+
+    context 'and the border is a non-zero value' do
+      let(:scss) { <<-SCSS }
+        p {
+          border: 5px;
+        }
+      SCSS
+
+      it { should_not report_lint }
+    end
+  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(:scss) { <<-SCSS }
+      p {
+        color: #fff;
+      }
+    SCSS
+
+    it { should_not report_lint }
+  end
+
+  context 'when a color is specified as a keyword' do
+    let(:scss) { <<-SCSS }
+      p {
+        color: white;
+      }
+    SCSS
+
+    it { should report_lint line: 2 }
+  end
+
+  context 'when a color keyword exists in a shorthand property' do
+    let(:scss) { <<-SCSS }
+      p {
+        border: 1px solid black;
+      }
+    SCSS
+
+    it { should report_lint line: 2 }
+  end
+
+  context 'when a property contains a color keyword as a string' do
+    let(:scss) { <<-SCSS }
+      p {
+        content: 'white';
+      }
+    SCSS
+
+    it { should_not report_lint }
+  end
+
+  context 'when a function call contains a color keyword' do
+    let(:scss) { <<-SCSS }
+      p {
+        color: function(red);
+      }
+    SCSS
+
+    it { should report_lint line: 2 }
+  end
+
+  context 'when a mixin include contains a color keyword' do
+    let(:scss) { <<-SCSS }
+      p {
+        @include some-mixin(red);
+      }
+    SCSS
+
+    it { should report_lint line: 2 }
+  end
+
+  context 'when the "transparent" color keyword is used' do
+    let(:scss) { <<-SCSS }
+      p {
+        @include mixin(transparent);
+      }
+    SCSS
+
+    it { should_not report_lint }
+  end
+
+  context 'when color keyword appears in a string identifier' do
+    let(:scss) { <<-SCSS }
+      p {
+        content: content-with-blue-in-name;
+      }
+    SCSS
+
+    it { should_not report_lint }
+  end
+end