Fix PyLint Comprehension Expressions (#38)

These are un-needed comprehension expressions that should just use
list method

Signed-off-by: David Brown <dmlb2000@gmail.com>

Author: dmlb2000

tests/expression_test.py

@@ -36,7 +36,7 @@ def test_failure_expressions(self):
         ]:
             print(path)
             expr = Path.parse_str(path)
-            self.assertFalse([x for x in expr.match(data)])
+            self.assertFalse(list(expr.match(data)))
 
     def test_unary_operator(self):
         """Test the unary not in a path."""
@@ -48,7 +48,7 @@ def test_unary_operator(self):
         ]
         expr = Path.parse_str('$[?(not @.bar)]')
         self.assertEqual(Path.parse_str(str(expr)), expr)
-        self.assertTrue([x for x in expr.match(data)])
+        self.assertTrue(list(expr.match(data)))
 
     def test_unary_boolean_operator(self):
         """Test the unary not in a path."""
@@ -60,7 +60,7 @@ def test_unary_boolean_operator(self):
         ]
         expr = Path.parse_str('$[?(not (@.bar or (@.fiz > 2 and @.biz > 2)))]')
         self.assertEqual(Path.parse_str(str(expr)), expr)
-        self.assertTrue([x for x in expr.match(data)])
+        self.assertTrue(list(expr.match(data)))
 
     def test_variadic_operator(self):
         """Test a variadic operator."""