Throw an exception when adding a field with an empty value

Author: Enric Sala

influxdb-client/Point.m

@@ -25,7 +25,9 @@
 
         % Add a field value
         function obj = field(obj, key, value)
-            if isfloat(value)
+            if isempty(value)
+                error('field:emptyValue', 'value of field "%s" is empty', key);
+            elseif isfloat(value)
                 obj.Fields{end + 1} = sprintf('%s=%.8g', key, value);
             elseif isinteger(value)
                 obj.Fields{end + 1} = sprintf('%s=%ii', key, value);

influxdb-client/Series.m

@@ -26,7 +26,7 @@
         % Add a field value
         function obj = field(obj, key, value)
             if isempty(value)
-                warning(['value of field ' key ' is empty']);
+                error('field:emptyValue', 'value of field "%s" is empty', key);
             elseif isnumeric(value) || islogical(value)
                 field = struct('key', key, 'value', value);
                 obj.Fields{end + 1} = field;

tests/PointTest.m

@@ -11,6 +11,11 @@ function build_fails_when_empty_fields(test)
             test.verifyError(f, 'toLine:emptyFields');
         end
 
+        function field_with_empty_value_fails(test)
+            f = @() Point('weather').fields('temperature', []);
+            test.verifyError(f, 'field:emptyValue');
+        end
+        
         function single_field(test)
             p = Point('weather').fields('temperature', 24.3);
             exp = 'weather temperature=24.3';

tests/SeriesTest.m

@@ -29,6 +29,12 @@ function fails_when_fields_are_not_set(test)
             test.verifyError(@() s.toLine(), 'toLine:emptyFields');
         end
 
+        function field_with_empty_value_fails(test)
+            f = @() Series('weather') ...
+                .fields('temperature', []);
+            test.verifyError(f, 'field:emptyValue');
+        end
+        
         function single_field(test)
             s = Series('weather') ...
                 .time(test.Time) ...