Style-dictionary: Add unit tests for es6 format

Created on 2 Oct 2018  路  5Comments  路  Source: amzn/style-dictionary

We are working to have better testing coverage and migrating our tests to Jest from Mocha. With some work, we should be able to test the es6 format, which we have been unable to do because Mocha does not understand es6.

https://github.com/facebook/jest#additional-configuration

Note: Work off of the 'jest' branch. That is our new branch to work off of for the testing upgrades. When everything is ready we will merge that branch back into develop and mainline.

hacktoberfest help wanted

All 5 comments

I've been writing a lot of tests using Jest recently, so I'd like to hop on board here. I'm assuming you want the other issues completed, or are you just looking for a complete overhaul of every single test?

Sorry @DarthOstrich looks like @davixyz beat you to the punch. One major thing we are doing right now is migrating our tests from Mocha to Jest, which is mostly done, but in the process we want to go through our tests, make sure they are all functioning properly, add tests that should be there, etc. I made a project to track this: https://github.com/amzn/style-dictionary/projects/4

If you want to hop in to our new Jest tests and see if there is anything that could be improved, that would be super helpful. For this project we have a 'jest' branch on the repository we are working off of. Right now on the branch, the previous tests have been migrated, but we are still looking to see where our tests could improve, either by adding more tests where there aren't any yet, or general housekeeping in the testing. We are also keen to add code coverage with Istanbul/NYC as well. If any of that interests you, take a look at the 'jest' branch and let me know what you think. If you have any questions or anything let me know!

@DarthOstrich see the conversation here: https://github.com/amzn/style-dictionary/pull/133
there are a few important points that are still open for discussions, and feedback/ideas/contributions are more than welcome :)

I think this could be closed with this:
https://github.com/amzn/style-dictionary/pull/149

but if we need more testing for this format, definitely leave it open

Yup I'm going to close this

Was this page helpful?
0 / 5 - 0 ratings

Related issues

chazzmoney picture chazzmoney  路  4Comments

tonyjwalt picture tonyjwalt  路  5Comments

matt-tyas picture matt-tyas  路  3Comments

tlouisse picture tlouisse  路  4Comments

valiksb picture valiksb  路  4Comments