test: more locks to avoid race conditions

nextras · Mar 2, 2025 · 65b9ac9 · 65b9ac9
1 parent b0a5e1b
commit 65b9ac9
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 0 deletions.
diff --git a/tests/cases/integration/platform.mysql.phpt b/tests/cases/integration/platform.mysql.phpt
@@ -23,6 +23,7 @@ class PlatformMysqlTest extends IntegrationTestCase
 {
 	public function testTables()
 	{
+		$this->lockConnection($this->connection);
 		$dbName = $this->connection->getConfig()['database'];
 		$tables = $this->connection->getPlatform()->getTables();
 
@@ -44,6 +45,7 @@ class PlatformMysqlTest extends IntegrationTestCase
 
 	public function testColumns()
 	{
+		$this->lockConnection($this->connection);
 		$columns = $this->connection->getPlatform()->getColumns('books');
 		$columns = array_map(function($table) {
 			return (array) $table;

diff --git a/tests/cases/integration/platform.postgres.phpt b/tests/cases/integration/platform.postgres.phpt
@@ -18,6 +18,7 @@ class PlatformPostgresTest extends IntegrationTestCase
 {
 	public function testTables()
 	{
+		$this->lockConnection($this->connection);
 		$tables = $this->connection->getPlatform()->getTables();
 
 		Assert::true(isset($tables["public.books"]));
@@ -37,6 +38,7 @@ class PlatformPostgresTest extends IntegrationTestCase
 
 	public function testColumns()
 	{
+		$this->lockConnection($this->connection);
 		$columns = $this->connection->getPlatform()->getColumns('books');
 		$columns = \array_map(function ($column) { return (array) $column; }, $columns);
 
@@ -191,6 +193,7 @@ class PlatformPostgresTest extends IntegrationTestCase
 
 	public function testForeignKeys()
 	{
+		$this->lockConnection($this->connection);
 		$keys = $this->connection->getPlatform()->getForeignKeys('books');
 		$keys = \array_map(function ($key) { return (array) $key; }, $keys);
 

diff --git a/tests/cases/integration/platform.sqlserver.phpt b/tests/cases/integration/platform.sqlserver.phpt
@@ -18,6 +18,7 @@ class PlatformSqlServerTest extends IntegrationTestCase
 {
 	public function testTables()
 	{
+		$this->lockConnection($this->connection);
 		$tables = $this->connection->getPlatform()->getTables();
 
 		Assert::true(isset($tables["dbo.books"]));
@@ -37,6 +38,7 @@ class PlatformSqlServerTest extends IntegrationTestCase
 
 	public function testColumns()
 	{
+		$this->lockConnection($this->connection);
 		$columns = $this->connection->getPlatform()->getColumns('books');
 		$columns = \array_map(function ($column) { return (array) $column; }, $columns);