Skip to content

Commit

Permalink
[ColorPicker]Fix HSB color format and add unit tests (#19634)
Browse files Browse the repository at this point in the history
* Change HSB color convert and add unit test

* Using HSV implementation to not duplicate code.
  • Loading branch information
sosssego committed Aug 2, 2022
1 parent 0d0aaca commit 6caf472
Show file tree
Hide file tree
Showing 2 changed files with 53 additions and 1 deletion.
5 changes: 4 additions & 1 deletion src/modules/colorPicker/ColorPickerUI/Helpers/ColorHelper.cs
Expand Up @@ -58,7 +58,10 @@ internal static (double red, double green, double blue) ConvertToDouble(Color co
/// <param name="color">The <see cref="Color"/> to convert</param>
/// <returns>The hue [0°..360°], saturation [0..1] and brightness [0..1] of the converted color</returns>
internal static (double hue, double saturation, double brightness) ConvertToHSBColor(Color color)
=> (color.GetHue(), color.GetSaturation(), color.GetBrightness());
{
// HSB and HSV represents the same color space
return ConvertToHSVColor(color);
}

/// <summary>
/// Convert a given <see cref="Color"/> to a HSI color (hue, saturation, intensity)
Expand Down
Expand Up @@ -114,6 +114,55 @@ public void ColorRGBtoHSVTest(double hue, double saturation, double value, int r
Assert.AreEqual(result.value * 100d, value, 0.2d);
}

// test values taken from https://de.wikipedia.org/wiki/HSV-Farbraum
[TestMethod]
[DataRow(000, 000, 000, 000, 000, 000)] // Black
[DataRow(000, 000, 100, 100, 100, 100)] // White
[DataRow(000, 100, 100, 100, 000, 000)] // Red
[DataRow(015, 100, 100, 100, 025, 000)] // Vermilion/Cinnabar
[DataRow(020, 075, 036, 036, 018, 009)] // Brown
[DataRow(030, 100, 100, 100, 050, 000)] // Orange
[DataRow(045, 100, 100, 100, 075, 000)] // Saffron
[DataRow(060, 100, 100, 100, 100, 000)] // Yellow
[DataRow(075, 100, 100, 075, 100, 000)] // Light green-yellow
[DataRow(090, 100, 100, 050, 100, 000)] // Green-yellow
[DataRow(105, 100, 100, 025, 100, 000)] // Lime
[DataRow(120, 100, 050, 000, 050, 000)] // Dark green
[DataRow(120, 100, 100, 000, 100, 000)] // Green
[DataRow(135, 100, 100, 000, 100, 025)] // Light blue-green
[DataRow(150, 100, 100, 000, 100, 050)] // Blue-green
[DataRow(165, 100, 100, 000, 100, 075)] // Green-cyan
[DataRow(180, 100, 100, 000, 100, 100)] // Cyan
[DataRow(195, 100, 100, 000, 075, 100)] // Blue-cyan
[DataRow(210, 100, 100, 000, 050, 100)] // Green-blue
[DataRow(225, 100, 100, 000, 025, 100)] // Light green-blue
[DataRow(240, 100, 100, 000, 000, 100)] // Blue
[DataRow(255, 100, 100, 025, 000, 100)] // Indigo
[DataRow(270, 100, 100, 050, 000, 100)] // Purple
[DataRow(285, 100, 100, 075, 000, 100)] // Blue-magenta
[DataRow(300, 100, 100, 100, 000, 100)] // Magenta
[DataRow(315, 100, 100, 100, 000, 075)] // Red-magenta
[DataRow(330, 100, 100, 100, 000, 050)] // Blue-red
[DataRow(345, 100, 100, 100, 000, 025)] // Light blue-red
public void ColorRGBtoHSBTest(double hue, double saturation, double value, int red, int green, int blue)
{
red = Convert.ToInt32(Math.Round(255d / 100d * red)); // [0%..100%] to [0..255]
green = Convert.ToInt32(Math.Round(255d / 100d * green)); // [0%..100%] to [0..255]
blue = Convert.ToInt32(Math.Round(255d / 100d * blue)); // [0%..100%] to [0..255]

var color = Color.FromArgb(255, red, green, blue);
var result = ColorHelper.ConvertToHSBColor(color);

// hue [0°..360°]
Assert.AreEqual(result.hue, hue, 0.2d);

// saturation[0..1]
Assert.AreEqual(result.saturation * 100d, saturation, 0.2d);

// value[0..1]
Assert.AreEqual(result.brightness * 100d, value, 0.2d);
}

[TestMethod]
[DataRow(000, 000, 000, 100, 000, 000, 000)] // Black
[DataRow(000, 000, 000, 000, 255, 255, 255)] // White
Expand Down

0 comments on commit 6caf472

Please sign in to comment.