-
Notifications
You must be signed in to change notification settings - Fork 2
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
Create tests #8
Comments
@pentzzsolt did you have a particular testing method in mind? oddbird/true ? |
I actually started implementing test cases using Hugo Giraudel's Sassy Tester function. My problem is that I haven't had the time to actually figure out what I want this to do, so I just left it unfinished for now. Hopefully in this new year I'll have the time to spend on this. |
Oh ok, because I wrote some tests for the generic case, as well as for my merging lists feature, using oddbird/true. But if you'd prefer SassyTester I could port my test cases over to that and make a PR them? The unit tests helped me reveal some odd behaviour with my initial implementation of merging-lists (lists in SASS are weird) so I've also patched the 'join-lists' behaviour in my fork too. |
Actually it looks like HugoGiraudel/SassyTester is unmaintained, so oddbird/true might be the better bet anyway. You can have a look at the tests I wrote here if you want to check prior to a PR: https://github.com/howardcox/sass-recursive-map-merge/blob/recursive-join-lists-%234/test/test.scss |
That's a lot of work! 😲 |
No description provided.
The text was updated successfully, but these errors were encountered: