Skip to content

Commit

Permalink
Fix test for Liquid rendering in Sass.
Browse files Browse the repository at this point in the history
  • Loading branch information
parkr committed Aug 30, 2014
1 parent 5330980 commit ebcb49e
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 3 deletions.
7 changes: 4 additions & 3 deletions features/rendering.feature
Expand Up @@ -21,11 +21,12 @@ Feature: Rendering
And I should not see "Ahoy, indeed!" in "_site/index.css"
And I should not see "Ahoy, indeed!" in "_site/index.js"

Scenario: Don't render liquid in Sass
Scenario: Render liquid in Sass
Given I have an "index.scss" page that contains ".foo-bar { color:{{site.color}}; }"
And I have a configuration file with "color" set to "red"
When I run jekyll build
Then the _site directory should not exist
And I should see "Invalid CSS after" in the build output
Then the _site directory should exist
And I should see ".foo-bar {\n color:red; }" in "_site/index.css"

Scenario: Don't render liquid in CoffeeScript
Given I have an "index.coffee" page that contains "hey='for {{site.animal}}'"
Expand Down
1 change: 1 addition & 0 deletions lib/jekyll/convertible.rb
Expand Up @@ -67,6 +67,7 @@ def transform
converter.convert output
rescue => e
Jekyll.logger.error "Conversion error:", "#{converter.class} encountered an error converting '#{path}'."
Jekyll.logger.error("Conversion error:", e.to_s)
raise e
end
end
Expand Down

0 comments on commit ebcb49e

Please sign in to comment.