Query builder delete tests

sys/db/query_builder.php

@@ -508,7 +508,10 @@ class Query_Builder {
 	public function delete($table, $where='')
 	{
 		// Set the where clause
+		if ( ! empty($where))
+		{
 			$this->where($where);
+		}
 
 		// Create the SQL and parameters
 		$sql = $this->_compile("delete", $table);
@@ -618,7 +621,7 @@ class Query_Builder {
 			break;
 		}
 		
-		//echo $sql.'<br />';
+		echo $sql.'<br />';
 		
 		return $sql;
 	}

tests/databases/firebird-qb.php

@@ -32,7 +32,7 @@ class FirebirdQBTest extends UnitTestCase {
 		$params->pass = 'masterkey';
 		$this->qb = new Query_Builder($params);
 		
-		//echo '<hr /> Firebird Queries <br />';
+		echo '<hr /> Firebird Queries <hr />';
 	}
 
 	function TestQBGet()
@@ -126,4 +126,11 @@ class FirebirdQBTest extends UnitTestCase {
 			
 		$this->assertTrue($query);
 	}
+	
+	function TestDelete()
+	{
+		$query = $this->qb->where('id', 4)->delete('create_test');
+			
+		$this->assertTrue($query);
+	}
 }

tests/databases/sqlite-qb.php

@@ -28,7 +28,7 @@
 		$params->host = 'localhost';
 		$this->qb = new Query_Builder($params);
 		
-		//echo '<hr /> SQLite Queries <br />';
+		echo '<hr /> SQLite Queries <hr />';
  	}
 	
 	function TestGet()
@@ -120,4 +120,11 @@
 			
 		$this->assertIsA($query, 'PDOStatement');
 	}
+	
+	function TestDelete()
+	{
+		$query = $this->qb->where('id', 4)->delete('create_test');
+			
+		$this->assertIsA($query, 'PDOStatement');
+	}
 }