RuntimeConfig: Set default variable state to Unspecified (googleapis#4738)

chemelnucfin · web-flow · commit 9cce5181c868 · 2018-01-16T09:24:24.000-08:00
diff --git a/runtimeconfig/google/cloud/runtimeconfig/variable.py b/runtimeconfig/google/cloud/runtimeconfig/variable.py
@@ -139,10 +139,10 @@ def state(self):
 
         :rtype: str
         :returns:
-            If set, one of "UPDATED", "DELETED", or
-            "VARIABLE_STATE_UNSPECIFIED", else ``None``.
+            If set, one of "UPDATED", "DELETED", or defaults to
+            "VARIABLE_STATE_UNSPECIFIED".
         """
-        return self._properties.get('state')
+        return self._properties.get('state', STATE_UNSPECIFIED)
 
     @property
     def update_time(self):
diff --git a/runtimeconfig/tests/unit/test_variable.py b/runtimeconfig/tests/unit/test_variable.py
@@ -56,13 +56,15 @@ def _verifyResourceProperties(self, variable, resource):
 
     def test_ctor(self):
         from google.cloud.runtimeconfig.config import Config
+        from google.cloud.runtimeconfig.variable import STATE_UNSPECIFIED
 
         client = _Client(project=self.PROJECT)
         config = Config(name=self.CONFIG_NAME, client=client)
         variable = self._make_one(name=self.VARIABLE_NAME, config=config)
         self.assertEqual(variable.name, self.VARIABLE_NAME)
         self.assertEqual(variable.full_name, self.PATH)
         self.assertEqual(variable.path, '/%s' % (self.PATH,))
+        self.assertEqual(variable.state, STATE_UNSPECIFIED)
         self.assertIs(variable.client, client)
 
     def test_ctor_w_no_name(self):
