Add test for Table.__getitem__ indexing

nilmdb/server/bulkdata.py

@@ -480,7 +480,7 @@ class Table(object):
 
     def __getitem__(self, row):
         """Extract timestamps from a row, with table[n] notation."""
-        if (row is None or row < 0 or row >= self.nrows):
+        if row < 0 or row >= self.nrows:
             raise IndexError("Index out of range")
         (subdir, filename, offset, count) = self._offset_from_row(row)
         f = self.file_open(subdir, filename)

tests/test_bulkdata.py

@@ -59,6 +59,8 @@ class TestBulkData(object):
         node = data.getnode("/foo/bar")
         with assert_raises(IndexError):
             x = get_node_slice(0)
+        with assert_raises(IndexError):
+            x = node[0]
         raw = []
         for i in range(1000):
             raw.append("%d 1 2 3 4 5 6 7 8\n" % (10000 + i))