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

Add tests #7

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Add tests #7

wants to merge 2 commits into from

Conversation

abimaelmartell
Copy link
Contributor

i added some tests and a dummy app to test against
tests can be executed with rake

for now all tests are green

@cmager
Copy link
Collaborator

cmager commented Jun 11, 2013

Hello thank you for adding tests to this gem!
I'm sorry i wasn't able to review it yet but i'll give you feedback by tomorrow.

Cheers, Chris

@cmager
Copy link
Collaborator

cmager commented Jun 13, 2013

Hi, well all assert_match tests in "stylesheet contain asset pipeline references to images" fail for me. They only pass if i remove "/asset" from the pass names. Are they passing for you?
Cheers.

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

Successfully merging this pull request may close these issues.

2 participants