Skip to content
Snippets Groups Projects
Commit 8f2d6063 authored by catch's avatar catch
Browse files

Issue #3417531 by longwave, mondrake: Remove withConsecutive() in pgsql SchemaTest

parent 267f1252
No related branches found
No related tags found
26 merge requests!8528Issue #3456871 by Tim Bozeman: Support NULL services,!3878Removed unused condition head title for views,!38582585169-10.1.x,!3818Issue #2140179: $entity->original gets stale between updates,!3742Issue #3328429: Create item list field formatter for displaying ordered and unordered lists,!3731Claro: role=button on status report items,!3668Resolve #3347842 "Deprecate the trusted",!3651Issue #3347736: Create new SDC component for Olivero (header-search),!3531Issue #3336994: StringFormatter always displays links to entity even if the user in context does not have access,!3355Issue #3209129: Scrolling problems when adding a block via layout builder,!3226Issue #2987537: Custom menu link entity type should not declare "bundle" entity key,!3154Fixes #2987987 - CSRF token validation broken on routes with optional parameters.,!3133core/modules/system/css/components/hidden.module.css,!2964Issue #2865710 : Dependencies from only one instance of a widget are used in display modes,!2812Issue #3312049: [Followup] Fix Drupal.Commenting.FunctionComment.MissingReturnType returns for NULL,!2378Issue #2875033: Optimize joins and table selection in SQL entity query implementation,!2334Issue #3228209: Add hasRole() method to AccountInterface,!2062Issue #3246454: Add weekly granularity to views date sort,!1105Issue #3025039: New non translatable field on translatable content throws error,!1073issue #3191727: Focus states on mobile second level navigation items fixed,!10223132456: Fix issue where views instances are emptied before an ajax request is complete,!877Issue #2708101: Default value for link text is not saved,!617Issue #3043725: Provide a Entity Handler for user cancelation,!579Issue #2230909: Simple decimals fail to pass validation,!560Move callback classRemove outside of the loop,!555Issue #3202493
Pipeline #85712 passed
Pipeline: drupal

#85715

    ......@@ -6,8 +6,7 @@
    use Drupal\pgsql\Driver\Database\pgsql\Schema;
    use Drupal\Tests\UnitTestCase;
    // cSpell:ignore conname
    use Prophecy\Argument;
    /**
    * @coversDefaultClass \Drupal\pgsql\Driver\Database\pgsql\Schema
    ......@@ -15,24 +14,6 @@
    */
    class SchemaTest extends UnitTestCase {
    /**
    * The PostgreSql DB connection.
    *
    * @var \PHPUnit\Framework\MockObject\MockObject|\Drupal\pgsql\Driver\Database\pgsql\Connection
    */
    protected $connection;
    /**
    * {@inheritdoc}
    */
    protected function setUp(): void {
    parent::setUp();
    $this->connection = $this->getMockBuilder('\Drupal\pgsql\Driver\Database\pgsql\Connection')
    ->disableOriginalConstructor()
    ->getMock();
    }
    /**
    * Tests whether the actual constraint name is correctly computed.
    *
    ......@@ -48,24 +29,20 @@ protected function setUp(): void {
    */
    public function testComputedConstraintName($table_name, $name, $expected) {
    $max_identifier_length = 63;
    $schema = new Schema($this->connection);
    $statement = $this->createMock('\Drupal\Core\Database\StatementInterface');
    $statement->expects($this->any())
    ->method('fetchField')
    ->willReturn($max_identifier_length);
    $connection = $this->prophesize('\Drupal\pgsql\Driver\Database\pgsql\Connection');
    $connection->getConnectionOptions()->willReturn([]);
    $connection->getPrefix()->willReturn('');
    $statement = $this->prophesize('\Drupal\Core\Database\StatementInterface');
    $statement->fetchField()->willReturn($max_identifier_length);
    $connection->query('SHOW max_identifier_length')->willReturn($statement->reveal());
    $this->connection->expects($this->exactly(2))
    ->method('query')
    ->withConsecutive(
    [$this->anything()],
    ["SELECT 1 FROM pg_constraint WHERE conname = '$expected'"],
    )
    ->willReturnOnConsecutiveCalls(
    $statement,
    $this->createMock('\Drupal\Core\Database\StatementInterface'),
    );
    $connection->query(Argument::containingString($expected))
    ->willReturn($this->prophesize('\Drupal\Core\Database\StatementInterface')->reveal())
    ->shouldBeCalled();
    $schema = new Schema($connection->reveal());
    $schema->constraintExists($table_name, $name);
    }
    ......
    ......@@ -2078,11 +2078,6 @@ parameters:
    count: 1
    path: modules/pgsql/src/Update10101.php
    -
    message: "#^Call to deprecated method withConsecutive\\(\\) of class PHPUnit\\\\Framework\\\\MockObject\\\\Builder\\\\InvocationMocker\\.$#"
    count: 1
    path: modules/pgsql/tests/src/Unit/SchemaTest.php
    -
    message: "#^Variable \\$responsive_image_styles in empty\\(\\) always exists and is not falsy\\.$#"
    count: 1
    ......
    0% Loading or .
    You are about to add 0 people to the discussion. Proceed with caution.
    Finish editing this message first!
    Please register or to comment