Add possibility to create Suits from chars

src/lib/pokerstove/peval/Suit.cpp

@@ -246,6 +246,13 @@ Suit::Suit(const std::string& str)
     : _suit(suit_code(str[0]))
 {}
 
+Suit::Suit(uint8_t c)
+{
+    if (isSuitChar(c))
+        _suit = suit_code(c);
+    else
+        _suit = c;
+}
 
 string Suit::str() const
 {

src/lib/pokerstove/peval/Suit.h

@@ -47,7 +47,7 @@ class Suit
      * object to be created.
      */
     explicit Suit(const std::string& str);
-    explicit Suit(uint8_t c)     : _suit(c) {}
+    explicit Suit(uint8_t c);
 
 
     /**

src/lib/pokerstove/peval/Suit.test.cpp

@@ -0,0 +1,38 @@
+#include <gtest/gtest.h>
+#include "Suit.h"
+
+using namespace pokerstove;
+
+const char* suits = "cdhsCDHS";
+
+TEST(Suit, ConstructorChars) {
+    char suitstr[] = "?";
+
+    // test the valid char range
+    for (int i=0; i<strlen(suits); i++) {
+        Suit s(suits[i]);
+        suitstr[0] = suits[i];
+        EXPECT_STRCASEEQ(suitstr, s.str().c_str());
+    }
+}
+
+TEST(Suit, ConstructorInts) {
+    char suitstr[] = "?";
+
+    // test the valid int range
+    for (int i=0; i<=3; i++) {
+        Suit s(i);
+        suitstr[0] = suits[i];
+        EXPECT_STREQ(suitstr, s.str().c_str());
+    }
+}
+
+TEST(Suit, ConstructorErrs) {
+    char suitstr[] = "?";
+
+    // test the invalid int range
+    for (int i=4; i<'C'; i++) {
+        Suit s(i);
+        EXPECT_STREQ(suitstr, s.str().c_str());
+    }
+}