That's just a small bit to decouple and make it phpunitestable.

Files: 
CommentFileSizeAuthor
#30 date-2003934-30.patch6.58 KBtim.plunkett
PASSED: [[SimpleTest]]: [MySQL] 57,250 pass(es).
[ View ]
#25 2003934-25.patch6.59 KBdamiankloip
PASSED: [[SimpleTest]]: [MySQL] 55,059 pass(es).
[ View ]
#25 interdiff-2003934-25.txt609 bytesdamiankloip
#23 2003934-23.patch6.58 KBdamiankloip
FAILED: [[SimpleTest]]: [MySQL] 55,805 pass(es), 4 fail(s), and 0 exception(s).
[ View ]
#23 interdiff-2003934-23.txt616 bytesdamiankloip
#19 drupal-2003934-19.patch6.59 KBdawehner
FAILED: [[SimpleTest]]: [MySQL] Failed to run tests: tests were executed, but no results were found.
[ View ]
#19 interdiff.txt1.01 KBdawehner
#15 2003934-15.patch6.6 KBdamiankloip
FAILED: [[SimpleTest]]: [MySQL] Failed to run tests: tests were executed, but no results were found.
[ View ]
#15 interdiff-2003934-15.txt1.96 KBdamiankloip
#13 drupal-2003934-13.patch6.13 KBdawehner
FAILED: [[SimpleTest]]: [MySQL] Failed to run tests: tests were executed, but no results were found.
[ View ]
#13 interdiff.txt1005 bytesdawehner
#9 drupal-2003934-9.patch6.09 KBdawehner
PASSED: [[SimpleTest]]: [MySQL] 56,595 pass(es).
[ View ]
#9 interdiff.txt812 bytesdawehner
#5 drupal-2003934-5.patch6.05 KBdawehner
FAILED: [[SimpleTest]]: [MySQL] 56,115 pass(es), 5 fail(s), and 3,905 exception(s).
[ View ]
#3 drupal-2003934-2.patch5.92 KBdawehner
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch drupal-2003934-2.patch. Unable to apply patch. See the log in the details link for more information.
[ View ]
#3 interdiff.txt658 bytesdawehner
#1 drupal-2003934-1.patch5.95 KBdawehner
FAILED: [[SimpleTest]]: [MySQL] Repository checkout: failed to checkout from [git://git.drupal.org/project/drupal.git].
[ View ]

Comments

Status:Active» Needs review
StatusFileSize
new5.95 KB
FAILED: [[SimpleTest]]: [MySQL] Repository checkout: failed to checkout from [git://git.drupal.org/project/drupal.git].
[ View ]

#1998466: Convert filter_xss_admin and similar function to an Xss component is the only global dependency which blocks from convering to unit tests.

Status:Needs review» Needs work

The last submitted patch, drupal-2003934-1.patch, failed testing.

Status:Needs work» Needs review
StatusFileSize
new658 bytes
new5.92 KB
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch drupal-2003934-2.patch. Unable to apply patch. See the log in the details link for more information.
[ View ]

#3: drupal-2003934-2.patch queued for re-testing.

StatusFileSize
new6.05 KB
FAILED: [[SimpleTest]]: [MySQL] 56,115 pass(es), 5 fail(s), and 3,905 exception(s).
[ View ]

Just a rerole.

Status:Needs review» Needs work

The last submitted patch, drupal-2003934-5.patch, failed testing.

Status:Needs work» Needs review

#5: drupal-2003934-5.patch queued for re-testing.

Status:Needs review» Needs work

The last submitted patch, drupal-2003934-5.patch, failed testing.

Status:Needs work» Needs review
StatusFileSize
new812 bytes
new6.09 KB
PASSED: [[SimpleTest]]: [MySQL] 56,595 pass(es).
[ View ]

Let's fix the failures.

Issue tags:+PHPUnit Blocker

+++ b/core/core.services.ymlundefined
@@ -427,3 +427,6 @@ services:
+  format_date:
+    class: Drupal\Core\Datetime\DateFormats

why not name the service date? then we can add more helper methods in the future

+++ b/core/lib/Drupal/Core/Datetime/DateFormats.phpundefined
@@ -0,0 +1,112 @@
+      $langcode = language(Language::TYPE_INTERFACE)->langcode;

we can inject language manager here and use instead

Status:Needs review» Needs work
Issue tags:-PHPUnit Blocker

The last submitted patch, drupal-2003934-9.patch, failed testing.

Status:Needs work» Needs review
Issue tags:+PHPUnit Blocker

#9: drupal-2003934-9.patch queued for re-testing.

StatusFileSize
new1005 bytes
new6.13 KB
FAILED: [[SimpleTest]]: [MySQL] Failed to run tests: tests were executed, but no results were found.
[ View ]

Yeah why not.

Status:Needs review» Needs work

The last submitted patch, drupal-2003934-13.patch, failed testing.

Status:Needs work» Needs review
StatusFileSize
new1.96 KB
new6.6 KB
FAILED: [[SimpleTest]]: [MySQL] Failed to run tests: tests were executed, but no results were found.
[ View ]

Rerolled and injected the language manager.

Status:Needs review» Needs work
Issue tags:-PHPUnit Blocker

The last submitted patch, 2003934-15.patch, failed testing.

Status:Needs work» Needs review

#15: 2003934-15.patch queued for re-testing.

Status:Needs review» Needs work
Issue tags:+PHPUnit Blocker

The last submitted patch, 2003934-15.patch, failed testing.

Status:Needs work» Needs review
StatusFileSize
new1.01 KB
new6.59 KB
FAILED: [[SimpleTest]]: [MySQL] Failed to run tests: tests were executed, but no results were found.
[ View ]

Let's use a central config variable.

ok sorry for this, but i just saw DrupalDateTime object..and i am sitting here and wondering why this logic shouldnt be in DrupalDateTime::format since it is the drupal specific object?

so i can do:

<?php
$date
= new DrupalDateTime(REQUEST_TIME);
$date->format('short');
?>

or even introduce a create() method to make this oneliner...

No, we don't want to do that because of #2003892: Convert date formats to config entities

Status:Needs review» Needs work

The last submitted patch, drupal-2003934-19.patch, failed testing.

Status:Needs work» Needs review
StatusFileSize
new616 bytes
new6.58 KB
FAILED: [[SimpleTest]]: [MySQL] 55,805 pass(es), 4 fail(s), and 0 exception(s).
[ View ]

We have a mismatch of service names.

Status:Needs review» Needs work

The last submitted patch, 2003934-23.patch, failed testing.

Status:Needs work» Needs review
StatusFileSize
new609 bytes
new6.59 KB
PASSED: [[SimpleTest]]: [MySQL] 55,059 pass(es).
[ View ]

Sorry, totally put the wrong constant in the language change :)

That nicely vouches for the test coverage I guess.

Component:views.module» base system

Nice! Not everything is views btw. :)

Status:Needs review» Reviewed & tested by the community

looks great to me:) thanks!

Needs reroll

curl https://drupal.org/files/2003934-25.patch | git a
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  6746  100  6746    0     0   6359      0  0:00:01  0:00:01 --:--:--  7588
error: patch failed: core/core.services.yml:440
error: core/core.services.yml: patch does not apply

Status:Reviewed & tested by the community» Needs work

Status:Needs work» Reviewed & tested by the community
StatusFileSize
new6.58 KB
PASSED: [[SimpleTest]]: [MySQL] 57,250 pass(es).
[ View ]

This patch and #2004506: Move standard_country_list() to Drupal\Core\Locale\Country both tried to add a service to the end of the core.services.yml.
Since it was just a failing of git and absolutely nothing changed, I'm reRTBCing.

Please don't credit me in the commit message.

Status:Reviewed & tested by the community» Fixed

Committed 0bfe623 and pushed to 8.x. Thanks!

Automatically closed -- issue fixed for 2 weeks with no activity.