Convert web tests to browser tests for REST module

Going forward, let's please stop doing individual module conversions and do larger chunks at once. See #2770921: Feb 21st: Convert chunk of WTB to BTB by just moving classes, changing use statements adding traits and https://www.drupal.org/core/scope#incomplete.

There are still WebTests in rest after the big bang, reopening the issue.

You're right, in the deprecated \Drupal\rest\Tests\RESTTestBase and its subclasses.

Took a quick look at \Drupal\rest\Tests\ResourceTest but that will require more then a quick look :)

Updated the IS to set some scope on this conversion

I'm wondering whether this case is the chance for us to provide a ApiTestBase, aka. something which provides methods like:

$this->request()

On the other hand I believe we should just convert the test over, given that we have a MUCH better test coverage in phpunit already.

#2863267: Convert web tests of views landed.

Here's a start.

Now the IS reflects the actual scope & status.

Also doing StyleSerializerTest. Just one last failure…

StyleSerializerTest That test takes sooooooooooo much time.

Completed StyleSerializerTest.

Now working on the last one.

This should be green.

Did a quick code review and it looks good! Haven't found anything yet.

+++ b/core/modules/rest/src/Tests/ResourceTest.php
@@ -3,16 +3,20 @@
-class ResourceTest extends RESTTestBase {
+class ResourceTest extends BrowserTestBase {

I think it's a good thing base classes are dropped. It does make the test a little less magical.

#15, that, and RESTTestBase is already deprecated anyway:

/**
 * Test helper class that provides a REST client method to send HTTP requests.
 *
 * @deprecated in Drupal 8.3.x-dev and will be removed before Drupal 9.0.0. Use \Drupal\Tests\rest\Functional\ResourceTestBase and \Drupal\Tests\rest\Functional\EntityResource\EntityResourceTestBase instead. Only retained for contributed module tests that may be using this base class.
 */
abstract class RESTTestBase extends WebTestBase {

Shall we also convert the RESTTestBase dependent tests also as the part of this and close this one as duplicate #2889882: Convert ResponseGeneratorTest, DbLogResourceTest, RestRegisterUserTest to BTB

No, I'd rather get this in separately — that'd mean progress! I think #2889882: Convert ResponseGeneratorTest, DbLogResourceTest, RestRegisterUserTest to BTB will be a bit more painful.

Let's move the new tests to their namespace. Changes look good on local.

$ ../vendor/bin/phpunit modules/rest/tests/src/Functional/Views/ExcludedFieldTokenTest.php
PHPUnit 4.8.35 by Sebastian Bergmann and contributors.

Testing Drupal\Tests\rest\Functional\Views\ExcludedFieldTokenTest
.

Time: 3.95 minutes, Memory: 4.00MB

OK (1 test, 2 assertions)

$ ../vendor/bin/phpunit modules/rest/tests/src/Functional/Views/StyleSerializerTest.php
PHPUnit 4.8.35 by Sebastian Bergmann and contributors.

Testing Drupal\Tests\rest\Functional\Views\StyleSerializerTest
.............

Time: 1.02 hours, Memory: 4.00MB

OK (13 tests, 130 assertions)

$ ../vendor/bin/phpunit modules/rest/tests/src/Functional/ResourceTest.php
PHPUnit 4.8.35 by Sebastian Bergmann and contributors.

Testing Drupal\Tests\rest\Functional\ResourceTest
....

Time: 11 minutes, Memory: 4.00MB

OK (4 tests, 95 assertions)

//Naveen

Let's move the new tests to their namespace.

+1

So… is this then RTBC? (@naveenvalecha, just moving files doesn't mean you can't RTBC anymore.)

Did another review pass, everything looks good. Setting to RTBC!

RTBC +1

#16 +1 It's good to leave deprecated RESTTestBase where it is. Let's RESTTestBase child classes(ResponseGeneratorTest, DbLogResourceTest, RestRegisterUserTest) use the BTB directly.

//Naveen

No, I'd rather get this in separately — that'd mean progress! I think #2889882: Convert RESTTestBase, ResponseGeneratorTest, DbLogResourceTest,ResourceTest, ResponseGeneratorTest WTB to BTB will be a bit more painful.

I agree, we have followed that strategy in multiple places.

1. +++ b/core/modules/rest/src/Tests/ResourceTest.php
   @@ -86,37 +96,55 @@ public function testAuthentication() {
   +      RequestOptions::HEADERS => ['Content-Type' => 'application/json'],
   +      RequestOptions::BODY => $serialized,
   

   These constants are nice!

2. +++ b/core/modules/rest/src/Tests/Views/StyleSerializerTest.php
   @@ -374,19 +365,19 @@ public function testResponseFormatConfiguration() {
   -    $this->assertTrue(strpos($headers['content-type'], 'text/xml') !== FALSE, 'The header Content-type is correct.');
   +    $this->assertSame($headers['Content-Type'], ['text/xml; charset=UTF-8']);
   
   @@ -606,7 +597,8 @@ public function testSerializerViewsUI() {
   +    $json_preview = $result[0]->getText();
   +    $this->assertSame($this->drupalGet('test/serialize/field'), $json_preview, 'The expected JSON preview output was found.');
   

   These are two instances where we switched to use assertSame, but we have the wrong order. Let's not introduce a small regression here.

Fixed #24.2.

Thank you @Wim Leers!

Committed/pushed to 8.5.x and cherry-picked to 8.4.x. Thanks!

🎉