sass 3.4.0 → 3.4.25

data/test/sass/script_test.rb

@@ -31,6 +31,11 @@ class SassScriptTest < MiniTest::Test
     assert_equal 0, Sass::Script::Value::Color.new([1, 2, 3, -0.1]).alpha
   end
 
+  def test_color_from_hex
+    assert_equal Sass::Script::Value::Color.new([0,0,0]), Sass::Script::Value::Color.from_hex('000000')
+    assert_equal Sass::Script::Value::Color.new([0,0,0]), Sass::Script::Value::Color.from_hex('#000000')
+  end
+
   def test_string_escapes
     assert_equal "'", resolve("\"'\"")
     assert_equal '"', resolve("\"\\\"\"")
@@ -39,6 +44,7 @@ class SassScriptTest < MiniTest::Test
     assert_equal "☃f", resolve("\"\\2603 f\"")
     assert_equal "☃x", resolve("\"\\2603x\"")
     assert_equal "\\2603", resolve("\"\\\\2603\"")
+    assert_equal "\#{foo}", resolve("\"\\\#{foo}\"")
 
     # U+FFFD is the replacement character, "�".
     assert_equal [0xFFFD].pack("U"), resolve("\"\\0\"")
@@ -66,12 +72,13 @@ class SassScriptTest < MiniTest::Test
     assert_equal '"☃abcdef"', resolve_quoted('"\2603 abcdef"')
     assert_equal '"\\\\"', resolve_quoted('"\\\\"')
     assert_equal '"foobar"', resolve_quoted("\"foo\\\nbar\"")
+    assert_equal '"#{foo}"', resolve_quoted("\"\\\#{foo}\"")
   end
 
   def test_color_names
     assert_equal "white", resolve("white")
     assert_equal "#ffffff", resolve("#ffffff")
-    assert_equal "#fffffe", resolve("white - #000001")
+    silence_warnings {assert_equal "#fffffe", resolve("white - #000001")}
     assert_equal "transparent", resolve("transparent")
     assert_equal "transparent", resolve("rgba(0, 0, 0, 0)")
   end
@@ -90,29 +97,56 @@ class SassScriptTest < MiniTest::Test
   end
 
   def test_rgba_color_math
-    assert_equal "rgba(50, 50, 100, 0.35)", resolve("rgba(1, 1, 2, 0.35) * rgba(50, 50, 50, 0.35)")
-    assert_equal "rgba(52, 52, 52, 0.25)", resolve("rgba(2, 2, 2, 0.25) + rgba(50, 50, 50, 0.25)")
+    silence_warnings {assert_equal "rgba(50, 50, 100, 0.35)", resolve("rgba(1, 1, 2, 0.35) * rgba(50, 50, 50, 0.35)")}
+    silence_warnings {assert_equal "rgba(52, 52, 52, 0.25)", resolve("rgba(2, 2, 2, 0.25) + rgba(50, 50, 50, 0.25)")}
 
     assert_raise_message(Sass::SyntaxError, "Alpha channels must be equal: rgba(1, 2, 3, 0.15) + rgba(50, 50, 50, 0.75)") do
-      resolve("rgba(1, 2, 3, 0.15) + rgba(50, 50, 50, 0.75)")
+      silence_warnings {resolve("rgba(1, 2, 3, 0.15) + rgba(50, 50, 50, 0.75)")}
     end
     assert_raise_message(Sass::SyntaxError, "Alpha channels must be equal: #123456 * rgba(50, 50, 50, 0.75)") do
-      resolve("#123456 * rgba(50, 50, 50, 0.75)")
+      silence_warnings {resolve("#123456 * rgba(50, 50, 50, 0.75)")}
     end
     assert_raise_message(Sass::SyntaxError, "Alpha channels must be equal: rgba(50, 50, 50, 0.75) / #123456") do
-      resolve("rgba(50, 50, 50, 0.75) / #123456")
+      silence_warnings {resolve("rgba(50, 50, 50, 0.75) / #123456")}
     end
   end
 
   def test_rgba_number_math
-    assert_equal "rgba(49, 49, 49, 0.75)", resolve("rgba(50, 50, 50, 0.75) - 1")
-    assert_equal "rgba(100, 100, 100, 0.75)", resolve("rgba(50, 50, 50, 0.75) * 2")
+    silence_warnings {assert_equal "rgba(49, 49, 49, 0.75)", resolve("rgba(50, 50, 50, 0.75) - 1")}
+    silence_warnings {assert_equal "rgba(100, 100, 100, 0.75)", resolve("rgba(50, 50, 50, 0.75) * 2")}
   end
 
   def test_rgba_rounding
     assert_equal "rgba(10, 1, 0, 0.12346)", resolve("rgba(10.0, 1.23456789, 0.0, 0.1234567)")
   end
 
+  def test_rgb_calc
+    assert_equal "rgb(calc(255 - 5), 0, 0)", resolve("rgb(calc(255 - 5), 0, 0)")
+  end
+
+  def test_rgba_calc
+    assert_equal "rgba(calc(255 - 5), 0, 0, 0.1)",
+      resolve("rgba(calc(255 - 5), 0, 0, 0.1)")
+    assert_equal "rgba(127, 0, 0, calc(0.1 + 0.5))",
+      resolve("rgba(127, 0, 0, calc(0.1 + 0.5))")
+  end
+
+  def test_rgba_shorthand_calc
+    assert_equal "rgba(255, 0, 0, calc(0.1 + 0.5))",
+      resolve("rgba(red, calc(0.1 + 0.5))")
+  end
+
+  def test_hsl_calc
+    assert_equal "hsl(calc(360 * 5 / 6), 50%, 50%)", resolve("hsl(calc(360 * 5 / 6), 50%, 50%)")
+  end
+
+  def test_hsla_calc
+    assert_equal "hsla(calc(360 * 5 / 6), 50%, 50%, 0.1)",
+      resolve("hsla(calc(360 * 5 / 6), 50%, 50%, 0.1)")
+    assert_equal "hsla(270, 50%, 50%, calc(0.1 + 0.1))",
+      resolve("hsla(270, 50%, 50%, calc(0.1 + 0.1))")
+  end
+
   def test_compressed_colors
     assert_equal "#123456", resolve("#123456", :style => :compressed)
     assert_equal "rgba(1,2,3,0.5)", resolve("rgba(1, 2, 3, 0.5)", :style => :compressed)
@@ -157,43 +191,45 @@ class SassScriptTest < MiniTest::Test
   end
 
   def test_interpolation_near_operators
-    assert_equal '3 , 7', resolve('#{1 + 2} , #{3 + 4}')
-    assert_equal '3, 7', resolve('#{1 + 2}, #{3 + 4}')
-    assert_equal '3 ,7', resolve('#{1 + 2} ,#{3 + 4}')
-    assert_equal '3,7', resolve('#{1 + 2},#{3 + 4}')
-    assert_equal '3, 7, 11', resolve('#{1 + 2}, #{3 + 4}, #{5 + 6}')
-    assert_equal '3, 7, 11', resolve('3, #{3 + 4}, 11')
-    assert_equal '3, 7, 11', resolve('3, 7, #{5 + 6}')
-
-    assert_equal '3 / 7', resolve('3 / #{3 + 4}')
-    assert_equal '3 /7', resolve('3 /#{3 + 4}')
-    assert_equal '3/ 7', resolve('3/ #{3 + 4}')
-    assert_equal '3/7', resolve('3/#{3 + 4}')
-
-    assert_equal '3 * 7', resolve('#{1 + 2} * 7')
-    assert_equal '3* 7', resolve('#{1 + 2}* 7')
-    assert_equal '3 *7', resolve('#{1 + 2} *7')
-    assert_equal '3*7', resolve('#{1 + 2}*7')
-
-    assert_equal '-3', resolve('-#{1 + 2}')
-    assert_equal '- 3', resolve('- #{1 + 2}')
-
-    assert_equal '5 + 3 * 7', resolve('5 + #{1 + 2} * #{3 + 4}')
-    assert_equal '5 +3 * 7', resolve('5 +#{1 + 2} * #{3 + 4}')
-    assert_equal '5+3 * 7', resolve('5+#{1 + 2} * #{3 + 4}')
-    assert_equal '3 * 7 + 5', resolve('#{1 + 2} * #{3 + 4} + 5')
-    assert_equal '3 * 7+ 5', resolve('#{1 + 2} * #{3 + 4}+ 5')
-    assert_equal '3 * 7+5', resolve('#{1 + 2} * #{3 + 4}+5')
-
-    assert_equal '5/3 + 7', resolve('5 / (#{1 + 2} + #{3 + 4})')
-    assert_equal '5/3 + 7', resolve('5 /(#{1 + 2} + #{3 + 4})')
-    assert_equal '5/3 + 7', resolve('5 /( #{1 + 2} + #{3 + 4} )')
-    assert_equal '3 + 7/5', resolve('(#{1 + 2} + #{3 + 4}) / 5')
-    assert_equal '3 + 7/5', resolve('(#{1 + 2} + #{3 + 4})/ 5')
-    assert_equal '3 + 7/5', resolve('( #{1 + 2} + #{3 + 4} )/ 5')
-
-    assert_equal '3 + 5', resolve('#{1 + 2} + 2 + 3')
-    assert_equal '3 +5', resolve('#{1 + 2} +2 + 3')
+    silence_warnings do
+      assert_equal '3 , 7', resolve('#{1 + 2} , #{3 + 4}')
+      assert_equal '3, 7', resolve('#{1 + 2}, #{3 + 4}')
+      assert_equal '3 ,7', resolve('#{1 + 2} ,#{3 + 4}')
+      assert_equal '3,7', resolve('#{1 + 2},#{3 + 4}')
+      assert_equal '3, 7, 11', resolve('#{1 + 2}, #{3 + 4}, #{5 + 6}')
+      assert_equal '3, 7, 11', resolve('3, #{3 + 4}, 11')
+      assert_equal '3, 7, 11', resolve('3, 7, #{5 + 6}')
+
+      assert_equal '3 / 7', resolve('3 / #{3 + 4}')
+      assert_equal '3 /7', resolve('3 /#{3 + 4}')
+      assert_equal '3/ 7', resolve('3/ #{3 + 4}')
+      assert_equal '3/7', resolve('3/#{3 + 4}')
+
+      assert_equal '3 * 7', resolve('#{1 + 2} * 7')
+      assert_equal '3* 7', resolve('#{1 + 2}* 7')
+      assert_equal '3 *7', resolve('#{1 + 2} *7')
+      assert_equal '3*7', resolve('#{1 + 2}*7')
+
+      assert_equal '-3', resolve('-#{1 + 2}')
+      assert_equal '- 3', resolve('- #{1 + 2}')
+
+      assert_equal '5 + 3 * 7', resolve('5 + #{1 + 2} * #{3 + 4}')
+      assert_equal '5 +3 * 7', resolve('5 +#{1 + 2} * #{3 + 4}')
+      assert_equal '5+3 * 7', resolve('5+#{1 + 2} * #{3 + 4}')
+      assert_equal '3 * 7 + 5', resolve('#{1 + 2} * #{3 + 4} + 5')
+      assert_equal '3 * 7+ 5', resolve('#{1 + 2} * #{3 + 4}+ 5')
+      assert_equal '3 * 7+5', resolve('#{1 + 2} * #{3 + 4}+5')
+
+      assert_equal '5/3 + 7', resolve('5 / (#{1 + 2} + #{3 + 4})')
+      assert_equal '5/3 + 7', resolve('5 /(#{1 + 2} + #{3 + 4})')
+      assert_equal '5/3 + 7', resolve('5 /( #{1 + 2} + #{3 + 4} )')
+      assert_equal '3 + 7/5', resolve('(#{1 + 2} + #{3 + 4}) / 5')
+      assert_equal '3 + 7/5', resolve('(#{1 + 2} + #{3 + 4})/ 5')
+      assert_equal '3 + 7/5', resolve('( #{1 + 2} + #{3 + 4} )/ 5')
+
+      assert_equal '3 + 5', resolve('#{1 + 2} + 2 + 3')
+      assert_equal '3 +5', resolve('#{1 + 2} +2 + 3')
+    end
   end
 
   def test_string_interpolation
@@ -206,6 +242,13 @@ class SassScriptTest < MiniTest::Test
     assert_equal "foo1bar5baz4bang", resolve('\'foo#{1 + "bar#{2 + 3}baz" + 4}bang\'')
   end
 
+  def test_interpolation_in_interpolation
+    assert_equal 'foo', resolve('#{#{foo}}')
+    assert_equal 'foo', resolve('"#{#{foo}}"')
+    assert_equal 'foo', resolve('#{"#{foo}"}')
+    assert_equal 'foo', resolve('"#{"#{foo}"}"')
+  end
+
   def test_interpolation_with_newline
     assert_equal "\nbang", resolve('"#{"\a "}bang"')
     assert_equal "\n\nbang", resolve('"#{"\a "}\a bang"')
@@ -341,6 +384,17 @@ SASS
     assert_equal "-1", resolve("-1")
   end
 
+  def test_subtraction_vs_minus_vs_identifier
+    assert_equal "0.25em", resolve("1em-.75")
+    assert_equal "0.25em", resolve("1em-0.75")
+    assert_equal "1em -0.75", resolve("1em -.75")
+    assert_equal "1em -0.75", resolve("1em -0.75")
+    assert_equal "1em- 0.75", resolve("1em- .75")
+    assert_equal "1em- 0.75", resolve("1em- 0.75")
+    assert_equal "0.25em", resolve("1em - .75")
+    assert_equal "0.25em", resolve("1em - 0.75")
+  end
+
   def test_string_ops
     assert_equal '"foo" "bar"', resolve('"foo" "bar"')
     assert_equal "true 1", resolve('true 1')
@@ -450,16 +504,33 @@ SASS
 
   def test_functions
     assert_equal "#80ff80", resolve("hsl(120, 100%, 75%)")
-    assert_equal "#81ff81", resolve("hsl(120, 100%, 75%) + #010001")
+    silence_warnings {assert_equal "#81ff81", resolve("hsl(120, 100%, 75%) + #010001")}
   end
 
   def test_operator_unit_conversion
     assert_equal "1.1cm", resolve("1cm + 1mm")
+    assert_equal "8q", resolve("4q + 1mm")
+    assert_equal "40.025cm", resolve("40cm + 1q")
     assert_equal "2in", resolve("1in + 96px")
     assert_equal "true", resolve("2mm < 1cm")
     assert_equal "true", resolve("10mm == 1cm")
-    assert_equal "true", resolve("1 == 1cm")
     assert_equal "true", resolve("1.1cm == 11mm")
+    assert_equal "true", resolve("2mm == 8q")
+    assert_equal "false", resolve("2px > 3q")
+
+    Sass::Deprecation.allow_double_warnings do
+      assert_warning(<<WARNING) {assert_equal "true", resolve("1 == 1cm")}
+DEPRECATION WARNING on line 1 of test_operator_unit_conversion_inline.sass:
+The result of `1 == 1cm` will be `false` in future releases of Sass.
+Unitless numbers will no longer be equal to the same numbers with units.
+WARNING
+
+      assert_warning(<<WARNING) {assert_equal "false", resolve("1 != 1cm")}
+DEPRECATION WARNING on line 1 of test_operator_unit_conversion_inline.sass:
+The result of `1 != 1cm` will be `true` in future releases of Sass.
+Unitless numbers will no longer be equal to the same numbers with units.
+WARNING
+    end
   end
 
   def test_length_units
@@ -469,6 +540,8 @@ SASS
     assert_equal "2.83465", resolve("(1mm/1pt)")
     assert_equal "1.33333", resolve("(1pt/1px)")
     assert_equal "0.01042", resolve("(1px/1in)")
+    assert_equal "1.05833", resolve("(1px/1q)")
+    assert_equal "0.05906", resolve("(1q/1pc)")
   end
 
   def test_angle_units
@@ -487,9 +560,9 @@ SASS
   end
 
   def test_resolution_units
-    assert_equal "2.54", resolve("(1dpi/1dpcm)")
-    assert_equal "37.79528", resolve("(1dpcm/1dppx)")
-    assert_equal "0.01042", resolve("(1dppx/1dpi)")
+    assert_equal "0.3937", resolve("(1dpi/1dpcm)")
+    assert_equal "0.02646", resolve("(1dpcm/1dppx)")
+    assert_equal "96", resolve("(1dppx/1dpi)")
   end
 
   def test_operations_have_options
@@ -524,6 +597,13 @@ SASS
     assert_equal "0.5", resolve("$var", {}, env("var" => eval("1px/2px")))
   end
 
+  # Regression test for issue 1786.
+  def test_slash_division_within_list
+    assert_equal "1 1/2 1/2", resolve("(1 1/2 1/2)")
+    assert_equal "1/2 1/2", resolve("(1/2 1/2)")
+    assert_equal "1/2", resolve("(1/2,)")
+  end
+
   def test_non_ident_colors_with_wrong_number_of_digits
     assert_raise_message(Sass::SyntaxError,
       'Invalid CSS after "": expected expression (e.g. 1px, bold), was "#1"') {eval("#1")}
@@ -540,7 +620,7 @@ SASS
   def test_case_insensitive_color_names
     assert_equal "BLUE", resolve("BLUE")
     assert_equal "rEd", resolve("rEd")
-    assert_equal "#7f4000", resolve("mix(GrEeN, ReD)")
+    assert_equal "#804000", resolve("mix(GrEeN, ReD)")
   end
 
   def test_empty_list
@@ -781,6 +861,112 @@ CSS
 SCSS
   end
 
+  def test_if_can_assign_to_global_variables
+    assert_equal <<CSS, render(<<SCSS, :syntax => :scss)
+.a {
+  b: 2; }
+CSS
+$var: 1;
+@if true {$var: 2}
+.a {b: $var}
+SCSS
+  end
+
+  def test_else_can_assign_to_global_variables
+    assert_equal <<CSS, render(<<SCSS, :syntax => :scss)
+.a {
+  b: 2; }
+CSS
+$var: 1;
+@if false {}
+@else {$var: 2}
+.a {b: $var}
+SCSS
+  end
+
+  def test_for_can_assign_to_global_variables
+    assert_equal <<CSS, render(<<SCSS, :syntax => :scss)
+.a {
+  b: 2; }
+CSS
+$var: 1;
+@for $i from 1 to 2 {$var: 2}
+.a {b: $var}
+SCSS
+  end
+
+  def test_each_can_assign_to_global_variables
+    assert_equal <<CSS, render(<<SCSS, :syntax => :scss)
+.a {
+  b: 2; }
+CSS
+$var: 1;
+@each $a in 1 {$var: 2}
+.a {b: $var}
+SCSS
+  end
+
+  def test_while_can_assign_to_global_variables
+    assert_equal <<CSS, render(<<SCSS, :syntax => :scss)
+.a {
+  b: 2; }
+CSS
+$var: 1;
+@while $var != 2 {$var: 2}
+.a {b: $var}
+SCSS
+  end
+
+  def test_if_doesnt_leak_local_variables
+    assert_raise_message(Sass::SyntaxError, 'Undefined variable: "$var".') do
+      render(<<SCSS, :syntax => :scss)
+@if true {$var: 1}
+.a {b: $var}
+SCSS
+    end
+  end
+
+  def test_else_doesnt_leak_local_variables
+    assert_raise_message(Sass::SyntaxError, 'Undefined variable: "$var".') do
+      render(<<SCSS, :syntax => :scss)
+@if false {}
+@else {$var: 1}
+.a {b: $var}
+SCSS
+    end
+  end
+
+  def test_for_doesnt_leak_local_variables
+    assert_raise_message(Sass::SyntaxError, 'Undefined variable: "$var".') do
+      render(<<SCSS, :syntax => :scss)
+@for $i from 1 to 2 {$var: 1}
+.a {b: $var}
+SCSS
+    end
+  end
+
+  def test_each_doesnt_leak_local_variables
+    assert_raise_message(Sass::SyntaxError, 'Undefined variable: "$var".') do
+      render(<<SCSS, :syntax => :scss)
+@each $a in 1 {$var: 1}
+.a {b: $var}
+SCSS
+    end
+  end
+
+  def test_while_doesnt_leak_local_variables
+    assert_raise_message(Sass::SyntaxError, 'Undefined variable: "$var".') do
+      render(<<SCSS, :syntax => :scss)
+$iter: true;
+@while $iter {
+  $var: 1;
+  $iter: false;
+}
+.a {b: $var}
+SCSS
+    end
+  end
+
   def test_color_format_is_preserved_by_default
     assert_equal "blue", resolve("blue")
     assert_equal "bLuE", resolve("bLuE")
@@ -804,7 +990,7 @@ SCSS
   end
 
   def test_color_format_isnt_preserved_when_modified
-    assert_equal "magenta", resolve("#f00 + #00f")
+    assert_equal "magenta", resolve("change-color(#f00, $blue: 255)")
   end
 
   def test_ids
@@ -857,6 +1043,11 @@ SCSS
 
   # Regression Tests
 
+  def test_interpolation_after_string
+    assert_equal '"foobar" 2', resolve('"foobar" #{2}')
+    assert_equal "calc(1 + 2) 3", resolve('calc(1 + 2) #{3}')
+  end
+
   def test_repeatedly_modified_color
     assert_equal(<<CSS, render(<<SASS))
 a {
@@ -912,7 +1103,7 @@ CSS
 a
   b: foo() / 2
 SASS
-end
+  end
 
   def test_funcall_has_higher_precedence_than_color_name
     assert_equal "teal(12)", resolve("teal(12)")
@@ -994,12 +1185,198 @@ SASS
     assert_equal "NaN", resolve("(0.0/0.0)")
   end
 
+  def test_equality_uses_an_epsilon
+    # At least on my machine, this calculation introduces a floating point error:
+    # 29.0 / 7 * 7
+    # => 29.000000000000004
+    assert_equal "true", resolve("29 == (29 / 7 * 7)")
+  end
+
+  def test_compressed_output_of_numbers_with_leading_zeros
+    assert_equal "1.5", resolve("1.5", :style => :compressed)
+    assert_equal ".5", resolve("0.5", :style => :compressed)
+    assert_equal "-.5", resolve("-0.5", :style => :compressed)
+    assert_equal "0.5", resolve("0.5", :style => :compact)
+  end
+
+  def test_interpolation_without_deprecation_warning
+    assert_no_warning {assert_equal "a", resolve('#{a}')}
+    assert_no_warning {assert_equal "abc", resolve('a#{b}c')}
+    assert_no_warning {assert_equal "+ a", resolve('+ #{a}')}
+    assert_no_warning {assert_equal "/ a", resolve('/ #{a}')}
+    assert_no_warning {assert_equal "1 / a", resolve('1 / #{a}')}
+    assert_no_warning {assert_equal "a / b", resolve('#{a} / #{b}')}
+    assert_no_warning {assert_equal "foo(1 = a)", resolve('foo(1 = #{a})')}
+    assert_no_warning {assert_equal "foo(a = b)", resolve('foo(#{a} = #{b})')}
+    assert_no_warning {assert_equal "-a", resolve('-#{a}')}
+    assert_no_warning {assert_equal "1-a", resolve('1-#{a}')}
+    assert_no_warning {assert_equal "a- 1", resolve('#{a}- 1')}
+    assert_no_warning {assert_equal "a-1", resolve('#{a}-1')}
+    assert_no_warning {assert_equal "a-b", resolve('#{a}-#{b}')}
+    assert_no_warning {assert_equal "a1", resolve('#{a}1')}
+    assert_no_warning {assert_equal "ab", resolve('#{a}b')}
+    assert_no_warning {assert_equal "1a", resolve('1#{a}')}
+    assert_no_warning {assert_equal "ba", resolve('b#{a}')}
+  end
+
+  def test_leading_interpolation_with_deprecation_warning
+    assert_equal "ab == 1", resolve_with_interp_warning('#{a + b} == 1')
+    assert_equal "ab != 1", resolve_with_interp_warning('#{a + b} != 1')
+    assert_equal "ab > 1", resolve_with_interp_warning('#{a + b} > 1')
+    assert_equal "ab >= 1", resolve_with_interp_warning('#{a + b} >= 1')
+    assert_equal "ab < 1", resolve_with_interp_warning('#{a + b} < 1')
+    assert_equal "ab <= 1", resolve_with_interp_warning('#{a + b} <= 1')
+    assert_equal "ab + 1", resolve_with_interp_warning('#{a + b} + 1')
+    assert_equal "ab * 1", resolve_with_interp_warning('#{a + b} * 1')
+    assert_equal "ab - 1", resolve_with_interp_warning('#{a + b} - 1')
+    assert_equal "ab % 1", resolve_with_interp_warning('#{a + b} % 1')
+    assert_equal(
+      "abvar",
+      resolve_with_interp_warning(
+        '#{a + b}$var', '"#{a + b}#{$var}"',
+        env('var' => Sass::Script::Value::String.new("var"))))
+    assert_equal(
+      "varab",
+      resolve_with_interp_warning(
+        '$var#{a + b}', '"#{$var}#{a + b}"',
+        env('var' => Sass::Script::Value::String.new("var"))))
+    assert_equal "ab1", resolve_with_interp_warning('#{a + b}(1)', '"#{a + b}1"')
+    assert_equal "1ab", resolve_with_interp_warning('(1)#{a + b}', '"1#{a + b}"')
+  end
+
+  def test_trailing_interpolation_with_deprecation_warning
+    assert_equal "not ab", resolve_with_interp_warning('not #{a + b}')
+    assert_equal "1 and ab", resolve_with_interp_warning('1 and #{a + b}')
+    assert_equal "1 or ab", resolve_with_interp_warning('1 or #{a + b}')
+    assert_equal "1 == ab", resolve_with_interp_warning('1 == #{a + b}')
+    assert_equal "1 != ab", resolve_with_interp_warning('1 != #{a + b}')
+    assert_equal "1 > ab", resolve_with_interp_warning('1 > #{a + b}')
+    assert_equal "1 >= ab", resolve_with_interp_warning('1 >= #{a + b}')
+    assert_equal "1 < ab", resolve_with_interp_warning('1 < #{a + b}')
+    assert_equal "1 <= ab", resolve_with_interp_warning('1 <= #{a + b}')
+    assert_equal "1 + ab", resolve_with_interp_warning('1 + #{a + b}')
+    assert_equal "1 * ab", resolve_with_interp_warning('1 * #{a + b}')
+    assert_equal "1 - ab", resolve_with_interp_warning('1 - #{a + b}')
+    assert_equal "1 % ab", resolve_with_interp_warning('1 % #{a + b}')
+    assert_equal "- ab", resolve_with_interp_warning('- #{a + b}')
+    assert_equal "1- ab", resolve_with_interp_warning('1- #{a + b}')
+    assert_equal "- ab 2 3", resolve_with_interp_warning('- #{a + b} 2 3', '"- #{a + b} #{2 3}"')
+  end
+
+  def test_brackteing_interpolation_with_deprecation_warning
+    assert_equal "ab == cd", resolve_with_interp_warning('#{a + b} == #{c + d}')
+    assert_equal "ab != cd", resolve_with_interp_warning('#{a + b} != #{c + d}')
+    assert_equal "ab > cd", resolve_with_interp_warning('#{a + b} > #{c + d}')
+    assert_equal "ab >= cd", resolve_with_interp_warning('#{a + b} >= #{c + d}')
+    assert_equal "ab < cd", resolve_with_interp_warning('#{a + b} < #{c + d}')
+    assert_equal "ab <= cd", resolve_with_interp_warning('#{a + b} <= #{c + d}')
+    assert_equal "ab + cd", resolve_with_interp_warning('#{a + b} + #{c + d}')
+    assert_equal "ab * cd", resolve_with_interp_warning('#{a + b} * #{c + d}')
+    assert_equal "ab - cd", resolve_with_interp_warning('#{a + b} - #{c + d}')
+    assert_equal "ab % cd", resolve_with_interp_warning('#{a + b} % #{c + d}')
+  end
+
+  def test_interp_warning_formatting
+    resolve_with_interp_warning('#{1} + 1', '"1 + 1"')
+    resolve_with_interp_warning('#{1} + "foo"', '\'1 + "foo"\'')
+    resolve_with_interp_warning('#{1} + \'foo\'', '\'1 + "foo"\'')
+
+    resolve_with_interp_warning('#{1} + "#{a + b}"', '\'1 + "#{a + b}"\'')
+    resolve_with_interp_warning('"#{a + b}" + #{1}', '\'"#{a + b}" + 1\'')
+    resolve_with_interp_warning('"#{a + b}" + #{1} + "#{c + d}"', '\'"#{a + b}" + 1 + "#{c + d}"\'')
+
+    resolve_with_interp_warning('#{1} + "\'"', '"1 + \\"\'\\""')
+    resolve_with_interp_warning('#{1} + \'"\'', '"1 + \'\\"\'"')
+    resolve_with_interp_warning('#{1} + "\'\\""', '"1 + \\"\'\\\\\\"\\""')
+  end
+
+  def test_inactive_lazy_interpolation_deprecation_warning
+    assert_equal '1, 2, 3', assert_no_warning {resolve('1, #{2}, 3')}
+    assert_equal '1, 2, 3', assert_no_warning {resolve('1, 2, #{3}')}
+    assert_equal '1,2,3', assert_no_warning {resolve('1,#{2},3')}
+    assert_equal '1 2 3', assert_no_warning {resolve('#{1} 2 3')}
+    assert_equal '1 2 3', assert_no_warning {resolve('1 #{2} 3')}
+    assert_equal '1 2 3', assert_no_warning {resolve('1 2 #{3}')}
+
+    assert_equal '+1 2 3', assert_no_warning {resolve('+#{1} 2 3')}
+    assert_equal '-1 2 3', assert_no_warning {resolve('-#{1} 2 3')}
+    assert_equal '/1 2 3', assert_no_warning {resolve('/#{1} 2 3')}
+
+    assert_equal '1, 2, 31', assert_no_warning {resolve('(1, #{2}, 3) + 1')}
+    assert_equal '11, 2, 3', assert_no_warning {resolve('1 + (1, #{2}, 3)')}
+
+    assert_equal 'a, b, c', assert_no_warning {resolve('selector-parse((a, #{b}, c))')}
+  end
+
+  def test_active_lazy_interpolation_deprecation_warning
+    Sass::Deprecation.allow_double_warnings do
+      assert_equal "1, 2, 3", resolve_with_lazy_interp_warning('quote((1, #{2}, 3))', '"1, 2, 3"')
+      assert_equal "1", resolve_with_lazy_interp_warning('length((1, #{2}, 3))', '"1, 2, 3"')
+      assert_equal "1, 2, 3", resolve_with_lazy_interp_warning('inspect((1, #{2}, 3))', '"1, 2, 3"')
+      assert_equal "string", resolve_with_lazy_interp_warning('type-of((1, #{2}, 3))', '"1, 2, 3"')
+
+      assert_equal "+1 2 3", resolve_with_lazy_interp_warning('quote((+#{1} 2 3))', '"+1 #{2 3}"')
+      assert_equal "/1 2 3", resolve_with_lazy_interp_warning('quote((/#{1} 2 3))', '"/1 #{2 3}"')
+      assert_equal "-1 2 3", resolve_with_lazy_interp_warning('quote((-#{1} 2 3))', '"-1 #{2 3}"')
+    end
+  end
+
+  def test_comparison_of_complex_units
+    # Tests for issue #1960
+    Sass::Deprecation.allow_double_warnings do
+      assert_warning(<<WARNING) do
+DEPRECATION WARNING on line 1 of test_comparison_of_complex_units_inline.sass:
+The result of `10 == 10px` will be `false` in future releases of Sass.
+Unitless numbers will no longer be equal to the same numbers with units.
+WARNING
+        assert_equal "true", resolve("10 == 2 * 5px")
+      end
+      assert_warning(<<WARNING) do
+DEPRECATION WARNING on line 1 of test_comparison_of_complex_units_inline.sass:
+The result of `10 == 10px*px` will be `false` in future releases of Sass.
+Unitless numbers will no longer be equal to the same numbers with units.
+WARNING
+        assert_equal "true", resolve("10 == 2px * 5px")
+      end
+    end
+
+    assert_equal "true", resolve("10px * 1px == 2px * 5px")
+    assert_equal "true", resolve("5px * 1px < 2px * 5px")
+  end
+
   private
 
+  def resolve_with_lazy_interp_warning(str, contents = nil, environment = env)
+    contents ||= "\"#{str}\""
+    result = assert_warning(<<WARNING) {resolve(str, {}, environment)}
+DEPRECATION WARNING on line 1 of #{filename_for_test}:
+\#{} interpolation near operators will be simplified in a future version of Sass.
+To preserve the current behavior, use quotes:
+
+  unquote(#{contents})
+WARNING
+    $_sass_deprecated_interp_warnings = nil
+    result
+  end
+
+  def resolve_with_interp_warning(str, contents = nil, environment = env)
+    contents ||= "\"#{str}\""
+    assert_warning(<<WARNING) {resolve(str, {}, environment)}
+DEPRECATION WARNING on line 1 of #{filename_for_test}:
+\#{} interpolation near operators will be simplified in a future version of Sass.
+To preserve the current behavior, use quotes:
+
+  unquote(#{contents})
+
+You can use the sass-convert command to automatically fix most cases.
+WARNING
+  end
+
   def resolve(str, opts = {}, environment = env)
     munge_filename opts
     val = eval(str, opts, environment)
     assert_kind_of Sass::Script::Value::Base, val
+    val.options = opts
     val.is_a?(Sass::Script::Value::String) ? val.value : val.to_s
   end