Create a test for views date handlers

This looks great.

+++ b/core/modules/views/lib/Drupal/views/Tests/Handler/ArgumentDateTest.php
@@ -0,0 +1,325 @@
+    $this->installSchema('user', 'role_permission');

I think this can be removed too once the handlers are moved to views.

+++ b/core/modules/views/lib/Drupal/views/Tests/Handler/ArgumentDateTest.php
@@ -0,0 +1,325 @@
+    $view->setDisplay('embed_3', array(1));
+    $this->executeView($view);

executeView($view, array(52))

oops

#10: drupal-1934558-10.patch queued for re-testing.

+++ b/core/modules/views/lib/Drupal/views/Tests/Handler/ArgumentDateTest.php
@@ -0,0 +1,326 @@
+  /**
+   * Stores the plugin ID used in the current test method.
+   *
+   * @var string
+   */
+  protected $usedPlugin = 'date';

Is this needed?

Agreed, this looks like great coverage!

+++ b/core/modules/views/lib/Drupal/views/Tests/Handler/ArgumentDateTest.phpundefined
@@ -0,0 +1,326 @@
+      'node_created_fulldate',
+      'node_created_day',
+      'node_created_month',
+      'node_created_week',
+      'node_created_year',
+      'node_created_year_month',

I guess this depends on whether this or the actual moving of the handlers gets in first.

Also, I think olli is right, that usedPlugin property doesn't seem to be used anywhere.

+    $this->executeView($view, array('2'));
...
+    $this->executeView($view, array('5'));

Not sure why but week is now 01-53, so '02' and '05' here. Or should we ensure the padding elsewhere?

Yeah 02 and 2 should be both accepted. (followup)

Looks good to me. I agree with the change made in #18.

In the follow-up, I think we would want day and month work the same way as week (both 01 and 1 are good).

Additionally, this issue made me wonder if we need to add a year handler "Year for the week" that can be used with the week handler and support sunday as the first day of week as well.

Yay, test coverage!

Committed and pushed to 8.x. Thanks!