Convert aggregator_test_feed() and aggregator_test_redirect() to a new style controller

Go testbot, go!

(Later maybe we should consider #2004022: Move aggregator_test module into its own directory.)

+++ b/core/modules/aggregator/tests/aggregator_test.moduleundefined
@@ -24,49 +17,6 @@ function aggregator_test_menu() {
-  if ($use_last_modified) {
-    drupal_add_http_header('Last-Modified', gmdate(DATE_RFC1123, $last_modified));
-  }
...
-  drupal_add_http_header('Expires', 'Sun, 19 Nov 1978 05:00:00 GMT');

+++ b/core/modules/aggregator/tests/lib/Drupal/aggregator_test/Controller/AggregatorTestController.phpundefined
@@ -0,0 +1,59 @@
+    $last_modified = new \DateTime('Sun, 19 Nov 1978 05:00:00 GMT');
...
+    if ($use_last_modified) {
+      $response->setLastModified($last_modified);
+    }

Is it just me that this logic is not 100% the same? Dries birthday got added before all the time.

+++ b/core/modules/aggregator/tests/aggregator_test.routing.ymlundefined
@@ -0,0 +1,8 @@
+    use_etag: '0'
+    use_last_modified: '0'

can't we also use false here which then get converted to FALSE?

Intresting. Have a look:

Before the patch:

http://localhost/drupal-8/aggregator/test-feed/1/1

GET /drupal-8/aggregator/test-feed/1/1 HTTP/1.1
Host: localhost
User-Agent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:21.0) Gecko/20100101 Firefox/21.0
Accept: text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8
Accept-Language: de-de,de;q=0.8,en-us;q=0.5,en;q=0.3
Accept-Encoding: gzip, deflate
Cookie: Drupal.toolbar.collapsed=0
Connection: keep-alive
Pragma: no-cache
Cache-Control: no-cache

HTTP/1.1 200 OK
Date: Tue, 28 May 2013 10:34:08 GMT
Server: Apache/2.2.22 (Ubuntu)
X-Powered-By: PHP/5.4.9-4ubuntu2
Last-Modified: Sun, 19 Nov 1978 05:00:00 +0000, Tue, 28 May 2013 10:34:08 +0000 <-- Notice two different Last-Modified
Etag: h__lfVU1SyY-w28Xq0pfxb8gcBNeuD-43cpt_zIe98w, "1369737248" <-- Notice two ETags
Expires: Sun, 19 Nov 1978 05:00:00 GMT, Sun, 19 Nov 1978 05:00:00 GMT <-- Notice the expires header twice
Cache-Control: must-revalidate, must-revalidate, no-cache, post-check=0, pre-check=0, private <-- Notice must-revalidate twice
X-UA-Compatible: IE=edge,chrome=1
Content-Language: en
Vary: Accept-Encoding
Content-Encoding: gzip
Keep-Alive: timeout=5, max=100
Connection: Keep-Alive
Transfer-Encoding: chunked
Content-Type: text/html; charset=UTF-8 <-- Notice the wrong content type

After the patch:

http://localhost/drupal-8/aggregator/test-feed/1/1

GET /drupal-8/aggregator/test-feed/1/1 HTTP/1.1
Host: localhost
User-Agent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:21.0) Gecko/20100101 Firefox/21.0
Accept: text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8
Accept-Language: de-de,de;q=0.8,en-us;q=0.5,en;q=0.3
Accept-Encoding: gzip, deflate
Cookie: Drupal.toolbar.collapsed=0
Connection: keep-alive
Pragma: no-cache
Cache-Control: no-cache

HTTP/1.1 200 OK
Date: Tue, 28 May 2013 10:36:40 GMT
Server: Apache/2.2.22 (Ubuntu)
X-Powered-By: PHP/5.4.9-4ubuntu2
Cache-Control: must-revalidate, no-cache, post-check=0, pre-check=0, private
Last-Modified: Tue, 28 May 2013 10:36:40 +0000 <-- Why is this so?
Etag: "1369737400" <-- Is this the generated ETag?
X-UA-Compatible: IE=edge,chrome=1
Content-Language: en
Expires: Sun, 19 Nov 1978 05:00:00 GMT <-- Looks correct
Accept-Ranges: bytes
content-transfer-encoding: binary
Vary: Accept-Encoding
Content-Encoding: gzip
Keep-Alive: timeout=5, max=100
Connection: Keep-Alive
Transfer-Encoding: chunked
Content-Type: application/rss+xml; charset=utf-8

Recorded with Firefox Live Http headers. I don't think I fully understand what's going on here.

lets just wont fix this
#1988802: [META] Rewrite test modules in system to provide better unit testing.

I'll help!

Whoops... @rabellamy is working on these :P

Marked #1987600: Convert aggregator_test_redirect() to a new style controller as duplicate of this issue.

+++ b/core/modules/aggregator/tests/modules/aggregator_test/lib/Drupal/aggregator_test/Controller/AggregatorTestRssController.php
@@ -0,0 +1,72 @@
+      drupal_add_http_header('Last-Modified', gmdate(DATE_RFC1123, $last_modified));
...
+      drupal_add_http_header('ETag', $etag);
...
+    drupal_add_http_header('Expires', 'Sun, 19 Nov 1978 05:00:00 GMT');
+    drupal_add_http_header('Cache-Control', 'must-revalidate');
+    drupal_add_http_header('Content-Type', 'application/rss+xml; charset=utf-8');

What you should do instead is to create a response object and set all this header onto it and finally just return it.

*sad trombone* I was going to help convert those drupal_add_http_headers but apparently Chicago Midway doesn't have Wi-Fi. Sorry :(

Try this...

#14: drupal8.aggregator-module.1987598-14.patch queued for re-testing.

Was missing params on the route for the optional parameters. Also, extending ControllerBase to get access to urlGenerator() and $this->redirect. Renamed methods to testFeed and testRedirect.

passing route to $this->redirect should resolve the last failure.

1. +++ w/core/modules/aggregator/tests/modules/aggregator_test/lib/Drupal/aggregator_test/Controller/AggregatorTestRssController.php
   @@ -0,0 +1,66 @@
   +
   +class AggregatorTestRssController extends ControllerBase {
   

   Can we have a little bit of docs on there?

2. +++ w/core/modules/aggregator/tests/modules/aggregator_test/lib/Drupal/aggregator_test/Controller/AggregatorTestRssController.php
   @@ -0,0 +1,66 @@
   +   * @param $use_etag
   +   *   Set TRUE to send an etag.
   +   */
   +  public function testFeed($use_last_modified, $use_etag, Request $request) {
   ...
   +
   +  public function testRedirect() {
   +    return $this->redirect('aggregator_test_feed', array(), 301);
   +  }
   +
   

   Missing @return

Thanks for your work on this issue! Please see #1971384-43: [META] Convert page callbacks to controllers for an update on the routing system conversion process.

Try this...

Whoops... paging Dr. Testbot...

#23: drupal8.aggregator-module.1987598-23.patch queued for re-testing.

#23: drupal8.aggregator-module.1987598-23.patch queued for re-testing.

A couple of random failures later ...

XD Thanks!

Committed and pushed to 8.x. Thanks!