From a8a6638ec740fc227a702203483048e1067e6ade Mon Sep 17 00:00:00 2001 From: Alex Pott Date: Sat, 13 Jun 2020 09:54:42 +0100 Subject: [PATCH] Issue #3139132 by Beakerboy, daffie: Query uses hardcoded LIMIT instead of queryRange() --- core/modules/big_pipe/tests/src/Functional/BigPipeTest.php | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/core/modules/big_pipe/tests/src/Functional/BigPipeTest.php b/core/modules/big_pipe/tests/src/Functional/BigPipeTest.php index acafd7d9aa5..21836de9d9b 100644 --- a/core/modules/big_pipe/tests/src/Functional/BigPipeTest.php +++ b/core/modules/big_pipe/tests/src/Functional/BigPipeTest.php @@ -192,7 +192,9 @@ class BigPipeTest extends BrowserTestBase { // Verify that the two expected exceptions are logged as errors. $this->assertEqual($log_count + 2, $connection->query('SELECT COUNT(*) FROM {watchdog}')->fetchField(), 'Two new watchdog entries.'); - $records = $connection->query('SELECT * FROM {watchdog} ORDER BY wid DESC LIMIT 2')->fetchAll(); + // Using the method queryRange() allows contrib database drivers the ability + // to insert their own limit and offset functionality. + $records = $connection->queryRange('SELECT * FROM {watchdog} ORDER BY wid DESC', 0, 2)->fetchAll(); $this->assertEqual(RfcLogLevel::ERROR, $records[0]->severity); $this->assertStringContainsString('Oh noes!', (string) unserialize($records[0]->variables)['@message']); $this->assertEqual(RfcLogLevel::ERROR, $records[1]->severity);