Fix bad overload logic, add another test

src/autowiring/config.h

@@ -29,12 +29,17 @@ namespace autowiring {
     explicit config(const T& value) { values[0] = value; }
     explicit config(T&& value) { values[0] = std::move(value); }
 
+    template<typename U>
+    explicit config(U&& value) {
+      values[0] = std::forward<U&&>(value);
+    }
+
   private:
     // Lock, used to move the asyncrhonous value to the value field
     autowiring::spin_lock lock;
 
     // Tracks whether the backing value has been updated
-    bool dirty = false;
+    bool dirty = true;
 
     // The index of the currently active value
     size_t valueIndex = 0;

src/autowiring/test/AutoConfigTest.cpp

@@ -12,14 +12,14 @@ class AutoConfigTest:
 namespace {
   class MyConfigurableClass {
   public:
-    autowiring::config<std::string> a = "Hello world!";
+    autowiring::config<std::string> a{ "Hello world!" };
     std::atomic<int> b = 929;
     std::atomic<unsigned int> bUnsigned = 92999;
     volatile bool c = false;
     volatile float d = 1.0f;
     volatile double e = 99.2;
     autowiring::observable<int> obs = 44;
-    autowiring::config<int> cfg = 929999;
+    autowiring::config<int> cfg{ 929999 };
 
     static autowiring::config_descriptor GetConfigDescriptor(void) {
       return {
@@ -44,6 +44,11 @@ namespace {
 static_assert(autowiring::has_getconfigdescriptor<MyConfigurableClass>::value, "Static new not correctly detected");
 static_assert(!autowiring::has_getconfigdescriptor<BadClass>::value, "Bad class cannot have a configuration descriptor");
 
+TEST_F(AutoConfigTest, ConfigFieldAssign) {
+  autowiring::config<std::string> x{ "Hello world!" };
+  ASSERT_FALSE(x.is_dirty()) << "Config values are assumed to be initially dirty";
+}
+
 TEST_F(AutoConfigTest, String) {
   MyConfigurableClass c;