Merge pull request #679 from mgreter/test/import-errors

mgreter · mgreter · commit 90a3f946a39b · 2016-01-11T05:10:26.000+01:00
Add error spec tests for wrong imports
diff --git a/spec/libsass-todo-tests/errors/import/control-else/error b/spec/libsass-todo-tests/errors/import/control-else/error
@@ -0,0 +1,3 @@
+Error: Import directives may not be used within control directives or mixins.
+        on line 3 of /sass/sass-spec/spec/errors/import/control-else/input.scss
+  Use --trace for backtrace.
diff --git a/spec/libsass-todo-tests/errors/import/control-else/expected_output.css b/spec/libsass-todo-tests/errors/import/control-else/expected_output.css
diff --git a/spec/libsass-todo-tests/errors/import/control-else/input.scss b/spec/libsass-todo-tests/errors/import/control-else/input.scss
@@ -0,0 +1,4 @@
+@if (false) {
+} @else {
+  @import '_option1';
+}
diff --git a/spec/libsass-todo-tests/errors/import/control-else/status b/spec/libsass-todo-tests/errors/import/control-else/status
@@ -0,0 +1 @@
+65
diff --git a/spec/libsass-todo-tests/errors/import/control-if/error b/spec/libsass-todo-tests/errors/import/control-if/error
@@ -0,0 +1,3 @@
+Error: Import directives may not be used within control directives or mixins.
+        on line 2 of /sass/sass-spec/spec/errors/import/control-if/input.scss
+  Use --trace for backtrace.
diff --git a/spec/libsass-todo-tests/errors/import/control-if/expected_output.css b/spec/libsass-todo-tests/errors/import/control-if/expected_output.css
diff --git a/spec/libsass-todo-tests/errors/import/control-if/input.scss b/spec/libsass-todo-tests/errors/import/control-if/input.scss
@@ -0,0 +1,3 @@
+@if (true) {
+  @import '_option1';
+}
diff --git a/spec/libsass-todo-tests/errors/import/control-if/status b/spec/libsass-todo-tests/errors/import/control-if/status
@@ -0,0 +1 @@
+65
diff --git a/spec/libsass-todo-tests/errors/import/loop/each/error b/spec/libsass-todo-tests/errors/import/loop/each/error
@@ -0,0 +1,3 @@
+Error: Import directives may not be used within control directives or mixins.
+        on line 2 of /sass/sass-spec/spec/errors/import/loop/each/input.scss
+  Use --trace for backtrace.
diff --git a/spec/libsass-todo-tests/errors/import/loop/each/expected_output.css b/spec/libsass-todo-tests/errors/import/loop/each/expected_output.css
diff --git a/spec/libsass-todo-tests/errors/import/loop/each/input.scss b/spec/libsass-todo-tests/errors/import/loop/each/input.scss
@@ -0,0 +1,3 @@
+@each $i in (1) {
+  @import '_option1';
+}
diff --git a/spec/libsass-todo-tests/errors/import/loop/each/status b/spec/libsass-todo-tests/errors/import/loop/each/status
@@ -0,0 +1 @@
+65
diff --git a/spec/libsass-todo-tests/errors/import/loop/for/error b/spec/libsass-todo-tests/errors/import/loop/for/error
@@ -0,0 +1,3 @@
+Error: Import directives may not be used within control directives or mixins.
+        on line 2 of /sass/sass-spec/spec/errors/import/loop/for/input.scss
+  Use --trace for backtrace.
diff --git a/spec/libsass-todo-tests/errors/import/loop/for/expected_output.css b/spec/libsass-todo-tests/errors/import/loop/for/expected_output.css
diff --git a/spec/libsass-todo-tests/errors/import/loop/for/input.scss b/spec/libsass-todo-tests/errors/import/loop/for/input.scss
@@ -0,0 +1,3 @@
+@for $i from 1 through 2 {
+  @import '_option1';
+}
diff --git a/spec/libsass-todo-tests/errors/import/loop/for/status b/spec/libsass-todo-tests/errors/import/loop/for/status
@@ -0,0 +1 @@
+65
diff --git a/spec/libsass-todo-tests/errors/import/loop/while/error b/spec/libsass-todo-tests/errors/import/loop/while/error
@@ -0,0 +1,3 @@
+Error: Import directives may not be used within control directives or mixins.
+        on line 3 of /sass/sass-spec/spec/errors/import/loop/while/input.scss
+  Use --trace for backtrace.
diff --git a/spec/libsass-todo-tests/errors/import/loop/while/expected_output.css b/spec/libsass-todo-tests/errors/import/loop/while/expected_output.css
diff --git a/spec/libsass-todo-tests/errors/import/loop/while/input.scss b/spec/libsass-todo-tests/errors/import/loop/while/input.scss
@@ -0,0 +1,5 @@
+$count: 0;
+@while ($count < 1) {
+  @import '_option1';
+  $count: $count + 1;
+}
diff --git a/spec/libsass-todo-tests/errors/import/loop/while/status b/spec/libsass-todo-tests/errors/import/loop/while/status
@@ -0,0 +1 @@
+65
diff --git a/spec/libsass-todo-tests/errors/import/mixin/control-else/inside/error b/spec/libsass-todo-tests/errors/import/mixin/control-else/inside/error
@@ -0,0 +1,3 @@
+Error: Import directives may not be used within control directives or mixins.
+        on line 4 of /sass/sass-spec/spec/errors/import/mixin/control-else/inside/input.scss
+  Use --trace for backtrace.
diff --git a/spec/libsass-todo-tests/errors/import/mixin/control-else/inside/expected_output.css b/spec/libsass-todo-tests/errors/import/mixin/control-else/inside/expected_output.css
diff --git a/spec/libsass-todo-tests/errors/import/mixin/control-else/inside/input.scss b/spec/libsass-todo-tests/errors/import/mixin/control-else/inside/input.scss
@@ -0,0 +1,10 @@
+@mixin do_import() {
+  @if (false) {
+  } @else {
+    @import '_option1';
+  }
+}
+
+foo {
+  @include do_import();
+}
diff --git a/spec/libsass-todo-tests/errors/import/mixin/control-else/inside/status b/spec/libsass-todo-tests/errors/import/mixin/control-else/inside/status
@@ -0,0 +1 @@
+65
diff --git a/spec/libsass-todo-tests/errors/import/mixin/control-else/outside/error b/spec/libsass-todo-tests/errors/import/mixin/control-else/outside/error
@@ -0,0 +1,3 @@
+Error: Import directives may not be used within control directives or mixins.
+        on line 2 of /sass/sass-spec/spec/errors/import/mixin/control-else/outside/input.scss
+  Use --trace for backtrace.
diff --git a/spec/libsass-todo-tests/errors/import/mixin/control-else/outside/expected_output.css b/spec/libsass-todo-tests/errors/import/mixin/control-else/outside/expected_output.css
diff --git a/spec/libsass-todo-tests/errors/import/mixin/control-else/outside/input.scss b/spec/libsass-todo-tests/errors/import/mixin/control-else/outside/input.scss
@@ -0,0 +1,8 @@
+@mixin do_import() {
+  @import '_option1';
+}
+
+@if (false) {
+} @else {
+  @include do_import();
+}
diff --git a/spec/libsass-todo-tests/errors/import/mixin/control-else/outside/status b/spec/libsass-todo-tests/errors/import/mixin/control-else/outside/status
@@ -0,0 +1 @@
+65
diff --git a/spec/libsass-todo-tests/errors/import/mixin/control-if/inside/error b/spec/libsass-todo-tests/errors/import/mixin/control-if/inside/error
@@ -0,0 +1,3 @@
+Error: Import directives may not be used within control directives or mixins.
+        on line 3 of /sass/sass-spec/spec/errors/import/mixin/control-if/inside/input.scss
+  Use --trace for backtrace.
diff --git a/spec/libsass-todo-tests/errors/import/mixin/control-if/inside/expected_output.css b/spec/libsass-todo-tests/errors/import/mixin/control-if/inside/expected_output.css
diff --git a/spec/libsass-todo-tests/errors/import/mixin/control-if/inside/input.scss b/spec/libsass-todo-tests/errors/import/mixin/control-if/inside/input.scss
@@ -0,0 +1,9 @@
+@mixin do_import() {
+  @if (true) {
+    @import '_option1';
+  }
+}
+
+foo {
+  @include do_import();
+}
diff --git a/spec/libsass-todo-tests/errors/import/mixin/control-if/inside/status b/spec/libsass-todo-tests/errors/import/mixin/control-if/inside/status
@@ -0,0 +1 @@
+65
diff --git a/spec/libsass-todo-tests/errors/import/mixin/control-if/outside/error b/spec/libsass-todo-tests/errors/import/mixin/control-if/outside/error
@@ -0,0 +1,3 @@
+Error: Import directives may not be used within control directives or mixins.
+        on line 2 of /sass/sass-spec/spec/errors/import/mixin/control-if/outside/input.scss
+  Use --trace for backtrace.
diff --git a/spec/libsass-todo-tests/errors/import/mixin/control-if/outside/expected_output.css b/spec/libsass-todo-tests/errors/import/mixin/control-if/outside/expected_output.css
diff --git a/spec/libsass-todo-tests/errors/import/mixin/control-if/outside/input.scss b/spec/libsass-todo-tests/errors/import/mixin/control-if/outside/input.scss
@@ -0,0 +1,7 @@
+@mixin do_import() {
+  @import '_option1';
+}
+
+@if (true) {
+  @include do_import();
+}
diff --git a/spec/libsass-todo-tests/errors/import/mixin/control-if/outside/status b/spec/libsass-todo-tests/errors/import/mixin/control-if/outside/status
@@ -0,0 +1 @@
+65
diff --git a/spec/libsass-todo-tests/errors/import/mixin/simple/inside/error b/spec/libsass-todo-tests/errors/import/mixin/simple/inside/error
@@ -0,0 +1,4 @@
+Error: Undefined variable: "$family".
+        on line 2 of /sass/sass-spec/spec/errors/import/mixin/simple/inside/input.scss, in `import-google-fonts'
+        from line 6 of /sass/sass-spec/spec/errors/import/mixin/simple/inside/input.scss
+  Use --trace for backtrace.
diff --git a/spec/libsass-todo-tests/errors/import/mixin/simple/inside/expected_output.css b/spec/libsass-todo-tests/errors/import/mixin/simple/inside/expected_output.css
diff --git a/spec/libsass-todo-tests/errors/import/mixin/simple/inside/input.scss b/spec/libsass-todo-tests/errors/import/mixin/simple/inside/input.scss
@@ -0,0 +1,7 @@
+@mixin import-google-fonts() {
+  @import url("http://fonts.googleapis.com/css?family=#{$family}");
+}
+foo {
+  $family: unquote("Droid+Sans");
+  @include import-google-fonts();
+}
diff --git a/spec/libsass-todo-tests/errors/import/mixin/simple/inside/status b/spec/libsass-todo-tests/errors/import/mixin/simple/inside/status
@@ -0,0 +1 @@
+65
diff --git a/spec/libsass-todo-tests/errors/import/mixin/simple/outside/expected.compact.css b/spec/libsass-todo-tests/errors/import/mixin/simple/outside/expected.compact.css
@@ -0,0 +1 @@
+@import url("http://fonts.googleapis.com/css?family=Droid+Sans");
diff --git a/spec/libsass-todo-tests/errors/import/mixin/simple/outside/expected.compressed.css b/spec/libsass-todo-tests/errors/import/mixin/simple/outside/expected.compressed.css
@@ -0,0 +1 @@
+@import url("http://fonts.googleapis.com/css?family=Droid+Sans");
diff --git a/spec/libsass-todo-tests/errors/import/mixin/simple/outside/expected.expanded.css b/spec/libsass-todo-tests/errors/import/mixin/simple/outside/expected.expanded.css
@@ -0,0 +1 @@
+@import url("http://fonts.googleapis.com/css?family=Droid+Sans");
diff --git a/spec/libsass-todo-tests/errors/import/mixin/simple/outside/expected_output.css b/spec/libsass-todo-tests/errors/import/mixin/simple/outside/expected_output.css
@@ -0,0 +1 @@
+@import url("http://fonts.googleapis.com/css?family=Droid+Sans");
diff --git a/spec/libsass-todo-tests/errors/import/mixin/simple/outside/input.scss b/spec/libsass-todo-tests/errors/import/mixin/simple/outside/input.scss
@@ -0,0 +1,5 @@
+@mixin import-google-fonts() {
+  @import url("http://fonts.googleapis.com/css?family=#{$family}");
+}
+$family: unquote("Droid+Sans");
+@include import-google-fonts();
diff --git a/spec/libsass-todo-tests/errors/import/simple/expected.compact.css b/spec/libsass-todo-tests/errors/import/simple/expected.compact.css
@@ -0,0 +1,6 @@
+@import url(hey1.css);
+@import url(cookie.css);
+@import url("hey2.css");
+@import url(fudge.css);
+@import url(bludge.css);
+div[name="hello"] { color: blue; }
diff --git a/spec/libsass-todo-tests/errors/import/simple/expected.compressed.css b/spec/libsass-todo-tests/errors/import/simple/expected.compressed.css
@@ -0,0 +1 @@
+@import url(hey1.css);@import url(cookie.css);@import url("hey2.css");@import url(fudge.css);@import url(bludge.css);div[name="hello"]{color:blue}
diff --git a/spec/libsass-todo-tests/errors/import/simple/expected.expanded.css b/spec/libsass-todo-tests/errors/import/simple/expected.expanded.css
@@ -0,0 +1,8 @@
+@import url(hey1.css);
+@import url(cookie.css);
+@import url("hey2.css");
+@import url(fudge.css);
+@import url(bludge.css);
+div[name="hello"] {
+  color: blue;
+}
diff --git a/spec/libsass-todo-tests/errors/import/simple/expected_output.css b/spec/libsass-todo-tests/errors/import/simple/expected_output.css
@@ -0,0 +1,7 @@
+@import url(hey1.css);
+@import url(cookie.css);
+@import url("hey2.css");
+@import url(fudge.css);
+@import url(bludge.css);
+div[name="hello"] {
+  color: blue; }
diff --git a/spec/libsass-todo-tests/errors/import/simple/input.scss b/spec/libsass-todo-tests/errors/import/simple/input.scss
@@ -0,0 +1,8 @@
+@import "hey1.css", "cookie.css", url("hey2.css"), "fudge.css";
+
+$foo:"goodbye";
+div[name="hello"] {
+  color: blue;
+}
+
+@import "bludge.css";

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,3 @@`
	`1`	`+Error: Import directives may not be used within control directives or mixins.`
	`2`	`+ on line 3 of /sass/sass-spec/spec/errors/import/control-else/input.scss`
	`3`	`+ Use --trace for backtrace.`
Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,3 @@`
	`1`	`+Error: Import directives may not be used within control directives or mixins.`
	`2`	`+ on line 2 of /sass/sass-spec/spec/errors/import/control-if/input.scss`
	`3`	`+ Use --trace for backtrace.`
Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,3 @@`
	`1`	`+@if (true) {`
	`2`	`+ @import '_option1';`
	`3`	`+}`