Adjust tests for named parameters in SQL

Author: annda

_test/QueryBuilderTest.php

@@ -23,29 +23,4 @@ public function test_join()
         $qb->addLeftJoin('second', 'fourth', 'fourth', 'second.foo=fourth.foo');
         $this->assertEquals(['first', 'second', 'fourth', 'third'], array_keys($qb->from));
     }
-
-    public function test_placeholders()
-    {
-        $qb = new QueryBuilder();
-
-
-        $foo = $qb->addValue('foo');
-        $bar = $qb->addValue('bar');
-
-        $input = "this is $foo and $bar and $foo again";
-        $expect = "this is ? and ? and ? again";
-        $values = ['foo', 'bar', 'foo'];
-
-        $output = $qb->fixPlaceholders($input);
-
-        $this->assertEquals($expect, $output[0]);
-        $this->assertEquals($values, $output[1]);
-    }
-
-    public function test_placeholderfail()
-    {
-        $this->expectException(StructException::class);
-        $qb = new QueryBuilder();
-        $qb->fixPlaceholders('this has unknown placeholder :!!val7!!:');
-    }
 }

_test/StructTest.php

@@ -111,7 +111,7 @@ protected function getLang($key)
      */
     protected function cleanWS($string)
     {
-        return preg_replace('/\s+/s', '', $string);
+        return preg_replace(['/\s+/s', '/\:val(\d{1,3})/'], ['', '?'], $string);
     }
 
     /**

_test/mock/QueryBuilder.php

@@ -8,18 +8,13 @@ class QueryBuilder extends meta\QueryBuilder
 {
     public $from;
 
-    public function fixPlaceholders($sql)
-    {
-        return parent::fixPlaceholders($sql);
-    }
-
     /**
      * for debugging where statements
      *
      * @return array ($sql, $opts)
      */
     public function getWhereSQL()
     {
-        return $this->fixPlaceholders($this->filters()->toSQL());
+        return [$this->filters()->toSQL(), array_values($this->values)];
     }
 }

meta/QueryBuilder.php

@@ -241,7 +241,7 @@ public function getSQL()
                 'ORDER BY ' . implode(",\n", $this->orderby) . "\n";
         }
 
-        return [$sql, $this->values];
+        return [$sql, array_values($this->values)];
     }
 
     /**