Do not allow anymore to create Variables or Attributes with an empty name

include/cdfpp/attribute.hpp

@@ -56,6 +56,10 @@ struct Attribute
     Attribute& operator=(const Attribute&) = default;
     Attribute(const std::string& name, attr_data_t&& data) : name { name }
     {
+        if (name.empty())
+        {
+            throw std::invalid_argument { "Attribute name cannot be empty" };
+        }
         this->data = std::move(data);
     }
 
@@ -177,6 +181,10 @@ struct VariableAttribute
     VariableAttribute& operator=(const VariableAttribute&) = default;
     VariableAttribute(const std::string& name, attr_data_t&& data) : name { name }
     {
+        if (name.empty())
+        {
+            throw std::invalid_argument { "Attribute name cannot be empty" };
+        }
         this->data = std::move(data);
     }
 

include/cdfpp/cdf-file.hpp

@@ -36,6 +36,8 @@
 template <class stream_t>
 inline stream_t& operator<<(stream_t& os, const cdf_map<std::string, cdf::Variable>& variables)
 {
+    std::for_each(std::cbegin(variables), std::cend(variables),
+        [&os](const auto& item) { item.second.__repr__(os, indent_t {}, false); });
     return os;
 }
 

include/cdfpp/variable.hpp

@@ -102,6 +102,10 @@ struct Variable
             , p_is_nrv { is_nrv }
             , p_compression { compression_type }
     {
+        if (name.empty())
+        {
+            throw std::invalid_argument { "Variable name cannot be empty" };
+        }
         if (this->majority() == cdf_majority::column)
         {
             majority::swap(_data(), p_shape);
@@ -120,6 +124,10 @@ struct Variable
             , p_is_nrv { is_nrv }
             , p_compression { compression_type }
     {
+        if (name.empty())
+        {
+            throw std::invalid_argument { "Variable name cannot be empty" };
+        }
     }
 
     inline bool operator==(const Variable& other) const

tests/python_variable_set_values/test.py

@@ -263,6 +263,20 @@ def test_filter_cdf_with_regex(self):
         self.assertIn("global_attr2", self.cdf.attributes)
         self.assertIn("global_attr3", self.cdf.attributes)
 
+class PycdfEmptyNamesAreNotAllowed(unittest.TestCase):
+    def test_variable_name_cannot_be_empty(self):
+        cdf = pycdfpp.CDF()
+        with self.assertRaises(ValueError):
+            cdf.add_variable("", values=np.arange(10))
+
+    def test_attribute_name_cannot_be_empty(self):
+        cdf = pycdfpp.CDF()
+        cdf.add_variable("var1", values=np.arange(10))
+        with self.assertRaises(ValueError):
+            cdf["var1"].add_attribute("", "value")
+        with self.assertRaises(ValueError):
+            cdf.add_attribute("", ["value"])
+
 
 if __name__ == '__main__':
     unittest.main()