Fix #239 - warn about bad brackets

src/Psalm/Type.php

@@ -40,6 +40,10 @@ abstract class Type
             $type_string = self::convertSquareBrackets($type_string);
         }
 
+        if (preg_match('/[\[\]()]/', $type_string)) {
+            throw new TypeParseTreeException('Invalid characters in type');
+        }
+
         $type_string = str_replace('?', 'null|', $type_string);
 
         $type_tokens = self::tokenize($type_string);

tests/TypeAlgebraTest.php

@@ -333,7 +333,7 @@ class TypeAlgebraTest extends TestCase
                             return false;
                         }
                         return true;
-                    }'
+                    }',
             ],
             'noParadoxAfterArrayAppending' => [
                 '<?php
@@ -361,7 +361,7 @@ class TypeAlgebraTest extends TestCase
                             return false;
                         }
                         return $errors;
-                    }'
+                    }',
             ],
             'noParadoxInCatch' => [
                 '<?php
@@ -384,8 +384,8 @@ class TypeAlgebraTest extends TestCase
                             if (!$arr) { return []; }
                         }
                         return $arr;
-                    }'
+                    }',
-            ]
+            ],
         ];
     }
 

tests/TypeParseTest.php

@@ -45,6 +45,16 @@ class TypeParseTest extends TestCase
         $this->assertSame('array<mixed, A|B>|C', (string) Type::parseString('A[]|B[]|C'));
     }
 
+    /**
+     * @expectedException \Psalm\Exception\TypeParseTreeException
+     *
+     * @return void
+     */
+    public function testInvalidType()
+    {
+        Type::parseString('array(A)');
+    }
+
     /**
      * @return void
      */