Commit cfda9ab1 authored by gbalduzz's avatar gbalduzz
Browse files

Removed extra group in vector of strings.

parent f45dca65

src/io/hdf5/hdf5_reader.cpp

+0 −4
Original line number Diff line number Diff line
@@ -88,11 +88,7 @@ bool HDF5Reader::execute(const std::string& name, std::vector<std::string>& valu 
  open_group("data");



  for (size_t l = 0; l < value.size(); l++) {

    open_group(std::to_string(l));



    execute(std::to_string(l), value[l]);



    close_group();

  }



  close_group();

src/io/hdf5/hdf5_writer.cpp

+0 −2
Original line number Diff line number Diff line
@@ -98,9 +98,7 @@ void HDF5Writer::execute(const std::string& name, 
    open_group("data");



    for (int i = 0; i < value.size(); ++i) {

      open_group(std::to_string(i));

      execute(std::to_string(i), value[i]);

      close_group();

    }



    close_group();

test/unit/io/hdf5_reader_writer_test.cpp

+4 −3
Original line number Diff line number Diff line
@@ -145,19 +145,20 @@ TEST(HDF5ReaderWriterTest, VectorOfArraysReadWrite) { 


TEST(HDF5ReaderWriterTest, VectorOfStringsReadWrite) {

  std::vector<std::string> s1{"foo", "bar", "baz"};

  const std::string filename = "test_vec_of_strings.hdf5";



  // Create test file.

  dca::io::HDF5Writer writer;

  writer.open_file("test.hdf5");

  writer.open_file(filename);

  writer.execute("strings", s1);

  writer.close_file();



  // Read test file.

  dca::io::HDF5Reader reader;

  reader.open_file("test.hdf5");

  reader.open_file(filename);



  std::vector<std::string> s2;

  reader.execute("strings", s2);

  EXPECT_TRUE(reader.execute("strings", s2));

  EXPECT_EQ(s1, s2);

}