proper default for iphone

[wxWidgets.git] / tests / graphics / colour.cpp

diff --git a/tests/graphics/colour.cpp b/tests/graphics/colour.cpp
index 0269e6bc1b37b9a41ff7a788a5747606b4278a92..40b92bbd63a3adfd772960dd73facc7f37e1fa0a 100644 (file)
--- a/tests/graphics/colour.cpp
+++ b/tests/graphics/colour.cpp
@@ -3,7 +3,6 @@
 // Purpose:     wxColour unit test
 // Author:      Vadim Zeitlin
 // Created:     2009-09-19
 // Purpose:     wxColour unit test
 // Author:      Vadim Zeitlin
 // Created:     2009-09-19

-// RCS-ID:      $Id$

 // Copyright:   (c) 2009 Vadim Zeitlin <vadim@wxwidgets.org>
 ///////////////////////////////////////////////////////////////////////////////
 
 // Copyright:   (c) 2009 Vadim Zeitlin <vadim@wxwidgets.org>
 ///////////////////////////////////////////////////////////////////////////////
 


@@ -18,50 +17,24 @@
 #endif
 
 #include "wx/colour.h"
 #endif
 
 #include "wx/colour.h"

+#include "asserthelper.h"

 
 // ----------------------------------------------------------------------------
 
 // ----------------------------------------------------------------------------

-// helper functions
+// helpers

 // ----------------------------------------------------------------------------
 
 // ----------------------------------------------------------------------------
 

-namespace
-{
-    // by default colour components values are output incorrectly because they
-    // are unsigned chars, define a small helper struct which formats them in
-    // a more useful way
-    struct ColourChannel
-    {
-        ColourChannel(unsigned char value) : m_value(value) { }
-
-        unsigned char m_value;
-    };
-
-    std::ostream& operator<<(std::ostream& os, const ColourChannel& cc)
-    {
-        os.width(2);
-        os.fill('0');
-        os << static_cast<int>(cc.m_value);
-        return os;
-    }
-} // anonymous namespace
-
-// this operator is needed to use CPPUNIT_ASSERT_EQUAL with wxColour objects
-std::ostream& operator<<(std::ostream& os, const wxColour& c)
-{
-    os << std::hex << std::noshowbase
-       << "("
-       << ColourChannel(c.Red()) << ", "
-       << ColourChannel(c.Green()) << ", "
-       << ColourChannel(c.Blue());
-
-    if ( const unsigned char a = c.Alpha() )
-    {
-        os << ", " << ColourChannel(a);
-    }
+// Check the colour components, with and without alpha.
+//
+// NB: These are macros and not functions to have correct line numbers in case
+//     of failure.
+#define ASSERT_EQUAL_RGB(c, r, g, b) \
+    CPPUNIT_ASSERT_EQUAL( r, (int)c.Red() ); \
+    CPPUNIT_ASSERT_EQUAL( g, (int)c.Green() ); \
+    CPPUNIT_ASSERT_EQUAL( b, (int)c.Blue() )

 
 

-    os << ")";
-
-    return os;
-}
+#define ASSERT_EQUAL_RGBA(c, r, g, b, a) \
+    ASSERT_EQUAL_RGB(c, r, g, b); \
+    CPPUNIT_ASSERT_EQUAL( a, (int)c.Alpha() )

 
 // ----------------------------------------------------------------------------
 // test class
 
 // ----------------------------------------------------------------------------
 // test class


@@ -75,9 +48,11 @@ public:
 private:
     CPPUNIT_TEST_SUITE( ColourTestCase );
         CPPUNIT_TEST( GetSetRGB );
 private:
     CPPUNIT_TEST_SUITE( ColourTestCase );
         CPPUNIT_TEST( GetSetRGB );

+        CPPUNIT_TEST( FromString );

     CPPUNIT_TEST_SUITE_END();
 
     void GetSetRGB();
     CPPUNIT_TEST_SUITE_END();
 
     void GetSetRGB();

+    void FromString();

 
     DECLARE_NO_COPY_CLASS(ColourTestCase)
 };
 
     DECLARE_NO_COPY_CLASS(ColourTestCase)
 };


@@ -85,7 +60,7 @@ private:
 // register in the unnamed registry so that these tests are run by default
 CPPUNIT_TEST_SUITE_REGISTRATION( ColourTestCase );
 
 // register in the unnamed registry so that these tests are run by default
 CPPUNIT_TEST_SUITE_REGISTRATION( ColourTestCase );
 

-// also include in it's own registry so that these tests can be run alone
+// also include in its own registry so that these tests can be run alone

 CPPUNIT_TEST_SUITE_NAMED_REGISTRATION( ColourTestCase, "ColourTestCase" );
 
 void ColourTestCase::GetSetRGB()
 CPPUNIT_TEST_SUITE_NAMED_REGISTRATION( ColourTestCase, "ColourTestCase" );
 
 void ColourTestCase::GetSetRGB()


@@ -106,12 +81,34 @@ void ColourTestCase::GetSetRGB()
     CPPUNIT_ASSERT_EQUAL( 0xdd, (int)c.Red() );
     CPPUNIT_ASSERT_EQUAL( 0xcc, (int)c.Green() );
     CPPUNIT_ASSERT_EQUAL( 0xbb, (int)c.Blue() );
     CPPUNIT_ASSERT_EQUAL( 0xdd, (int)c.Red() );
     CPPUNIT_ASSERT_EQUAL( 0xcc, (int)c.Green() );
     CPPUNIT_ASSERT_EQUAL( 0xbb, (int)c.Blue() );

+
+    // wxX11 doesn't support alpha at all currently.
+#ifndef __WXX11__

     CPPUNIT_ASSERT_EQUAL( 0xaa, (int)c.Alpha() );
     CPPUNIT_ASSERT_EQUAL( 0xaa, (int)c.Alpha() );

+#endif // __WXX11__

 
     // FIXME: at least under wxGTK wxColour ctor doesn't take alpha channel
     //        into account: bug or feature?
     //CPPUNIT_ASSERT_EQUAL( wxColour(0xaabbccdd), c );
     CPPUNIT_ASSERT_EQUAL( 0xbbccdd, c.GetRGB() );
 
     // FIXME: at least under wxGTK wxColour ctor doesn't take alpha channel
     //        into account: bug or feature?
     //CPPUNIT_ASSERT_EQUAL( wxColour(0xaabbccdd), c );
     CPPUNIT_ASSERT_EQUAL( 0xbbccdd, c.GetRGB() );

+#ifndef __WXX11__

     CPPUNIT_ASSERT_EQUAL( 0xaabbccdd, c.GetRGBA() );
     CPPUNIT_ASSERT_EQUAL( 0xaabbccdd, c.GetRGBA() );

+#endif // __WXX11__
+}
+
+void ColourTestCase::FromString()
+{
+    ASSERT_EQUAL_RGB( wxColour("rgb(11, 22, 33)"), 11, 22, 33 );
+    ASSERT_EQUAL_RGBA( wxColour("rgba(11, 22, 33, 0.5)"), 11, 22, 33, 128 );
+    ASSERT_EQUAL_RGBA( wxColour("rgba( 11, 22, 33, 0.5 )"), 11, 22, 33, 128 );
+
+    ASSERT_EQUAL_RGB( wxColour("#aabbcc"), 0xaa, 0xbb, 0xcc );
+
+    ASSERT_EQUAL_RGB( wxColour("red"), 0xff, 0, 0 );
+
+    wxColour col;
+    CPPUNIT_ASSERT( !wxFromString("rgb(1, 2)", &col) );
+    CPPUNIT_ASSERT( !wxFromString("rgba(1, 2, 3.456)", &col) );
+    CPPUNIT_ASSERT( !wxFromString("rgba(1, 2, 3.456, foo)", &col) );

 }
 
 }