scss-lint 0.25.1 → 0.26.0

data/spec/scss_lint/linter/mergeable_selector_spec.rb

@@ -0,0 +1,283 @@
+require 'spec_helper'
+
+describe SCSSLint::Linter::MergeableSelector do
+  context 'when single root' do
+    let(:css) { <<-CSS }
+      p {
+      }
+    CSS
+
+    it { should_not report_lint }
+  end
+
+  context 'when different roots' do
+    let(:css) { <<-CSS }
+      p {
+        background: #000;
+        margin: 5px;
+      }
+      a {
+        background: #000;
+        margin: 5px;
+      }
+    CSS
+
+    it { should_not report_lint }
+  end
+
+  context 'when different roots with matching inner rules' do
+    let(:css) { <<-CSS }
+      p {
+        background: #000;
+        margin: 5px;
+        .foo {
+          color: red;
+        }
+      }
+      a {
+        background: #000;
+        margin: 5px;
+        .foo {
+          color: red;
+        }
+      }
+    CSS
+
+    it { should_not report_lint }
+  end
+
+  context 'when multi-selector roots and parital rule match' do
+    let(:css) { <<-CSS }
+      p, a {
+        background: #000;
+        margin: 5px;
+        .foo {
+          color: red;
+        }
+      }
+      a {
+        background: #000;
+        margin: 5px;
+        .foo {
+          color: red;
+        }
+      }
+    CSS
+
+    it { should_not report_lint }
+  end
+
+  context 'when nested and unnested selectors match' do
+    let(:css) { <<-CSS }
+      a.current {
+        background: #000;
+        margin: 5px;
+        .foo {
+          color: red;
+        }
+      }
+      a {
+        &.current {
+          background: #000;
+          margin: 5px;
+          .foo {
+            color: red;
+          }
+        }
+      }
+    CSS
+
+    context 'when force_nesting is enabled' do
+      let(:linter_config) { { 'force_nesting' => true } }
+      it { should report_lint }
+    end
+
+    context 'when force_nesting is disabled' do
+      let(:linter_config) { { 'force_nesting' => false } }
+      it { should_not report_lint }
+    end
+  end
+
+  context 'when same class roots' do
+    let(:css) { <<-CSS }
+      .warn {
+        font-weight: bold;
+      }
+      .warn {
+        color: #f00;
+        @extend .warn;
+        a {
+          color: #ccc;
+        }
+      }
+    CSS
+
+    it { should report_lint }
+  end
+
+  context 'when same compond selector roots' do
+    let(:css) { <<-CSS }
+      .warn .alert {
+        font-weight: bold;
+      }
+      .warn .alert {
+        color: #f00;
+        @extend .warn;
+        a {
+          color: #ccc;
+        }
+      }
+    CSS
+
+    it { should report_lint }
+  end
+
+  context 'when same class roots separated by another class' do
+    let(:css) { <<-CSS }
+      .warn {
+        font-weight: bold;
+      }
+      .foo {
+        color: red;
+      }
+      .warn {
+        color: #f00;
+        @extend .warn;
+        a {
+          color: #ccc;
+        }
+      }
+    CSS
+
+    it { should report_lint }
+  end
+
+  context 'when rule in a mixin @include matches a root root' do
+    let(:css) { <<-CSS }
+      p {
+        font-weight: bold;
+      }
+      @include enhance(small-tablet) {
+        p {
+          font-weight: normal;
+        }
+      }
+    CSS
+
+    it { should_not report_lint }
+  end
+
+  context 'when rule in a mixin definition matches a root rule' do
+    let(:css) { <<-CSS }
+      p {
+        font-weight: bold;
+      }
+      @mixin my-mixin {
+        p {
+          font-weight: normal;
+        }
+      }
+    CSS
+
+    it { should_not report_lint }
+  end
+
+  context 'when rule in a media directive matches a root rule' do
+    let(:css) { <<-CSS }
+      p {
+        font-weight: bold;
+      }
+      @media only screen and (min-device-pixel-ratio: 1.5) {
+        p {
+          font-weight: normal;
+        }
+      }
+    CSS
+
+    it { should_not report_lint }
+  end
+
+  context 'when rule in a keyframes directive matches a root rule' do
+    let(:css) { <<-CSS }
+      @keyframes slideouttoleft {
+        from {
+          transform: translateX(0);
+        }
+
+        to {
+          transform: translateX(-100%);
+        }
+      }
+
+      @keyframes slideouttoright {
+        from {
+          transform: translateX(0);
+        }
+
+        to {
+          transform: translateX(100%);
+        }
+      }
+    CSS
+
+    it { should_not report_lint }
+  end
+
+  context 'when there are duplicate rules nested in a rule set' do
+    let(:css) { <<-CSS }
+      .foo {
+        .bar {
+          font-weight: bold;
+        }
+        .baz {
+          font-weight: bold;
+        }
+        .bar {
+          color: #ff0;
+        }
+      }
+    CSS
+
+    it { should report_lint }
+  end
+
+  context 'when force_nesting is enabled' do
+    let(:linter_config) { { 'force_nesting' => true } }
+
+    context 'when one of the duplicate rules is in a comma sequence' do
+      let(:css) { <<-CSS }
+        .foo,
+        .bar {
+          color: #000;
+        }
+        .foo {
+          color: #f00;
+        }
+      CSS
+
+      it { should_not report_lint }
+    end
+
+    context 'when rules start with the same prefix but are not the same' do
+      let(:css) { <<-CSS }
+        .foo {
+          color: #000;
+        }
+        .foobar {
+          color: #f00;
+        }
+      CSS
+
+      it { should_not report_lint }
+    end
+
+    context 'when a rule contains interpolation' do
+      let(:css) { <<-CSS }
+        .\#{$class-name} {}
+        .foobar {}
+      CSS
+
+      it { should_not report_lint }
+    end
+  end
+end

data/spec/scss_lint/linter/name_format_spec.rb

@@ -0,0 +1,206 @@
+require 'spec_helper'
+
+describe SCSSLint::Linter::NameFormat do
+  context 'when no variable, functions, or mixin declarations exist' do
+    let(:css) { <<-CSS }
+      p {
+        margin: 0;
+      }
+    CSS
+
+    it { should_not report_lint }
+  end
+
+  context 'when a variable name contains a hyphen' do
+    let(:css) { '$content-padding: 10px;' }
+
+    it { should_not report_lint }
+  end
+
+  context 'when a variable name contains an underscore' do
+    let(:css) { '$content_padding: 10px;' }
+
+    it { should report_lint line: 1 }
+  end
+
+  context 'when a variable name contains an uppercase character' do
+    let(:css) { '$contentPadding: 10px;' }
+
+    it { should report_lint line: 1 }
+  end
+
+  context 'when a function is declared with a capital letter' do
+    let(:css) { <<-CSS }
+      @function badFunction() {
+      }
+    CSS
+
+    it { should report_lint line: 1 }
+  end
+
+  context 'when a function is declared with an underscore' do
+    let(:css) { <<-CSS }
+      @function bad_function() {
+      }
+    CSS
+
+    it { should report_lint line: 1 }
+  end
+
+  context 'when a mixin is declared with a capital letter' do
+    let(:css) { <<-CSS }
+      @mixin badMixin() {
+      }
+    CSS
+
+    it { should report_lint line: 1 }
+  end
+
+  context 'when a mixin is declared with an underscore' do
+    let(:css) { <<-CSS }
+      @mixin bad_mixin() {
+      }
+    CSS
+
+    it { should report_lint line: 1 }
+  end
+
+  context 'when no invalid usages exist' do
+    let(:css) { <<-CSS }
+      p {
+        margin: 0;
+      }
+    CSS
+
+    it { should_not report_lint }
+  end
+
+  context 'when a referenced variable name has a capital letter' do
+    let(:css) { <<-CSS }
+      p {
+        margin: $badVar;
+      }
+    CSS
+
+    it { should report_lint line: 2 }
+  end
+
+  context 'when a referenced variable name has an underscore' do
+    let(:css) { <<-CSS }
+      p {
+        margin: $bad_var;
+      }
+    CSS
+
+    it { should report_lint line: 2 }
+  end
+
+  context 'when a referenced function name has a capital letter' do
+    let(:css) { <<-CSS }
+      p {
+        margin: badFunc();
+      }
+    CSS
+
+    it { should report_lint line: 2 }
+  end
+
+  context 'when a referenced function name has an underscore' do
+    let(:css) { <<-CSS }
+      p {
+        margin: bad_func();
+      }
+    CSS
+
+    it { should report_lint line: 2 }
+  end
+
+  context 'when an included mixin name has a capital letter' do
+    let(:css) { '@include badMixin();' }
+
+    it { should report_lint line: 1 }
+  end
+
+  context 'when an included mixin name has an underscore' do
+    let(:css) { '@include bad_mixin();' }
+
+    it { should report_lint line: 1 }
+  end
+
+  context 'when function is a transform function' do
+    %w[rotateX rotateY rotateZ
+       scaleX scaleY scaleZ
+       skewX skewY
+       translateX translateY translateZ].each do |function|
+      let(:css) { <<-CSS }
+        p {
+          transform: #{function}(2);
+        }
+      CSS
+
+      it { should_not report_lint }
+    end
+  end
+
+  context 'when mixin is a transform function' do
+    let(:css) { <<-CSS }
+      p {
+        @include translateX(2);
+      }
+    CSS
+
+    it { should_not report_lint }
+  end
+
+  context 'when a mixin contains keyword arguments with underscores' do
+    let(:css) { '@include mixin($some_var: 4);' }
+
+    it { should report_lint }
+  end
+
+  context 'when a mixin contains keyword arguments with hyphens' do
+    let(:css) { '@include mixin($some-var: 4);' }
+
+    it { should_not report_lint }
+  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) { '@extend %block;' }
+
+      it { should_not report_lint }
+    end
+
+    context 'when a name contains a single underscore' do
+      let(:css) { '@extend %block_thing;' }
+
+      it { should report_lint }
+    end
+
+    context 'when a name contains a single hyphen' do
+      let(:css) { '@extend %block-thing;' }
+
+      it { should_not report_lint }
+    end
+
+    context 'when a name contains a double hyphen' do
+      let(:css) { '@extend %block--thing;' }
+
+      it { should_not report_lint }
+    end
+
+    context 'when a name contains a double underscore' do
+      let(:css) { '@extend %block__thing;' }
+
+      it { should_not report_lint }
+    end
+
+    context 'when a name contains a double underscore and double hyphen' do
+      let(:css) { '@extend %block--thing__word;' }
+
+      it { should_not report_lint }
+    end
+  end
+end