Allow tests to be skipped

[lloydk]

As mentioned in color-js/color.js#428 it would be nice if tests could be skipped.

This could be as simple as adding a skip property to a test or set of tests. E.g.

{
	name: "hwb()",
	skip: "hwb color space not implemented yet",
	tests: [
		{
			args: "hwb(180 20% 30%)",
			expect: '{"spaceId":"hwb","coords":[180,20,30],"alpha":1}'
		},
	]
}

The test runners would output the number of skipped tests.

[LeaVerou]

Hey @lloydk, I’ve started working on this and I had a question: how did you envision the value of skip to be used? I thought it would just print something like "1/6 skipped".

[lloydk]

Hey @lloydk, I’ve started working on this and I had a question: how did you envision the value of skip to be used? I thought it would just print something like "1/6 skipped".

That's exactly what I was thinking.

[LeaVerou]

Ok, I pushed an experimental implementation, would love it if you could try it and give me feedback! (hasn't published to npm yet though)

[lloydk]

It would be nice if the skip property was inherited so I could easily skip a group of tests. Or be able to set it at the top level of a test file and then set a specific test to skip: false to have only a single test run (useful for debugging).

I would use yellow instead of dim for the output but that's just my personal preference.

[LeaVerou]

It was meant to be inherited! If it's not, that's a bug!

[LeaVerou]

Have you tried it and it does not inherit?

[lloydk]

Have you tried it and it does not inherit?

I tried it and it didn't work

[LeaVerou]

Ah bummer. Can you open an issue about that bug?

[lloydk]

Ah bummer. Can you open an issue about that bug?

Created a PR #15, it was a simple fix.

[LeaVerou]

Ah bummer. Can you open an issue about that bug?

Created a PR #15, it was a simple fix.

You rock man! Thanks so much.