mirror of
https://github.com/phpbb/phpbb.git
synced 2025-05-09 00:55:23 +02:00
[ticket/11469] Split tests and skip multi_insert if unavailable
PHPBB3-11469
This commit is contained in:
parent
53f9e2131c
commit
9606ccc202
@ -23,34 +23,10 @@ class phpbb_dbal_sql_insert_buffer_test extends phpbb_database_test_case
|
|||||||
$db->sql_freeresult($result);
|
$db->sql_freeresult($result);
|
||||||
}
|
}
|
||||||
|
|
||||||
public function insert_buffer_data()
|
public function test_multi_insert_disabled_insert_and_flush()
|
||||||
{
|
{
|
||||||
$db = $this->new_dbal();
|
$db = $this->new_dbal();
|
||||||
|
$db->multi_insert = false;
|
||||||
if ($db->multi_insert)
|
|
||||||
{
|
|
||||||
// Test with enabled and disabled multi_insert
|
|
||||||
return array(
|
|
||||||
array(true),
|
|
||||||
array(false),
|
|
||||||
);
|
|
||||||
}
|
|
||||||
else
|
|
||||||
{
|
|
||||||
// Only test with disabled multi_insert, the DB doesn't support it
|
|
||||||
return array(
|
|
||||||
array(false),
|
|
||||||
);
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
/**
|
|
||||||
* @dataProvider insert_buffer_data
|
|
||||||
*/
|
|
||||||
public function test_insert_and_flush($force_multi_insert)
|
|
||||||
{
|
|
||||||
$db = $this->new_dbal();
|
|
||||||
$db->multi_insert = $force_multi_insert;
|
|
||||||
|
|
||||||
$buffer = new phpbb_db_sql_insert_buffer($db, 'phpbb_config', 2);
|
$buffer = new phpbb_db_sql_insert_buffer($db, 'phpbb_config', 2);
|
||||||
|
|
||||||
@ -63,14 +39,7 @@ class phpbb_dbal_sql_insert_buffer_test extends phpbb_database_test_case
|
|||||||
'is_dynamic' => '0',
|
'is_dynamic' => '0',
|
||||||
));
|
));
|
||||||
|
|
||||||
if ($db->multi_insert)
|
|
||||||
{
|
|
||||||
$this->assert_config_count($db, 2);
|
|
||||||
}
|
|
||||||
else
|
|
||||||
{
|
|
||||||
$this->assert_config_count($db, 3);
|
$this->assert_config_count($db, 3);
|
||||||
}
|
|
||||||
|
|
||||||
// Manually flush
|
// Manually flush
|
||||||
$buffer->flush();
|
$buffer->flush();
|
||||||
@ -78,13 +47,38 @@ class phpbb_dbal_sql_insert_buffer_test extends phpbb_database_test_case
|
|||||||
$this->assert_config_count($db, 3);
|
$this->assert_config_count($db, 3);
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
public function test_multi_insert_enabled_insert_and_flush()
|
||||||
* @dataProvider insert_buffer_data
|
|
||||||
*/
|
|
||||||
public function test_insert_with_flush($force_multi_insert)
|
|
||||||
{
|
{
|
||||||
$db = $this->new_dbal();
|
$db = $this->new_dbal();
|
||||||
$db->multi_insert = $force_multi_insert;
|
|
||||||
|
if (!$db->multi_insert)
|
||||||
|
{
|
||||||
|
$this->markTestSkipped('Database does not support multi_insert');
|
||||||
|
}
|
||||||
|
|
||||||
|
$buffer = new phpbb_db_sql_insert_buffer($db, 'phpbb_config', 2);
|
||||||
|
|
||||||
|
$this->assert_config_count($db, 2);
|
||||||
|
|
||||||
|
// This call can be buffered
|
||||||
|
$buffer->insert(array(
|
||||||
|
'config_name' => 'name1',
|
||||||
|
'config_value' => 'value1',
|
||||||
|
'is_dynamic' => '0',
|
||||||
|
));
|
||||||
|
|
||||||
|
$this->assert_config_count($db, 2);
|
||||||
|
|
||||||
|
// Manually flush
|
||||||
|
$buffer->flush();
|
||||||
|
|
||||||
|
$this->assert_config_count($db, 3);
|
||||||
|
}
|
||||||
|
|
||||||
|
public function test_multi_insert_disabled_insert_with_flush()
|
||||||
|
{
|
||||||
|
$db = $this->new_dbal();
|
||||||
|
$db->multi_insert = false;
|
||||||
|
|
||||||
$buffer = new phpbb_db_sql_insert_buffer($db, 'phpbb_config', 2);
|
$buffer = new phpbb_db_sql_insert_buffer($db, 'phpbb_config', 2);
|
||||||
|
|
||||||
@ -106,13 +100,39 @@ class phpbb_dbal_sql_insert_buffer_test extends phpbb_database_test_case
|
|||||||
$this->assert_config_count($db, 4);
|
$this->assert_config_count($db, 4);
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
public function test_multi_insert_enabled_insert_with_flush()
|
||||||
* @dataProvider insert_buffer_data
|
|
||||||
*/
|
|
||||||
public function test_insert_all_and_flush($force_multi_insert)
|
|
||||||
{
|
{
|
||||||
$db = $this->new_dbal();
|
$db = $this->new_dbal();
|
||||||
$db->multi_insert = $force_multi_insert;
|
|
||||||
|
if (!$db->multi_insert)
|
||||||
|
{
|
||||||
|
$this->markTestSkipped('Database does not support multi_insert');
|
||||||
|
}
|
||||||
|
|
||||||
|
$buffer = new phpbb_db_sql_insert_buffer($db, 'phpbb_config', 2);
|
||||||
|
|
||||||
|
$this->assert_config_count($db, 2);
|
||||||
|
|
||||||
|
$buffer->insert(array(
|
||||||
|
'config_name' => 'name1',
|
||||||
|
'config_value' => 'value1',
|
||||||
|
'is_dynamic' => '0',
|
||||||
|
));
|
||||||
|
|
||||||
|
// This call flushes the values
|
||||||
|
$buffer->insert(array(
|
||||||
|
'config_name' => 'name2',
|
||||||
|
'config_value' => 'value2',
|
||||||
|
'is_dynamic' => '0',
|
||||||
|
));
|
||||||
|
|
||||||
|
$this->assert_config_count($db, 4);
|
||||||
|
}
|
||||||
|
|
||||||
|
public function test_multi_insert_disabled_insert_all_and_flush()
|
||||||
|
{
|
||||||
|
$db = $this->new_dbal();
|
||||||
|
$db->multi_insert = false;
|
||||||
|
|
||||||
$buffer = new phpbb_db_sql_insert_buffer($db, 'phpbb_config', 2);
|
$buffer = new phpbb_db_sql_insert_buffer($db, 'phpbb_config', 2);
|
||||||
|
|
||||||
@ -136,13 +156,44 @@ class phpbb_dbal_sql_insert_buffer_test extends phpbb_database_test_case
|
|||||||
),
|
),
|
||||||
));
|
));
|
||||||
|
|
||||||
if ($db->multi_insert)
|
$this->assert_config_count($db, 5);
|
||||||
|
}
|
||||||
|
|
||||||
|
public function test_multi_insert_enabled_insert_all_and_flush()
|
||||||
{
|
{
|
||||||
|
$db = $this->new_dbal();
|
||||||
|
|
||||||
|
if (!$db->multi_insert)
|
||||||
|
{
|
||||||
|
$this->markTestSkipped('Database does not support multi_insert');
|
||||||
|
}
|
||||||
|
|
||||||
|
$buffer = new phpbb_db_sql_insert_buffer($db, 'phpbb_config', 2);
|
||||||
|
|
||||||
|
$this->assert_config_count($db, 2);
|
||||||
|
|
||||||
|
$buffer->insert_all(array(
|
||||||
|
array(
|
||||||
|
'config_name' => 'name1',
|
||||||
|
'config_value' => 'value1',
|
||||||
|
'is_dynamic' => '0',
|
||||||
|
),
|
||||||
|
array(
|
||||||
|
'config_name' => 'name2',
|
||||||
|
'config_value' => 'value2',
|
||||||
|
'is_dynamic' => '0',
|
||||||
|
),
|
||||||
|
array(
|
||||||
|
'config_name' => 'name3',
|
||||||
|
'config_value' => 'value3',
|
||||||
|
'is_dynamic' => '0',
|
||||||
|
),
|
||||||
|
));
|
||||||
|
|
||||||
$this->assert_config_count($db, 4);
|
$this->assert_config_count($db, 4);
|
||||||
|
|
||||||
// Manually flush
|
// Manually flush
|
||||||
$buffer->flush();
|
$buffer->flush();
|
||||||
}
|
|
||||||
|
|
||||||
$this->assert_config_count($db, 5);
|
$this->assert_config_count($db, 5);
|
||||||
}
|
}
|
||||||
|
Loading…
x
Reference in New Issue
Block a user