Expand Screenplay tests to use Color

This might resolve the issues I'm having with FF/Safari tests
which fail, because they use different representations of color.

Author: craigfowler

CSF.Screenplay.Selenium/Color.cs

@@ -337,6 +337,30 @@ public static Color Parse(string webColorValue)
         /// <param name="sysColor">The <see cref="System.Drawing.Color"/></param>
         /// <returns><see langword="false"/> if the two different colors represent the same sRGB color; <see langword="true"/> if not.</returns>
         public static bool operator !=(Color color, SysColor sysColor) => !(color == sysColor);
+
+        /// <summary>
+        /// Gets a value indicating whether the specified <see cref="Color"/> and <see cref="System.Drawing.Color"/>
+        /// are equal.
+        /// </summary>
+        /// <remarks>
+        /// <para>Despite the type differences, two colors are equal if they represent the same sRGB color.</para>
+        /// </remarks>
+        /// <param name="color">The <see cref="Color"/></param>
+        /// <param name="sysColor">The <see cref="System.Drawing.Color"/></param>
+        /// <returns><see langword="true"/> if the two different colors represent the same sRGB color; <see langword="false"/> if not.</returns>
+        public static bool operator ==(SysColor sysColor, Color color) => color.Equals(sysColor);
+
+        /// <summary>
+        /// Gets a value indicating whether the specified <see cref="Color"/> and <see cref="System.Drawing.Color"/>
+        /// are not equal.
+        /// </summary>
+        /// <remarks>
+        /// <para>Despite the type differences, two colors are equal if they represent the same sRGB color.</para>
+        /// </remarks>
+        /// <param name="color">The <see cref="Color"/></param>
+        /// <param name="sysColor">The <see cref="System.Drawing.Color"/></param>
+        /// <returns><see langword="false"/> if the two different colors represent the same sRGB color; <see langword="true"/> if not.</returns>
+        public static bool operator !=(SysColor sysColor, Color color) => !(color == sysColor);
 
         /// <summary>
         /// Gets a value indicating whether the specified <see cref="Color"/> and <see cref="string"/> representation of a color
@@ -366,6 +390,34 @@ public static Color Parse(string webColorValue)
         /// <returns><see langword="false"/> if the current color and the string color representation indicate the same sRGB color; <see langword="true"/> if not.</returns>
         public static bool operator !=(Color color, string stringColor) => !(color == stringColor);
 
+        /// <summary>
+        /// Gets a value indicating whether the specified <see cref="Color"/> and <see cref="string"/> representation of a color
+        /// are equal.
+        /// </summary>
+        /// <remarks>
+        /// <para>A color is equal to a string if the result of <see cref="TryParse(string, out Color)"/> is <see langword="true"/> 
+        /// and if the resulting parsed <see cref="Color"/> instance is equal to the current color.
+        /// Note that this means that a color is never equal to a string which is not a valid color representation, or which is <see langword="null"/>.</para>
+        /// </remarks>
+        /// <param name="color">The <see cref="Color"/></param>
+        /// <param name="stringColor">The string which represents a color</param>
+        /// <returns><see langword="true"/> if the current color and the string color representation indicate the same sRGB color; <see langword="false"/> if not.</returns>
+        public static bool operator ==(string stringColor, Color color) => color.Equals(stringColor);
+
+        /// <summary>
+        /// Gets a value indicating whether the specified <see cref="Color"/> and <see cref="string"/> representation of a color
+        /// are not equal.
+        /// </summary>
+        /// <remarks>
+        /// <para>A color is equal to a string if the result of <see cref="TryParse(string, out Color)"/> is <see langword="true"/> 
+        /// and if the resulting parsed <see cref="Color"/> instance is equal to the current color.
+        /// Note that this means that a color is never equal to a string which is not a valid color representation, or which is <see langword="null"/>.</para>
+        /// </remarks>
+        /// <param name="color">The <see cref="Color"/></param>
+        /// <param name="stringColor">The string which represents a color</param>
+        /// <returns><see langword="false"/> if the current color and the string color representation indicate the same sRGB color; <see langword="true"/> if not.</returns>
+        public static bool operator !=(string stringColor, Color color) => !(color == stringColor);
+
  #if !RECORD_STRUCT_SUPPORT
        /// <summary>
         /// Gets a value indicating whether the two specified <see cref="Color"/> instances are equal.

Tests/CSF.Screenplay.Selenium.Tests/Actions/ExecuteJavaScriptTests.cs

@@ -24,6 +24,6 @@ public async Task ExecuteJavaScriptShouldBeAbleToExecuteAScriptWithParameters(IS
         await When(webster).AttemptsTo(ExecuteCustomScript(scriptBody).WithTheName("a script that changes the BG colour of an element").WithTheArguments("textContent", "#F00"));
         var backgroundColor = await Then(webster).Should(ReadFromTheElement(textContent).TheCssProperty("background-color"));
 
-        Assert.That(backgroundColor, Is.EqualTo("rgba(255, 0, 0, 1)"));
+        Assert.That(backgroundColor, Is.EqualTo(Colors.RED));
     }
 }

Tests/CSF.Screenplay.Selenium.Tests/Builders/QueryPredicatePrototypeBuilderTests.cs

@@ -175,7 +175,7 @@ public void CssPropertyShouldCreateASpecificationThatUsesGetCssValue(QueryPredic
             .Setup(e => e.GetCssValue("color"))
             .Returns("rgba(0, 0, 255, 1)");
 
-        var sut = builder.CssProperty("color", v => v == "rgba(255, 0, 0, 1)").GetElementSpecification();
+        var sut = builder.CssProperty("color", v => v == Colors.RED).GetElementSpecification();
 
         Assert.Multiple(() =>
         {
@@ -196,7 +196,7 @@ public void CssPropertyWithValueShouldCreateASpecificationThatUsesGetCssValue(Qu
             .Setup(e => e.GetCssValue("color"))
             .Returns("rgba(0, 0, 255, 1)");
 
-        var sut = builder.CssProperty("color", "rgba(255, 0, 0, 1)").GetElementSpecification();
+        var sut = builder.CssProperty("color", Colors.RED.ToRgbaString()).GetElementSpecification();
 
         Assert.Multiple(() =>
         {

Tests/CSF.Screenplay.Selenium.Tests/Queries/QueriesTests.cs

@@ -49,7 +49,7 @@ public async Task ReadingTheCssPropertyFromAnElementShouldReturnTheCorrectResult
         await Given(webster).WasAbleTo(OpenTheUrl(testPage));
         var result = await When(webster).AttemptsTo(ReadFromTheElement(backgroundDiv).TheCssProperty("background-color"));
 
-        Assert.That(result, Is.EqualTo("rgba(170, 170, 255, 1)"));
+        Assert.That(result, Is.EqualTo(Selenium.Color.Parse("rgba(170, 170, 255, 1)")));
     }
 
     [Test, Screenplay]