Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added spec for libsass issue 713 #167

Closed
wants to merge 2 commits into from
Closed

Added spec for libsass issue 713 #167

wants to merge 2 commits into from

Conversation

KittyGiraudel
Copy link
Contributor

I am not sure what I should write in expected_output.css since the expected result is a Sass error. I'll edit as needed.

@KittyGiraudel KittyGiraudel changed the title Added spec test for libsass issue 713 Added spec for libsass issue 713 Dec 11, 2014
@xzyfer
Copy link
Contributor

xzyfer commented Dec 11, 2014

This is blocked on #136.

@KittyGiraudel
Copy link
Contributor Author

Alright.

@xzyfer
Copy link
Contributor

xzyfer commented Dec 11, 2014

@hugogiraudel since each of the three function results in an error can you please create a tests for each with the appropriate error message in the expected_output.scss

spec/libsass-todo-issues/issue_713/
|- and
  |- input.scss
  |- expected_output.scss
|- or
  |- input.scss
  |- expected_output.scss
|- not
  |- input.scss
  |- expected_output.scss

@KittyGiraudel
Copy link
Contributor Author

Appropriate error message according to the Ruby compiler?

@KittyGiraudel
Copy link
Contributor Author

Done.

@xzyfer
Copy link
Contributor

xzyfer commented Dec 11, 2014

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants