Do not include empty lists/dicts in NWBFile repr(). Fix #940 (#998)

Author: rly

src/pynwb/core.py

@@ -165,7 +165,8 @@ def __repr__(self):
         template = "\n{} {}\nFields:\n""".format(getattr(self, 'name'), type(self))
         for k in sorted(self.fields):  # sorted to enable tests
             v = self.fields[k]
-            template += "  {}: {}\n".format(k, self.__smart_str(v, 1))
+            if not hasattr(v, '__len__') or len(v) > 0:
+                template += "  {}: {}\n".format(k, self.__smart_str(v, 1))
         return template
 
     @staticmethod

tests/unit/test_core.py

@@ -335,22 +335,11 @@ def test_print_file(self):
     name <class 'pynwb.base.TimeSeries'>,
     name2 <class 'pynwb.base.TimeSeries'>
   }
-  analysis: { }
-  devices: { }
-  electrode_groups: { }
   epoch_tags: {
     tag1,
     tag2
   }
   epochs: epochs <class 'pynwb.epoch.TimeIntervals'>
-  ic_electrodes: { }
-  imaging_planes: { }
-  intervals: { }
-  lab_meta_data: { }
-  ogen_sites: { }
-  processing: { }
-  stimulus: { }
-  stimulus_template: { }
 """)