Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Niklas Fiekas’s picture

Assigned: Unassigned » Niklas Fiekas

Doing this now.

Niklas Fiekas’s picture

Assigned: Niklas Fiekas » Unassigned
Status: Active » Needs review
FileSize
106.44 KB
54.2 KB

Ok, all classes converted.

I have also attached the concatenated output of git blame -C <new-file.php> for all the new files (best viewed without the automatic line wrapping of Dreditor - open in a new tab and zoom out or something, to get an overview), showing that most of the code is directly and unchanged coming from the old test file.

RobLoach’s picture

FileSize
54.2 KB

The IP Acronymn is camel-cased :-) .

aspilicious’s picture

Status: Needs review » Needs work
+++ b/core/modules/system/lib/Drupal/system/Tests/Bootstrap/GetFilenameTest.phpundefined
@@ -0,0 +1,55 @@
+ * Definition of Drupal\system\Tests\Bootstrap\GetFilenameTest.

GetFilenameTest

+++ b/core/modules/system/lib/Drupal/system/Tests/Bootstrap/GetFilenameTest.phpundefined
@@ -0,0 +1,55 @@
+class BootstrapGetFilenameTest extends UnitTestBase {

BootstrapGetFilenameTest

Same thing, different name

+++ b/core/modules/system/lib/Drupal/system/Tests/Bootstrap/ResettableStaticTest.phpundefined
@@ -0,0 +1,49 @@
+class ResettableStaticTest extends UnitTestBase {

In the other conversions I always had a "UnitTest" suffix. I would like to be consistent but I'm not going to fight for it.

4 days to next Drupal core point release.

Niklas Fiekas’s picture

Status: Needs work » Needs review
FileSize
1.66 KB
54.22 KB

Fixed that. Also using the UnitTest suffix, now.

RobLoach’s picture

FileSize
54.22 KB

Camel-case acronyms :-) .

Niklas Fiekas’s picture

Heh ... lol. Thanks!

aspilicious’s picture

Status: Needs review » Needs work
+++ b/core/modules/system/lib/Drupal/system/Tests/Bootstrap/GetFilenameUnitTest.phpundefined
@@ -0,0 +1,55 @@
+ * Definition of Drupal\system\Tests\Bootstrap\GetFilenameTest.

+class GetFilenameUnitTest extends UnitTestBase {

Mismatch code vs doc classname

+++ b/core/modules/system/lib/Drupal/system/Tests/Bootstrap/ResettableStaticUnitTest.phpundefined
@@ -0,0 +1,49 @@
+ * Definition of Drupal\system\Tests\Bootstrap\ResettableStaticTest.
...
+class ResettableStaticUnitTest extends UnitTestBase {

Mismatch code vs doc classname

3 days to next Drupal core point release.

+++ b/core/modules/system/lib/Drupal/system/Tests/Bootstrap/TimerTest.phpundefined
@@ -0,0 +1,45 @@
+class TimerTest extends UnitTestBase {

use UnitTest as a suffix or don't, as we are doing it alrdy in this patch and else we should keep doing that.

3 days to next Drupal core point release.

Niklas Fiekas’s picture

Thanks for reviewing!

aspilicious’s picture

Status: Needs review » Needs work

Srry missed a few unit tests while reviewing!

+++ b/core/modules/system/lib/Drupal/system/Tests/Bootstrap/MiscTest.phpundefined
@@ -1,461 +1,18 @@
+class MiscTest extends UnitTestBase {

Found more unit tests

+++ b/core/modules/system/lib/Drupal/system/Tests/Bootstrap/OverrideServerVariablesTest.phpundefined
@@ -2,32 +2,55 @@
+class OverrideServerVariablesTest extends UnitTestBase {

Another one

2 days to next Drupal core point release.

Niklas Fiekas’s picture

Status: Needs work » Needs review
FileSize
2.13 KB
121.86 KB

Changed that.

aspilicious’s picture

Issue tags: -PSR-0

Status: Needs review » Needs work
Issue tags: +PSR-0

The last submitted patch, 1598556-bootstrap-tests-psr-0-11.patch, failed testing.

Niklas Fiekas’s picture

Status: Needs work » Needs review
FileSize
54.31 KB

Merged the changes from the kernel branch. (Yay, for that it's in!)

aspilicious’s picture

Status: Needs review » Reviewed & tested by the community

Good to go

Dries’s picture

Dries’s picture

I think this patch needs a re-roll. Asking for a re-test.

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 1598556-bootstrap-tests-psr-0-14.patch, failed testing.

Niklas Fiekas’s picture

Status: Needs work » Needs review
FileSize
54.28 KB

Rerolled.

aspilicious’s picture

Status: Needs review » Reviewed & tested by the community

green again ;)

Dries’s picture

Status: Reviewed & tested by the community » Fixed

Great - glad things are back to green. Committed to 8.x. Thanks!

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