When I run the PHPUnit tests against the latest Drupal 8 HEAD I'm getting an error. The 'bootstrap' option is prefixed with 4 dashes instead of 2:

$ ./drupalci config:show current
---------------- Start config set: CURRENT DCI ENVIRONMENT ----------------
Defined in ~/.drupalci/config:
DCI_UseLocalCodeBase: /home/pieter/v/drupal/drupal
DCI_PHPVersion: 5.5
[pieter@thinkarch drupalci_testbot (dev $=)]$ ./drupalci run phpunit
...
Container /jolly_albattani created from image drupalci/web-5.5 with ID b4761c5c
Executing on container instance b4761c5c:
/var/www/html/core/vendor/phpunit/phpunit/phpunit  ----bootstrap /var/www/html/core/tests/bootstrap.php /var/www/html/core/tests/Drupal/Tests
Command created as exec id d517e7fa
PHPUnit 4.8.6 by Sebastian Bergmann and contributors.

unrecognized option ----bootstrap
CommentFileSizeAuthor
#2 2574925-2.patch693 bytespfrenssen

Comments

pfrenssen created an issue. See original summary.

pfrenssen’s picture

Status: Active » Needs review
StatusFileSize
new693 bytes
dawehner’s picture

+++ b/src/DrupalCI/Plugin/JobTypes/phpunit/PHPUnitJob.php
@@ -36,7 +36,7 @@ class PHPUnitJob extends JobBase {
-    'DCI_RunOptions' => "--bootstrap /var/www/html/core/tests/bootstrap.php",
+    'DCI_RunOptions' => "bootstrap /var/www/html/core/tests/bootstrap.php",

Can't we simply just use -c core ?

isntall’s picture

Status: Needs review » Reviewed & tested by the community
Issue tags: +Needs deployment

This patch has been reviewed and can be committed.

As far as comment number 3 that sill should be discussed.

isntall’s picture

Assigned: Unassigned » isntall
Status: Reviewed & tested by the community » Fixed
Issue tags: -Needs deployment

This has been merged into dev, production and is now live.

Status: Fixed » Closed (fixed)

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