Financial-Times / o-grid

Responsive grid system
http://registry.origami.ft.com/components/o-grid
94 stars 17 forks source link

True test #163

Closed gvonkoss closed 6 years ago

gvonkoss commented 6 years ago

This PR is incorporating True, and removing the Ruby framework that has been built up for the tests. It's testing o-grids public functions and mixins, quite literally, but is ignoring the private ones (integration...?).

I didn't test oGridGenerate because that really generates every conceivable variation of helper classes under the sun, and it might be a bit much? But maybe not because it's important that those are right.

Thoughts and opinions welcomed and encouraged. (I think I broke Karma).

There are a few mixins in here which may no longer be relevant (oGridTargetIE8?) And there are a few other things that have crept up while I've been testing. @JakeChampion and I have also spoken about this to some extent, but I will open an issue with specifics.