Replace HslColor for HSLColor from painting

[calvintam236]

No description provided.

[calvintam236]

@WieFel any thoughts?

[WieFel]

@calvintam236 sorry, I forgot to press submit review button 😬

[calvintam236]

@WieFel @dsyrstad do you have a list of expected result from test? Upon checking, the test is basically empty, not checking for output. I can write the test if someone can gave me the parameter input and expected output.

[WieFel]

I would say

1. Copy every "old" version of the functions I commented on.
2. Put them in the test file and name them e.g. lightenOld(), etc.
3. For every function, loop from through the range of min to max allowed value. E.g. in the case of lighten from 0 to 100, for example with step of 10.
4. Compare output of "old" with "new" function and assert they are equal.

[calvintam236]

I would say

1. Copy every "old" version of the functions I commented on.
2. Put them in the test file and name them e.g. lightenOld(), etc.
3. For every function, loop from through the range of min to max allowed value. E.g. in the case of lighten from 0 to 100, for example with step of 10.
4. Compare output of "old" with "new" function and assert they are equal.

I am referring to writing tests for CI. It does not work for testing in the future.

[WieFel]

@calvintam236 sorry for not answering. I am very busy with work.
Just try to come up with some ideas yourself and we can discuss it ;)

[calvintam236]

Blocked by #35

[calvintam236]

@WieFel I just added test for HSL. They are passed so it should be okay.

[calvintam236]

Note that the ci/format test can be ignored at this moment as the tests will be updated again in #41.