MORPH
SIGN IN SIGN UP
laravel / framework UNCLAIMED

Laravel is a web application framework with expressive, elegant syntax.

34605 0 0 PHP
Normal View History Raw
Add tests to cover any added or modified code
2020-12-08 17:07:49 -05:00
<?php
namespace Illuminate\Tests\Database;
use Illuminate\Database\Connection;
use Illuminate\Database\Query\Processors\PostgresProcessor;
use Illuminate\Database\Schema\Grammars\PostgresGrammar;
use Illuminate\Database\Schema\PostgresBuilder;
use Mockery as m;
use PHPUnit\Framework\TestCase;
class DatabasePostgresBuilderTest extends TestCase
{
[9.x] Adds support for Parallel Testing (#35778) * Adds parallel testing * Fixes "Class MysqlBuilder" not found * Updates wording when drivers do not support create/drop databases * Avoids "getenv" and "putenv" in favour of $_SERVER * Moves "createDatabaseIfNotExists" to "createDatabase" * Adds support to "Postgres" * Fixes CS * Uses a database per process regardless of the used database testing trait * Apply fixes from StyleCI * Removes unused catch * Refactors into service providers * Apply fixes from StyleCI * Removes token resolver from the public API * Renames Parallel Testing register callbacks * Adds "setUp" process and "tearDown" test case * Updates Parallel Testing facade * Suffixes _test_ on storage too * Fixes missing envs for process callbacks * Apply fixes from StyleCI * Ensures "drops" only happen when needed * Apply fixes from StyleCI * Adds support for SQLite file databases * Adds tests for SQLite file databases * Lower case create/drop statements * Adds support for SQL Server * Apply fixes from StyleCI * Adds tests against Parallel Testing callbacks * Adds tests for parallel console output * Apply fixes from StyleCI * Removes unused lines in tests * Calls Parallel Testing setUp callbacks after refresh app only * Cleans folders for parallel testing * Makes test databases persist * Apply fixes from StyleCI * Resolve parallel testing callbacks from container * Apply fixes from StyleCI * Update TestDatabases.php * Update ParallelTesting.php * Renames "refresh-databases" to "recreate-databases" Co-authored-by: Nuno Maduro <nunomaduro@users.noreply.github.com> Co-authored-by: Taylor Otwell <taylor@laravel.com>
2021-01-13 15:20:19 +01:00
public function testCreateDatabase()
{
$connection = m::mock(Connection::class);
[12.x] Fix accessing `Connection` property in `Grammar` classes (#54487) * fix connection references * formatting * formatting * formatting
2025-02-12 20:00:57 +03:30
$grammar = new PostgresGrammar($connection);
[9.x] Adds support for Parallel Testing (#35778) * Adds parallel testing * Fixes "Class MysqlBuilder" not found * Updates wording when drivers do not support create/drop databases * Avoids "getenv" and "putenv" in favour of $_SERVER * Moves "createDatabaseIfNotExists" to "createDatabase" * Adds support to "Postgres" * Fixes CS * Uses a database per process regardless of the used database testing trait * Apply fixes from StyleCI * Removes unused catch * Refactors into service providers * Apply fixes from StyleCI * Removes token resolver from the public API * Renames Parallel Testing register callbacks * Adds "setUp" process and "tearDown" test case * Updates Parallel Testing facade * Suffixes _test_ on storage too * Fixes missing envs for process callbacks * Apply fixes from StyleCI * Ensures "drops" only happen when needed * Apply fixes from StyleCI * Adds support for SQLite file databases * Adds tests for SQLite file databases * Lower case create/drop statements * Adds support for SQL Server * Apply fixes from StyleCI * Adds tests against Parallel Testing callbacks * Adds tests for parallel console output * Apply fixes from StyleCI * Removes unused lines in tests * Calls Parallel Testing setUp callbacks after refresh app only * Cleans folders for parallel testing * Makes test databases persist * Apply fixes from StyleCI * Resolve parallel testing callbacks from container * Apply fixes from StyleCI * Update TestDatabases.php * Update ParallelTesting.php * Renames "refresh-databases" to "recreate-databases" Co-authored-by: Nuno Maduro <nunomaduro@users.noreply.github.com> Co-authored-by: Taylor Otwell <taylor@laravel.com>
2021-01-13 15:20:19 +01:00
$connection->shouldReceive('getConfig')->once()->with('charset')->andReturn('utf8');
$connection->shouldReceive('getSchemaGrammar')->once()->andReturn($grammar);
$connection->shouldReceive('statement')->once()->with(
'create database "my_temporary_database" encoding "utf8"'
)->andReturn(true);
$builder = $this->getBuilder($connection);
$builder->createDatabase('my_temporary_database');
}
public function testDropDatabaseIfExists()
{
$connection = m::mock(Connection::class);
[12.x] Fix accessing `Connection` property in `Grammar` classes (#54487) * fix connection references * formatting * formatting * formatting
2025-02-12 20:00:57 +03:30
$grammar = new PostgresGrammar($connection);
[9.x] Adds support for Parallel Testing (#35778) * Adds parallel testing * Fixes "Class MysqlBuilder" not found * Updates wording when drivers do not support create/drop databases * Avoids "getenv" and "putenv" in favour of $_SERVER * Moves "createDatabaseIfNotExists" to "createDatabase" * Adds support to "Postgres" * Fixes CS * Uses a database per process regardless of the used database testing trait * Apply fixes from StyleCI * Removes unused catch * Refactors into service providers * Apply fixes from StyleCI * Removes token resolver from the public API * Renames Parallel Testing register callbacks * Adds "setUp" process and "tearDown" test case * Updates Parallel Testing facade * Suffixes _test_ on storage too * Fixes missing envs for process callbacks * Apply fixes from StyleCI * Ensures "drops" only happen when needed * Apply fixes from StyleCI * Adds support for SQLite file databases * Adds tests for SQLite file databases * Lower case create/drop statements * Adds support for SQL Server * Apply fixes from StyleCI * Adds tests against Parallel Testing callbacks * Adds tests for parallel console output * Apply fixes from StyleCI * Removes unused lines in tests * Calls Parallel Testing setUp callbacks after refresh app only * Cleans folders for parallel testing * Makes test databases persist * Apply fixes from StyleCI * Resolve parallel testing callbacks from container * Apply fixes from StyleCI * Update TestDatabases.php * Update ParallelTesting.php * Renames "refresh-databases" to "recreate-databases" Co-authored-by: Nuno Maduro <nunomaduro@users.noreply.github.com> Co-authored-by: Taylor Otwell <taylor@laravel.com>
2021-01-13 15:20:19 +01:00
$connection->shouldReceive('getSchemaGrammar')->once()->andReturn($grammar);
$connection->shouldReceive('statement')->once()->with(
'drop database if exists "my_database_a"'
)->andReturn(true);
$builder = $this->getBuilder($connection);
$builder->dropDatabaseIfExists('my_database_a');
}
PostgresBuilder fixes for renamed `config('database.connections.pgsql.search_path')` (#41215) 1. `PostgresBuilder::parseSchemaAndTable()` * Needs a fallback to <= 8.x `config('database.connections.pgsql.schema')` when 9.x renamed `config('database.connections.pgsql.search_path')` is missing. * Remove duplicate `$user` variable `config('database.connections.pgsql.username')` replacement handling already done by `parseSearchPath()`. 2. `PostgresBuilder::getAllTables()` + `getAllViews()` + `parseSchemaAndTable()` Apply the `parseSearchPath()` fixes applied to PostgresConnector: https://github.com/laravel/framework/pull/41088 3. `DatabasePostgresBuilderTest` Add more test cases and use terse method names instead of extensive comments to concisely communicate each case.
2022-02-24 10:00:19 -05:00
public function testHasTableWhenSchemaUnqualifiedAndSearchPathMissing()
Add tests to cover any added or modified code
2020-12-08 17:07:49 -05:00
{
$connection = $this->getConnection();
$connection->shouldReceive('getConfig')->with('search_path')->andReturn(null);
PostgresBuilder fixes for renamed `config('database.connections.pgsql.search_path')` (#41215) 1. `PostgresBuilder::parseSchemaAndTable()` * Needs a fallback to <= 8.x `config('database.connections.pgsql.schema')` when 9.x renamed `config('database.connections.pgsql.search_path')` is missing. * Remove duplicate `$user` variable `config('database.connections.pgsql.username')` replacement handling already done by `parseSearchPath()`. 2. `PostgresBuilder::getAllTables()` + `getAllViews()` + `parseSchemaAndTable()` Apply the `parseSearchPath()` fixes applied to PostgresConnector: https://github.com/laravel/framework/pull/41088 3. `DatabasePostgresBuilderTest` Add more test cases and use terse method names instead of extensive comments to concisely communicate each case.
2022-02-24 10:00:19 -05:00
$connection->shouldReceive('getConfig')->with('schema')->andReturn(null);
Add tests to cover any added or modified code
2020-12-08 17:07:49 -05:00
$grammar = m::mock(PostgresGrammar::class);
$connection->shouldReceive('getSchemaGrammar')->once()->andReturn($grammar);
[11.x] Improve `Schema::hasTable()` performance (#53006) * improve has table performance * fix tests * fix tests * fix tests
2024-10-03 21:18:01 +03:30
$grammar->shouldReceive('compileTableExists')->andReturn('sql');
$connection->shouldReceive('scalar')->with('sql')->andReturn(1);
Add tests to cover any added or modified code
2020-12-08 17:07:49 -05:00
$connection->shouldReceive('getTablePrefix');
$builder = $this->getBuilder($connection);
[11.x] Fix inconsistent database parsing on PostgreSQL (#49148) * fix inconsistent database parsing on PostgreSQL * fix on getIndexes * fix on getForeignKeys
2023-12-13 21:59:01 +03:30
$this->assertTrue($builder->hasTable('foo'));
$this->assertTrue($builder->hasTable('public.foo'));
Add tests to cover any added or modified code
2020-12-08 17:07:49 -05:00
}
PostgresBuilder fixes for renamed `config('database.connections.pgsql.search_path')` (#41215) 1. `PostgresBuilder::parseSchemaAndTable()` * Needs a fallback to <= 8.x `config('database.connections.pgsql.schema')` when 9.x renamed `config('database.connections.pgsql.search_path')` is missing. * Remove duplicate `$user` variable `config('database.connections.pgsql.username')` replacement handling already done by `parseSearchPath()`. 2. `PostgresBuilder::getAllTables()` + `getAllViews()` + `parseSchemaAndTable()` Apply the `parseSearchPath()` fixes applied to PostgresConnector: https://github.com/laravel/framework/pull/41088 3. `DatabasePostgresBuilderTest` Add more test cases and use terse method names instead of extensive comments to concisely communicate each case.
2022-02-24 10:00:19 -05:00
public function testHasTableWhenSchemaUnqualifiedAndSearchPathFilled()
Add tests to cover any added or modified code
2020-12-08 17:07:49 -05:00
{
$connection = $this->getConnection();
$connection->shouldReceive('getConfig')->with('search_path')->andReturn('myapp,public');
$grammar = m::mock(PostgresGrammar::class);
$connection->shouldReceive('getSchemaGrammar')->once()->andReturn($grammar);
[11.x] Improve `Schema::hasTable()` performance (#53006) * improve has table performance * fix tests * fix tests * fix tests
2024-10-03 21:18:01 +03:30
$grammar->shouldReceive('compileTableExists')->andReturn('sql');
$connection->shouldReceive('scalar')->with('sql')->andReturn(1);
Add tests to cover any added or modified code
2020-12-08 17:07:49 -05:00
$connection->shouldReceive('getTablePrefix');
$builder = $this->getBuilder($connection);
[11.x] Fix inconsistent database parsing on PostgreSQL (#49148) * fix inconsistent database parsing on PostgreSQL * fix on getIndexes * fix on getForeignKeys
2023-12-13 21:59:01 +03:30
$this->assertTrue($builder->hasTable('foo'));
$this->assertTrue($builder->hasTable('myapp.foo'));
Add tests to cover any added or modified code
2020-12-08 17:07:49 -05:00
}
PostgresBuilder fixes for renamed `config('database.connections.pgsql.search_path')` (#41215) 1. `PostgresBuilder::parseSchemaAndTable()` * Needs a fallback to <= 8.x `config('database.connections.pgsql.schema')` when 9.x renamed `config('database.connections.pgsql.search_path')` is missing. * Remove duplicate `$user` variable `config('database.connections.pgsql.username')` replacement handling already done by `parseSearchPath()`. 2. `PostgresBuilder::getAllTables()` + `getAllViews()` + `parseSchemaAndTable()` Apply the `parseSearchPath()` fixes applied to PostgresConnector: https://github.com/laravel/framework/pull/41088 3. `DatabasePostgresBuilderTest` Add more test cases and use terse method names instead of extensive comments to concisely communicate each case.
2022-02-24 10:00:19 -05:00
public function testHasTableWhenSchemaUnqualifiedAndSearchPathFallbackFilled()
{
$connection = $this->getConnection();
$connection->shouldReceive('getConfig')->with('search_path')->andReturn(null);
$connection->shouldReceive('getConfig')->with('schema')->andReturn(['myapp', 'public']);
$grammar = m::mock(PostgresGrammar::class);
$connection->shouldReceive('getSchemaGrammar')->once()->andReturn($grammar);
[11.x] Improve `Schema::hasTable()` performance (#53006) * improve has table performance * fix tests * fix tests * fix tests
2024-10-03 21:18:01 +03:30
$grammar->shouldReceive('compileTableExists')->andReturn('sql');
$connection->shouldReceive('scalar')->with('sql')->andReturn(1);
PostgresBuilder fixes for renamed `config('database.connections.pgsql.search_path')` (#41215) 1. `PostgresBuilder::parseSchemaAndTable()` * Needs a fallback to <= 8.x `config('database.connections.pgsql.schema')` when 9.x renamed `config('database.connections.pgsql.search_path')` is missing. * Remove duplicate `$user` variable `config('database.connections.pgsql.username')` replacement handling already done by `parseSearchPath()`. 2. `PostgresBuilder::getAllTables()` + `getAllViews()` + `parseSchemaAndTable()` Apply the `parseSearchPath()` fixes applied to PostgresConnector: https://github.com/laravel/framework/pull/41088 3. `DatabasePostgresBuilderTest` Add more test cases and use terse method names instead of extensive comments to concisely communicate each case.
2022-02-24 10:00:19 -05:00
$connection->shouldReceive('getTablePrefix');
$builder = $this->getBuilder($connection);
[11.x] Fix inconsistent database parsing on PostgreSQL (#49148) * fix inconsistent database parsing on PostgreSQL * fix on getIndexes * fix on getForeignKeys
2023-12-13 21:59:01 +03:30
$this->assertTrue($builder->hasTable('foo'));
$this->assertTrue($builder->hasTable('myapp.foo'));
PostgresBuilder fixes for renamed `config('database.connections.pgsql.search_path')` (#41215) 1. `PostgresBuilder::parseSchemaAndTable()` * Needs a fallback to <= 8.x `config('database.connections.pgsql.schema')` when 9.x renamed `config('database.connections.pgsql.search_path')` is missing. * Remove duplicate `$user` variable `config('database.connections.pgsql.username')` replacement handling already done by `parseSearchPath()`. 2. `PostgresBuilder::getAllTables()` + `getAllViews()` + `parseSchemaAndTable()` Apply the `parseSearchPath()` fixes applied to PostgresConnector: https://github.com/laravel/framework/pull/41088 3. `DatabasePostgresBuilderTest` Add more test cases and use terse method names instead of extensive comments to concisely communicate each case.
2022-02-24 10:00:19 -05:00
}
public function testHasTableWhenSchemaUnqualifiedAndSearchPathIsUserVariable()
Account for special '$user' variable appearing as first schema in search_path (#35567) Given that the default search_path value in a PostgreSQL installation is '"$user", public', or perhaps just '"$user"', as may be the case if hardened against CVE-2018-1058, it is preferable to account for the possibility that an end-user may wish to configure a PostgreSQL database connection in Laravel to mimic said default. Now, if '$user' is the first schema in the search_path, the PostgresBuilder will resolve that schema name to the username defined on the database connection whenever appropriate, e.g., in the hasTable() and getColumnListing() methods.
2020-12-11 09:16:03 -05:00
{
$connection = $this->getConnection();
$connection->shouldReceive('getConfig')->with('username')->andReturn('foouser');
$connection->shouldReceive('getConfig')->with('search_path')->andReturn('$user');
$grammar = m::mock(PostgresGrammar::class);
$connection->shouldReceive('getSchemaGrammar')->once()->andReturn($grammar);
[11.x] Improve `Schema::hasTable()` performance (#53006) * improve has table performance * fix tests * fix tests * fix tests
2024-10-03 21:18:01 +03:30
$grammar->shouldReceive('compileTableExists')->andReturn('sql');
$connection->shouldReceive('scalar')->with('sql')->andReturn(1);
Account for special '$user' variable appearing as first schema in search_path (#35567) Given that the default search_path value in a PostgreSQL installation is '"$user", public', or perhaps just '"$user"', as may be the case if hardened against CVE-2018-1058, it is preferable to account for the possibility that an end-user may wish to configure a PostgreSQL database connection in Laravel to mimic said default. Now, if '$user' is the first schema in the search_path, the PostgresBuilder will resolve that schema name to the username defined on the database connection whenever appropriate, e.g., in the hasTable() and getColumnListing() methods.
2020-12-11 09:16:03 -05:00
$connection->shouldReceive('getTablePrefix');
$builder = $this->getBuilder($connection);
[11.x] Fix inconsistent database parsing on PostgreSQL (#49148) * fix inconsistent database parsing on PostgreSQL * fix on getIndexes * fix on getForeignKeys
2023-12-13 21:59:01 +03:30
$this->assertTrue($builder->hasTable('foo'));
$this->assertTrue($builder->hasTable('foouser.foo'));
Account for special '$user' variable appearing as first schema in search_path (#35567) Given that the default search_path value in a PostgreSQL installation is '"$user", public', or perhaps just '"$user"', as may be the case if hardened against CVE-2018-1058, it is preferable to account for the possibility that an end-user may wish to configure a PostgreSQL database connection in Laravel to mimic said default. Now, if '$user' is the first schema in the search_path, the PostgresBuilder will resolve that schema name to the username defined on the database connection whenever appropriate, e.g., in the hasTable() and getColumnListing() methods.
2020-12-11 09:16:03 -05:00
}
PostgresBuilder fixes for renamed `config('database.connections.pgsql.search_path')` (#41215) 1. `PostgresBuilder::parseSchemaAndTable()` * Needs a fallback to <= 8.x `config('database.connections.pgsql.schema')` when 9.x renamed `config('database.connections.pgsql.search_path')` is missing. * Remove duplicate `$user` variable `config('database.connections.pgsql.username')` replacement handling already done by `parseSearchPath()`. 2. `PostgresBuilder::getAllTables()` + `getAllViews()` + `parseSchemaAndTable()` Apply the `parseSearchPath()` fixes applied to PostgresConnector: https://github.com/laravel/framework/pull/41088 3. `DatabasePostgresBuilderTest` Add more test cases and use terse method names instead of extensive comments to concisely communicate each case.
2022-02-24 10:00:19 -05:00
public function testHasTableWhenSchemaQualifiedAndSearchPathMismatches()
Add tests to cover any added or modified code
2020-12-08 17:07:49 -05:00
{
$connection = $this->getConnection();
$connection->shouldReceive('getConfig')->with('search_path')->andReturn('public');
$grammar = m::mock(PostgresGrammar::class);
$connection->shouldReceive('getSchemaGrammar')->once()->andReturn($grammar);
[11.x] Improve `Schema::hasTable()` performance (#53006) * improve has table performance * fix tests * fix tests * fix tests
2024-10-03 21:18:01 +03:30
$grammar->shouldReceive('compileTableExists')->andReturn('sql');
$connection->shouldReceive('scalar')->with('sql')->andReturn(1);
Add tests to cover any added or modified code
2020-12-08 17:07:49 -05:00
$connection->shouldReceive('getTablePrefix');
$builder = $this->getBuilder($connection);
[11.x] Fix inconsistent database parsing on PostgreSQL (#49148) * fix inconsistent database parsing on PostgreSQL * fix on getIndexes * fix on getForeignKeys
2023-12-13 21:59:01 +03:30
$this->assertTrue($builder->hasTable('myapp.foo'));
Add tests to cover any added or modified code
2020-12-08 17:07:49 -05:00
}
PostgresBuilder fixes for renamed `config('database.connections.pgsql.search_path')` (#41215) 1. `PostgresBuilder::parseSchemaAndTable()` * Needs a fallback to <= 8.x `config('database.connections.pgsql.schema')` when 9.x renamed `config('database.connections.pgsql.search_path')` is missing. * Remove duplicate `$user` variable `config('database.connections.pgsql.username')` replacement handling already done by `parseSearchPath()`. 2. `PostgresBuilder::getAllTables()` + `getAllViews()` + `parseSchemaAndTable()` Apply the `parseSearchPath()` fixes applied to PostgresConnector: https://github.com/laravel/framework/pull/41088 3. `DatabasePostgresBuilderTest` Add more test cases and use terse method names instead of extensive comments to concisely communicate each case.
2022-02-24 10:00:19 -05:00
public function testHasTableWhenDatabaseAndSchemaQualifiedAndSearchPathMismatches()
Add tests to cover any added or modified code
2020-12-08 17:07:49 -05:00
{
[11.x] Fix inconsistent database parsing on PostgreSQL (#49148) * fix inconsistent database parsing on PostgreSQL * fix on getIndexes * fix on getForeignKeys
2023-12-13 21:59:01 +03:30
$this->expectException(\InvalidArgumentException::class);
Add tests to cover any added or modified code
2020-12-08 17:07:49 -05:00
$connection = $this->getConnection();
$grammar = m::mock(PostgresGrammar::class);
$connection->shouldReceive('getSchemaGrammar')->once()->andReturn($grammar);
$builder = $this->getBuilder($connection);
$builder->hasTable('mydatabase.myapp.foo');
}
PostgresBuilder fixes for renamed `config('database.connections.pgsql.search_path')` (#41215) 1. `PostgresBuilder::parseSchemaAndTable()` * Needs a fallback to <= 8.x `config('database.connections.pgsql.schema')` when 9.x renamed `config('database.connections.pgsql.search_path')` is missing. * Remove duplicate `$user` variable `config('database.connections.pgsql.username')` replacement handling already done by `parseSearchPath()`. 2. `PostgresBuilder::getAllTables()` + `getAllViews()` + `parseSchemaAndTable()` Apply the `parseSearchPath()` fixes applied to PostgresConnector: https://github.com/laravel/framework/pull/41088 3. `DatabasePostgresBuilderTest` Add more test cases and use terse method names instead of extensive comments to concisely communicate each case.
2022-02-24 10:00:19 -05:00
public function testGetColumnListingWhenSchemaUnqualifiedAndSearchPathMissing()
Add tests to cover any added or modified code
2020-12-08 17:07:49 -05:00
{
$connection = $this->getConnection();
$connection->shouldReceive('getConfig')->with('search_path')->andReturn(null);
PostgresBuilder fixes for renamed `config('database.connections.pgsql.search_path')` (#41215) 1. `PostgresBuilder::parseSchemaAndTable()` * Needs a fallback to <= 8.x `config('database.connections.pgsql.schema')` when 9.x renamed `config('database.connections.pgsql.search_path')` is missing. * Remove duplicate `$user` variable `config('database.connections.pgsql.username')` replacement handling already done by `parseSearchPath()`. 2. `PostgresBuilder::getAllTables()` + `getAllViews()` + `parseSchemaAndTable()` Apply the `parseSearchPath()` fixes applied to PostgresConnector: https://github.com/laravel/framework/pull/41088 3. `DatabasePostgresBuilderTest` Add more test cases and use terse method names instead of extensive comments to concisely communicate each case.
2022-02-24 10:00:19 -05:00
$connection->shouldReceive('getConfig')->with('schema')->andReturn(null);
Add tests to cover any added or modified code
2020-12-08 17:07:49 -05:00
$grammar = m::mock(PostgresGrammar::class);
$connection->shouldReceive('getSchemaGrammar')->once()->andReturn($grammar);
[12.x] Enhance multi-database support (#54274) * enhance multi-schema support * formatting * formatting * formatting * formatting * formatting * formatting * Update SchemaBuilderSchemaNameTest.php * formatting * formatting --------- Co-authored-by: Taylor Otwell <taylor@laravel.com>
2025-01-23 02:37:00 +03:30
$grammar->shouldReceive('compileColumns')->with(null, 'foo')->andReturn('sql');
[11.x] Fix inconsistent database parsing on PostgreSQL (#49148) * fix inconsistent database parsing on PostgreSQL * fix on getIndexes * fix on getForeignKeys
2023-12-13 21:59:01 +03:30
$connection->shouldReceive('selectFromWriteConnection')->with('sql')->andReturn([['name' => 'some_column']]);
Add tests to cover any added or modified code
2020-12-08 17:07:49 -05:00
$connection->shouldReceive('getTablePrefix');
$processor = m::mock(PostgresProcessor::class);
$connection->shouldReceive('getPostProcessor')->andReturn($processor);
[10.x] Add support for getting native columns' attributes (#48357) * add support for getting native columns' attributes * fix test and formatting * deprecate `processColumnListing` * use table-valued pragma on SQLite * fix a typo * revert changes on a deprecated function * formatting --------- Co-authored-by: Taylor Otwell <taylor@laravel.com>
2023-10-26 21:50:57 +03:30
$processor->shouldReceive('processColumns')->andReturn([['name' => 'some_column']]);
Add tests to cover any added or modified code
2020-12-08 17:07:49 -05:00
$builder = $this->getBuilder($connection);
$builder->getColumnListing('foo');
}
PostgresBuilder fixes for renamed `config('database.connections.pgsql.search_path')` (#41215) 1. `PostgresBuilder::parseSchemaAndTable()` * Needs a fallback to <= 8.x `config('database.connections.pgsql.schema')` when 9.x renamed `config('database.connections.pgsql.search_path')` is missing. * Remove duplicate `$user` variable `config('database.connections.pgsql.username')` replacement handling already done by `parseSearchPath()`. 2. `PostgresBuilder::getAllTables()` + `getAllViews()` + `parseSchemaAndTable()` Apply the `parseSearchPath()` fixes applied to PostgresConnector: https://github.com/laravel/framework/pull/41088 3. `DatabasePostgresBuilderTest` Add more test cases and use terse method names instead of extensive comments to concisely communicate each case.
2022-02-24 10:00:19 -05:00
public function testGetColumnListingWhenSchemaUnqualifiedAndSearchPathFilled()
Add tests to cover any added or modified code
2020-12-08 17:07:49 -05:00
{
$connection = $this->getConnection();
$connection->shouldReceive('getConfig')->with('search_path')->andReturn('myapp,public');
$grammar = m::mock(PostgresGrammar::class);
$connection->shouldReceive('getSchemaGrammar')->once()->andReturn($grammar);
[12.x] Enhance multi-database support (#54274) * enhance multi-schema support * formatting * formatting * formatting * formatting * formatting * formatting * Update SchemaBuilderSchemaNameTest.php * formatting * formatting --------- Co-authored-by: Taylor Otwell <taylor@laravel.com>
2025-01-23 02:37:00 +03:30
$grammar->shouldReceive('compileColumns')->with(null, 'foo')->andReturn('sql');
[11.x] Fix inconsistent database parsing on PostgreSQL (#49148) * fix inconsistent database parsing on PostgreSQL * fix on getIndexes * fix on getForeignKeys
2023-12-13 21:59:01 +03:30
$connection->shouldReceive('selectFromWriteConnection')->with('sql')->andReturn([['name' => 'some_column']]);
Add tests to cover any added or modified code
2020-12-08 17:07:49 -05:00
$connection->shouldReceive('getTablePrefix');
$processor = m::mock(PostgresProcessor::class);
$connection->shouldReceive('getPostProcessor')->andReturn($processor);
[10.x] Add support for getting native columns' attributes (#48357) * add support for getting native columns' attributes * fix test and formatting * deprecate `processColumnListing` * use table-valued pragma on SQLite * fix a typo * revert changes on a deprecated function * formatting --------- Co-authored-by: Taylor Otwell <taylor@laravel.com>
2023-10-26 21:50:57 +03:30
$processor->shouldReceive('processColumns')->andReturn([['name' => 'some_column']]);
Add tests to cover any added or modified code
2020-12-08 17:07:49 -05:00
$builder = $this->getBuilder($connection);
$builder->getColumnListing('foo');
}
PostgresBuilder fixes for renamed `config('database.connections.pgsql.search_path')` (#41215) 1. `PostgresBuilder::parseSchemaAndTable()` * Needs a fallback to <= 8.x `config('database.connections.pgsql.schema')` when 9.x renamed `config('database.connections.pgsql.search_path')` is missing. * Remove duplicate `$user` variable `config('database.connections.pgsql.username')` replacement handling already done by `parseSearchPath()`. 2. `PostgresBuilder::getAllTables()` + `getAllViews()` + `parseSchemaAndTable()` Apply the `parseSearchPath()` fixes applied to PostgresConnector: https://github.com/laravel/framework/pull/41088 3. `DatabasePostgresBuilderTest` Add more test cases and use terse method names instead of extensive comments to concisely communicate each case.
2022-02-24 10:00:19 -05:00
public function testGetColumnListingWhenSchemaUnqualifiedAndSearchPathIsUserVariable()
Account for special '$user' variable appearing as first schema in search_path (#35567) Given that the default search_path value in a PostgreSQL installation is '"$user", public', or perhaps just '"$user"', as may be the case if hardened against CVE-2018-1058, it is preferable to account for the possibility that an end-user may wish to configure a PostgreSQL database connection in Laravel to mimic said default. Now, if '$user' is the first schema in the search_path, the PostgresBuilder will resolve that schema name to the username defined on the database connection whenever appropriate, e.g., in the hasTable() and getColumnListing() methods.
2020-12-11 09:16:03 -05:00
{
$connection = $this->getConnection();
$connection->shouldReceive('getConfig')->with('username')->andReturn('foouser');
$connection->shouldReceive('getConfig')->with('search_path')->andReturn('$user');
$grammar = m::mock(PostgresGrammar::class);
$connection->shouldReceive('getSchemaGrammar')->once()->andReturn($grammar);
[12.x] Enhance multi-database support (#54274) * enhance multi-schema support * formatting * formatting * formatting * formatting * formatting * formatting * Update SchemaBuilderSchemaNameTest.php * formatting * formatting --------- Co-authored-by: Taylor Otwell <taylor@laravel.com>
2025-01-23 02:37:00 +03:30
$grammar->shouldReceive('compileColumns')->with(null, 'foo')->andReturn('sql');
[11.x] Fix inconsistent database parsing on PostgreSQL (#49148) * fix inconsistent database parsing on PostgreSQL * fix on getIndexes * fix on getForeignKeys
2023-12-13 21:59:01 +03:30
$connection->shouldReceive('selectFromWriteConnection')->with('sql')->andReturn([['name' => 'some_column']]);
Account for special '$user' variable appearing as first schema in search_path (#35567) Given that the default search_path value in a PostgreSQL installation is '"$user", public', or perhaps just '"$user"', as may be the case if hardened against CVE-2018-1058, it is preferable to account for the possibility that an end-user may wish to configure a PostgreSQL database connection in Laravel to mimic said default. Now, if '$user' is the first schema in the search_path, the PostgresBuilder will resolve that schema name to the username defined on the database connection whenever appropriate, e.g., in the hasTable() and getColumnListing() methods.
2020-12-11 09:16:03 -05:00
$connection->shouldReceive('getTablePrefix');
$processor = m::mock(PostgresProcessor::class);
$connection->shouldReceive('getPostProcessor')->andReturn($processor);
[10.x] Add support for getting native columns' attributes (#48357) * add support for getting native columns' attributes * fix test and formatting * deprecate `processColumnListing` * use table-valued pragma on SQLite * fix a typo * revert changes on a deprecated function * formatting --------- Co-authored-by: Taylor Otwell <taylor@laravel.com>
2023-10-26 21:50:57 +03:30
$processor->shouldReceive('processColumns')->andReturn([['name' => 'some_column']]);
Account for special '$user' variable appearing as first schema in search_path (#35567) Given that the default search_path value in a PostgreSQL installation is '"$user", public', or perhaps just '"$user"', as may be the case if hardened against CVE-2018-1058, it is preferable to account for the possibility that an end-user may wish to configure a PostgreSQL database connection in Laravel to mimic said default. Now, if '$user' is the first schema in the search_path, the PostgresBuilder will resolve that schema name to the username defined on the database connection whenever appropriate, e.g., in the hasTable() and getColumnListing() methods.
2020-12-11 09:16:03 -05:00
$builder = $this->getBuilder($connection);
$builder->getColumnListing('foo');
}
PostgresBuilder fixes for renamed `config('database.connections.pgsql.search_path')` (#41215) 1. `PostgresBuilder::parseSchemaAndTable()` * Needs a fallback to <= 8.x `config('database.connections.pgsql.schema')` when 9.x renamed `config('database.connections.pgsql.search_path')` is missing. * Remove duplicate `$user` variable `config('database.connections.pgsql.username')` replacement handling already done by `parseSearchPath()`. 2. `PostgresBuilder::getAllTables()` + `getAllViews()` + `parseSchemaAndTable()` Apply the `parseSearchPath()` fixes applied to PostgresConnector: https://github.com/laravel/framework/pull/41088 3. `DatabasePostgresBuilderTest` Add more test cases and use terse method names instead of extensive comments to concisely communicate each case.
2022-02-24 10:00:19 -05:00
public function testGetColumnListingWhenSchemaQualifiedAndSearchPathMismatches()
Add tests to cover any added or modified code
2020-12-08 17:07:49 -05:00
{
$connection = $this->getConnection();
$connection->shouldReceive('getConfig')->with('search_path')->andReturn('public');
$grammar = m::mock(PostgresGrammar::class);
$connection->shouldReceive('getSchemaGrammar')->once()->andReturn($grammar);
[11.x] Fix inconsistent database parsing on PostgreSQL (#49148) * fix inconsistent database parsing on PostgreSQL * fix on getIndexes * fix on getForeignKeys
2023-12-13 21:59:01 +03:30
$grammar->shouldReceive('compileColumns')->with('myapp', 'foo')->andReturn('sql');
$connection->shouldReceive('selectFromWriteConnection')->with('sql')->andReturn([['name' => 'some_column']]);
Add tests to cover any added or modified code
2020-12-08 17:07:49 -05:00
$connection->shouldReceive('getTablePrefix');
$processor = m::mock(PostgresProcessor::class);
$connection->shouldReceive('getPostProcessor')->andReturn($processor);
[10.x] Add support for getting native columns' attributes (#48357) * add support for getting native columns' attributes * fix test and formatting * deprecate `processColumnListing` * use table-valued pragma on SQLite * fix a typo * revert changes on a deprecated function * formatting --------- Co-authored-by: Taylor Otwell <taylor@laravel.com>
2023-10-26 21:50:57 +03:30
$processor->shouldReceive('processColumns')->andReturn([['name' => 'some_column']]);
Add tests to cover any added or modified code
2020-12-08 17:07:49 -05:00
$builder = $this->getBuilder($connection);
$builder->getColumnListing('myapp.foo');
}
PostgresBuilder fixes for renamed `config('database.connections.pgsql.search_path')` (#41215) 1. `PostgresBuilder::parseSchemaAndTable()` * Needs a fallback to <= 8.x `config('database.connections.pgsql.schema')` when 9.x renamed `config('database.connections.pgsql.search_path')` is missing. * Remove duplicate `$user` variable `config('database.connections.pgsql.username')` replacement handling already done by `parseSearchPath()`. 2. `PostgresBuilder::getAllTables()` + `getAllViews()` + `parseSchemaAndTable()` Apply the `parseSearchPath()` fixes applied to PostgresConnector: https://github.com/laravel/framework/pull/41088 3. `DatabasePostgresBuilderTest` Add more test cases and use terse method names instead of extensive comments to concisely communicate each case.
2022-02-24 10:00:19 -05:00
public function testGetColumnWhenDatabaseAndSchemaQualifiedAndSearchPathMismatches()
Add tests to cover any added or modified code
2020-12-08 17:07:49 -05:00
{
[11.x] Fix inconsistent database parsing on PostgreSQL (#49148) * fix inconsistent database parsing on PostgreSQL * fix on getIndexes * fix on getForeignKeys
2023-12-13 21:59:01 +03:30
$this->expectException(\InvalidArgumentException::class);
Add tests to cover any added or modified code
2020-12-08 17:07:49 -05:00
$connection = $this->getConnection();
$connection->shouldReceive('getConfig')->with('search_path')->andReturn('public');
$grammar = m::mock(PostgresGrammar::class);
$connection->shouldReceive('getSchemaGrammar')->once()->andReturn($grammar);
$builder = $this->getBuilder($connection);
$builder->getColumnListing('mydatabase.myapp.foo');
}
PostgresBuilder fixes for renamed `config('database.connections.pgsql.search_path')` (#41215) 1. `PostgresBuilder::parseSchemaAndTable()` * Needs a fallback to <= 8.x `config('database.connections.pgsql.schema')` when 9.x renamed `config('database.connections.pgsql.search_path')` is missing. * Remove duplicate `$user` variable `config('database.connections.pgsql.username')` replacement handling already done by `parseSearchPath()`. 2. `PostgresBuilder::getAllTables()` + `getAllViews()` + `parseSchemaAndTable()` Apply the `parseSearchPath()` fixes applied to PostgresConnector: https://github.com/laravel/framework/pull/41088 3. `DatabasePostgresBuilderTest` Add more test cases and use terse method names instead of extensive comments to concisely communicate each case.
2022-02-24 10:00:19 -05:00
public function testDropAllTablesWhenSearchPathIsString()
[9.x] Fix several lingering issues with PostgreSQL search_path handling (#35588) * Fix issue with special $user search path variable not being resolved For any scenario in which PostgreSQL's internal "information_schema" is queried using a schema/schemas derived from the search_path configured on the connection, it is necessary to resolve PostgreSQL's special "$user" variable to the actual username set on the connection. While this resolution is performed correctly in the related parseSchemaAndTable() method, it was not being performed here, prior to this change. * Update variable name to reflect what it actually represents * Add new tests to cover search_path behavior when dropping all tables
2020-12-12 22:19:37 -05:00
{
$connection = $this->getConnection();
$connection->shouldReceive('getConfig')->with('search_path')->andReturn('public');
$connection->shouldReceive('getConfig')->with('dont_drop')->andReturn(['foo']);
$grammar = m::mock(PostgresGrammar::class);
[10.x] Get tables and views info (#49020) * add getTables and getViews * fix tests * fix a typo * add more integration tests * add more integration tests * remove redundant schema on mysql * formatting --------- Co-authored-by: Taylor Otwell <taylor@laravel.com>
2023-11-27 19:47:31 +03:30
$processor = m::mock(PostgresProcessor::class);
[9.x] Fix several lingering issues with PostgreSQL search_path handling (#35588) * Fix issue with special $user search path variable not being resolved For any scenario in which PostgreSQL's internal "information_schema" is queried using a schema/schemas derived from the search_path configured on the connection, it is necessary to resolve PostgreSQL's special "$user" variable to the actual username set on the connection. While this resolution is performed correctly in the related parseSchemaAndTable() method, it was not being performed here, prior to this change. * Update variable name to reflect what it actually represents * Add new tests to cover search_path behavior when dropping all tables
2020-12-12 22:19:37 -05:00
$connection->shouldReceive('getSchemaGrammar')->once()->andReturn($grammar);
[10.x] Get tables and views info (#49020) * add getTables and getViews * fix tests * fix a typo * add more integration tests * add more integration tests * remove redundant schema on mysql * formatting --------- Co-authored-by: Taylor Otwell <taylor@laravel.com>
2023-11-27 19:47:31 +03:30
$connection->shouldReceive('getPostProcessor')->andReturn($processor);
$grammar->shouldReceive('compileTables')->andReturn('sql');
[12.x] Enhance multi-database support (#54274) * enhance multi-schema support * formatting * formatting * formatting * formatting * formatting * formatting * Update SchemaBuilderSchemaNameTest.php * formatting * formatting --------- Co-authored-by: Taylor Otwell <taylor@laravel.com>
2025-01-23 02:37:00 +03:30
$processor->shouldReceive('processTables')->once()->andReturn([['name' => 'users', 'schema' => 'public', 'schema_qualified_name' => 'public.users']]);
$connection->shouldReceive('selectFromWriteConnection')->with('sql')->andReturn([['name' => 'users', 'schema' => 'public', 'schema_qualified_name' => 'public.users']]);
[10.x] Get tables and views info (#49020) * add getTables and getViews * fix tests * fix a typo * add more integration tests * add more integration tests * remove redundant schema on mysql * formatting --------- Co-authored-by: Taylor Otwell <taylor@laravel.com>
2023-11-27 19:47:31 +03:30
$grammar->shouldReceive('compileDropAllTables')->with(['public.users'])->andReturn('drop table "public"."users" cascade');
[9.x] Fix Postgres driver not dropping all tables & views (#41701) * formatting array_map() without an array reference reads better than array_walk(). * Fix Postgres driver not dropping all tables & views Drop queries don't qualify names with their search path so only the first object found is dropped. e.g., if a "migrations" table exists in two schemas, Schema::dropAllTables() leaves one of those tables. Start schema-qualifying these names to ensure each object is removed by command 'php artisan db:wipe'. Co-authored-by: Ben Johnson <ben@indiehd.com>
2022-03-28 10:15:39 -04:00
$connection->shouldReceive('statement')->with('drop table "public"."users" cascade');
[9.x] Fix several lingering issues with PostgreSQL search_path handling (#35588) * Fix issue with special $user search path variable not being resolved For any scenario in which PostgreSQL's internal "information_schema" is queried using a schema/schemas derived from the search_path configured on the connection, it is necessary to resolve PostgreSQL's special "$user" variable to the actual username set on the connection. While this resolution is performed correctly in the related parseSchemaAndTable() method, it was not being performed here, prior to this change. * Update variable name to reflect what it actually represents * Add new tests to cover search_path behavior when dropping all tables
2020-12-12 22:19:37 -05:00
$builder = $this->getBuilder($connection);
$builder->dropAllTables();
}
PostgresBuilder fixes for renamed `config('database.connections.pgsql.search_path')` (#41215) 1. `PostgresBuilder::parseSchemaAndTable()` * Needs a fallback to <= 8.x `config('database.connections.pgsql.schema')` when 9.x renamed `config('database.connections.pgsql.search_path')` is missing. * Remove duplicate `$user` variable `config('database.connections.pgsql.username')` replacement handling already done by `parseSearchPath()`. 2. `PostgresBuilder::getAllTables()` + `getAllViews()` + `parseSchemaAndTable()` Apply the `parseSearchPath()` fixes applied to PostgresConnector: https://github.com/laravel/framework/pull/41088 3. `DatabasePostgresBuilderTest` Add more test cases and use terse method names instead of extensive comments to concisely communicate each case.
2022-02-24 10:00:19 -05:00
public function testDropAllTablesWhenSearchPathIsStringOfMany()
{
$connection = $this->getConnection();
$connection->shouldReceive('getConfig')->with('username')->andReturn('foouser');
$connection->shouldReceive('getConfig')->with('search_path')->andReturn('"$user", public, foo_bar-Baz.Áüõß');
$connection->shouldReceive('getConfig')->with('dont_drop')->andReturn(['foo']);
$grammar = m::mock(PostgresGrammar::class);
[10.x] Get tables and views info (#49020) * add getTables and getViews * fix tests * fix a typo * add more integration tests * add more integration tests * remove redundant schema on mysql * formatting --------- Co-authored-by: Taylor Otwell <taylor@laravel.com>
2023-11-27 19:47:31 +03:30
$processor = m::mock(PostgresProcessor::class);
PostgresBuilder fixes for renamed `config('database.connections.pgsql.search_path')` (#41215) 1. `PostgresBuilder::parseSchemaAndTable()` * Needs a fallback to <= 8.x `config('database.connections.pgsql.schema')` when 9.x renamed `config('database.connections.pgsql.search_path')` is missing. * Remove duplicate `$user` variable `config('database.connections.pgsql.username')` replacement handling already done by `parseSearchPath()`. 2. `PostgresBuilder::getAllTables()` + `getAllViews()` + `parseSchemaAndTable()` Apply the `parseSearchPath()` fixes applied to PostgresConnector: https://github.com/laravel/framework/pull/41088 3. `DatabasePostgresBuilderTest` Add more test cases and use terse method names instead of extensive comments to concisely communicate each case.
2022-02-24 10:00:19 -05:00
$connection->shouldReceive('getSchemaGrammar')->once()->andReturn($grammar);
[10.x] Get tables and views info (#49020) * add getTables and getViews * fix tests * fix a typo * add more integration tests * add more integration tests * remove redundant schema on mysql * formatting --------- Co-authored-by: Taylor Otwell <taylor@laravel.com>
2023-11-27 19:47:31 +03:30
$connection->shouldReceive('getPostProcessor')->andReturn($processor);
[12.x] Enhance multi-database support (#54274) * enhance multi-schema support * formatting * formatting * formatting * formatting * formatting * formatting * Update SchemaBuilderSchemaNameTest.php * formatting * formatting --------- Co-authored-by: Taylor Otwell <taylor@laravel.com>
2025-01-23 02:37:00 +03:30
$processor->shouldReceive('processTables')->once()->andReturn([['name' => 'users', 'schema' => 'foouser', 'schema_qualified_name' => 'foouser.users']]);
[10.x] Get tables and views info (#49020) * add getTables and getViews * fix tests * fix a typo * add more integration tests * add more integration tests * remove redundant schema on mysql * formatting --------- Co-authored-by: Taylor Otwell <taylor@laravel.com>
2023-11-27 19:47:31 +03:30
$grammar->shouldReceive('compileTables')->andReturn('sql');
[12.x] Enhance multi-database support (#54274) * enhance multi-schema support * formatting * formatting * formatting * formatting * formatting * formatting * Update SchemaBuilderSchemaNameTest.php * formatting * formatting --------- Co-authored-by: Taylor Otwell <taylor@laravel.com>
2025-01-23 02:37:00 +03:30
$connection->shouldReceive('selectFromWriteConnection')->with('sql')->andReturn([['name' => 'users', 'schema' => 'foouser', 'schema_qualified_name' => 'foouser.users']]);
[10.x] Get tables and views info (#49020) * add getTables and getViews * fix tests * fix a typo * add more integration tests * add more integration tests * remove redundant schema on mysql * formatting --------- Co-authored-by: Taylor Otwell <taylor@laravel.com>
2023-11-27 19:47:31 +03:30
$grammar->shouldReceive('compileDropAllTables')->with(['foouser.users'])->andReturn('drop table "foouser"."users" cascade');
[9.x] Fix Postgres driver not dropping all tables & views (#41701) * formatting array_map() without an array reference reads better than array_walk(). * Fix Postgres driver not dropping all tables & views Drop queries don't qualify names with their search path so only the first object found is dropped. e.g., if a "migrations" table exists in two schemas, Schema::dropAllTables() leaves one of those tables. Start schema-qualifying these names to ensure each object is removed by command 'php artisan db:wipe'. Co-authored-by: Ben Johnson <ben@indiehd.com>
2022-03-28 10:15:39 -04:00
$connection->shouldReceive('statement')->with('drop table "foouser"."users" cascade');
PostgresBuilder fixes for renamed `config('database.connections.pgsql.search_path')` (#41215) 1. `PostgresBuilder::parseSchemaAndTable()` * Needs a fallback to <= 8.x `config('database.connections.pgsql.schema')` when 9.x renamed `config('database.connections.pgsql.search_path')` is missing. * Remove duplicate `$user` variable `config('database.connections.pgsql.username')` replacement handling already done by `parseSearchPath()`. 2. `PostgresBuilder::getAllTables()` + `getAllViews()` + `parseSchemaAndTable()` Apply the `parseSearchPath()` fixes applied to PostgresConnector: https://github.com/laravel/framework/pull/41088 3. `DatabasePostgresBuilderTest` Add more test cases and use terse method names instead of extensive comments to concisely communicate each case.
2022-02-24 10:00:19 -05:00
$builder = $this->getBuilder($connection);
$builder->dropAllTables();
}
public function testDropAllTablesWhenSearchPathIsArrayOfMany()
[9.x] Fix several lingering issues with PostgreSQL search_path handling (#35588) * Fix issue with special $user search path variable not being resolved For any scenario in which PostgreSQL's internal "information_schema" is queried using a schema/schemas derived from the search_path configured on the connection, it is necessary to resolve PostgreSQL's special "$user" variable to the actual username set on the connection. While this resolution is performed correctly in the related parseSchemaAndTable() method, it was not being performed here, prior to this change. * Update variable name to reflect what it actually represents * Add new tests to cover search_path behavior when dropping all tables
2020-12-12 22:19:37 -05:00
{
$connection = $this->getConnection();
$connection->shouldReceive('getConfig')->with('username')->andReturn('foouser');
PostgresBuilder fixes for renamed `config('database.connections.pgsql.search_path')` (#41215) 1. `PostgresBuilder::parseSchemaAndTable()` * Needs a fallback to <= 8.x `config('database.connections.pgsql.schema')` when 9.x renamed `config('database.connections.pgsql.search_path')` is missing. * Remove duplicate `$user` variable `config('database.connections.pgsql.username')` replacement handling already done by `parseSearchPath()`. 2. `PostgresBuilder::getAllTables()` + `getAllViews()` + `parseSchemaAndTable()` Apply the `parseSearchPath()` fixes applied to PostgresConnector: https://github.com/laravel/framework/pull/41088 3. `DatabasePostgresBuilderTest` Add more test cases and use terse method names instead of extensive comments to concisely communicate each case.
2022-02-24 10:00:19 -05:00
$connection->shouldReceive('getConfig')->with('search_path')->andReturn([
'$user',
'"dev"',
"'test'",
'spaced schema',
]);
[9.x] Fix several lingering issues with PostgreSQL search_path handling (#35588) * Fix issue with special $user search path variable not being resolved For any scenario in which PostgreSQL's internal "information_schema" is queried using a schema/schemas derived from the search_path configured on the connection, it is necessary to resolve PostgreSQL's special "$user" variable to the actual username set on the connection. While this resolution is performed correctly in the related parseSchemaAndTable() method, it was not being performed here, prior to this change. * Update variable name to reflect what it actually represents * Add new tests to cover search_path behavior when dropping all tables
2020-12-12 22:19:37 -05:00
$connection->shouldReceive('getConfig')->with('dont_drop')->andReturn(['foo']);
$grammar = m::mock(PostgresGrammar::class);
[10.x] Get tables and views info (#49020) * add getTables and getViews * fix tests * fix a typo * add more integration tests * add more integration tests * remove redundant schema on mysql * formatting --------- Co-authored-by: Taylor Otwell <taylor@laravel.com>
2023-11-27 19:47:31 +03:30
$processor = m::mock(PostgresProcessor::class);
[9.x] Fix several lingering issues with PostgreSQL search_path handling (#35588) * Fix issue with special $user search path variable not being resolved For any scenario in which PostgreSQL's internal "information_schema" is queried using a schema/schemas derived from the search_path configured on the connection, it is necessary to resolve PostgreSQL's special "$user" variable to the actual username set on the connection. While this resolution is performed correctly in the related parseSchemaAndTable() method, it was not being performed here, prior to this change. * Update variable name to reflect what it actually represents * Add new tests to cover search_path behavior when dropping all tables
2020-12-12 22:19:37 -05:00
$connection->shouldReceive('getSchemaGrammar')->once()->andReturn($grammar);
[10.x] Get tables and views info (#49020) * add getTables and getViews * fix tests * fix a typo * add more integration tests * add more integration tests * remove redundant schema on mysql * formatting --------- Co-authored-by: Taylor Otwell <taylor@laravel.com>
2023-11-27 19:47:31 +03:30
$connection->shouldReceive('getPostProcessor')->andReturn($processor);
[12.x] Enhance multi-database support (#54274) * enhance multi-schema support * formatting * formatting * formatting * formatting * formatting * formatting * Update SchemaBuilderSchemaNameTest.php * formatting * formatting --------- Co-authored-by: Taylor Otwell <taylor@laravel.com>
2025-01-23 02:37:00 +03:30
$processor->shouldReceive('processTables')->once()->andReturn([['name' => 'users', 'schema' => 'foouser', 'schema_qualified_name' => 'foouser.users']]);
[10.x] Get tables and views info (#49020) * add getTables and getViews * fix tests * fix a typo * add more integration tests * add more integration tests * remove redundant schema on mysql * formatting --------- Co-authored-by: Taylor Otwell <taylor@laravel.com>
2023-11-27 19:47:31 +03:30
$grammar->shouldReceive('compileTables')->andReturn('sql');
[12.x] Enhance multi-database support (#54274) * enhance multi-schema support * formatting * formatting * formatting * formatting * formatting * formatting * Update SchemaBuilderSchemaNameTest.php * formatting * formatting --------- Co-authored-by: Taylor Otwell <taylor@laravel.com>
2025-01-23 02:37:00 +03:30
$connection->shouldReceive('selectFromWriteConnection')->with('sql')->andReturn([['name' => 'users', 'schema' => 'foouser', 'schema_qualified_name' => 'foouser.users']]);
[10.x] Get tables and views info (#49020) * add getTables and getViews * fix tests * fix a typo * add more integration tests * add more integration tests * remove redundant schema on mysql * formatting --------- Co-authored-by: Taylor Otwell <taylor@laravel.com>
2023-11-27 19:47:31 +03:30
$grammar->shouldReceive('compileDropAllTables')->with(['foouser.users'])->andReturn('drop table "foouser"."users" cascade');
[9.x] Fix Postgres driver not dropping all tables & views (#41701) * formatting array_map() without an array reference reads better than array_walk(). * Fix Postgres driver not dropping all tables & views Drop queries don't qualify names with their search path so only the first object found is dropped. e.g., if a "migrations" table exists in two schemas, Schema::dropAllTables() leaves one of those tables. Start schema-qualifying these names to ensure each object is removed by command 'php artisan db:wipe'. Co-authored-by: Ben Johnson <ben@indiehd.com>
2022-03-28 10:15:39 -04:00
$connection->shouldReceive('statement')->with('drop table "foouser"."users" cascade');
[9.x] Fix several lingering issues with PostgreSQL search_path handling (#35588) * Fix issue with special $user search path variable not being resolved For any scenario in which PostgreSQL's internal "information_schema" is queried using a schema/schemas derived from the search_path configured on the connection, it is necessary to resolve PostgreSQL's special "$user" variable to the actual username set on the connection. While this resolution is performed correctly in the related parseSchemaAndTable() method, it was not being performed here, prior to this change. * Update variable name to reflect what it actually represents * Add new tests to cover search_path behavior when dropping all tables
2020-12-12 22:19:37 -05:00
$builder = $this->getBuilder($connection);
$builder->dropAllTables();
}
Add tests to cover any added or modified code
2020-12-08 17:07:49 -05:00
protected function getConnection()
{
return m::mock(Connection::class);
}
protected function getBuilder($connection)
{
return new PostgresBuilder($connection);
}
protected function getGrammar()
{
return new PostgresGrammar;
}
}
Reference in New Issue Copy Permalink