I'm adding tests to Panels Everywhere and the testrunner is giving a strange error:

00:00:16.361 Entering setup_checkout_git().
00:00:16.362 Performing git checkout of git://git.drupal.org/project/panels_everywhere.git  branch to /var/lib/drupalci/web/jenkins-default-220086/sites/all/modules/panels_everywhere.
00:00:16.364 Git Command: git clone -b  --depth 1 git://git.drupal.org/project/panels_everywhere.git '/var/lib/drupalci/web/jenkins-default-220086/sites/all/modules/panels_everywhere'
00:00:16.366 Too many arguments.

The -b argument needs a value passed to it. OTOH, a Metatag test runner passes the branch to it:

00:00:15.949 Entering setup_checkout_git().
00:00:15.949 Performing git checkout of git://git.drupal.org/project/metatag.git 7.x-1.x branch to /var/lib/drupalci/web/jenkins-default-218031/sites/all/modules/metatag.
00:00:15.951 Git Command: git clone -b 7.x-1.x --depth 1 git://git.drupal.org/project/metatag.git '/var/lib/drupalci/web/jenkins-default-218031/sites/all/modules/metatag'

Any idea why that would be happening.

Comments

DamienMcKenna created an issue. See original summary.

legovaer’s picture

legovaer
he/him
Primary language Dutch
Location Duffel
commented

The issue doesn't seem to be related to the TestRunner itself. It seems like the Drupal.org integration is not sending the branch to the TestRunner which needs to be tested against as you can see here.

The parameter DCI_AdditionalRepositories is missing the branch name:

As-is: git,git://git.drupal.org/project/panels_everywhere.git,,sites/all/modules/panels_everywhere,1;git,git://git.drupal.org/project/ctools.git,7.x-1.10,sites/all/modules/ctools,1;git,git://git.drupal.org/project/panels.git,7.x-3.7,sites/all/modules/panels,1;

As-should: git,git://git.drupal.org/project/panels_everywhere.git,7.x-1.x-dev,sites/all/modules/panels_everywhere,1;git,git://git.drupal.org/project/ctools.git,7.x-1.10,sites/all/modules/ctools,1;git,git://git.drupal.org/project/panels.git,7.x-3.7,sites/all/modules/panels,1;

damienmckenna’s picture

damienmckenna
Location TN, USA
commented

Any idea what could cause that?

damienmckenna’s picture

damienmckenna
Location TN, USA
commented

So any idea on how to fix this problem? I'd like to add some tests to Panels Everywhere and this is cramping my style ;-)

Mixologic’s picture

Mixologic
He/Him/His
Primary language English
Location Portland, OR
commented
Project: DrupalCI: Test Runner » Project issue file test
Version: » 7.x-3.x-dev
Component: Testrunner Tests » Testing Configuration

Sorry for the long delay, been seriously heads down in refactoring the drupalci codebase. This is probably something on drupal.org's side of not sending the branch name, which is probably due to a hiccup somewhere. I'll file it under Pift for now, and ping @drumm for ideas.

drumm’s picture

drumm
he/him
Location NY, US
commented
Assigned: Unassigned » drumm

Looking into this, Drupal.org lost track of which tag that release has.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented

Thanks!

drumm’s picture

drumm
he/him
Location NY, US
commented
Status: Active » Fixed

Done, and also updating 14 other 7.x & 8.x dev releases in the same situation. I think there’s a point in the release creation process after selecting the label and before packaging where Drupal.org is tracking the label by ID, but not preventing pushing a delete tag.

I requeued tests for Panels Everywhere and it looks like it is running into #2645590: Ensure that simpletest job doesn't "fail" testing if no tests are present now.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented

Thanks! I was working on writing tests in #2804901: Add tests for all basic functionality, now the test runs will work :)

Status: Fixed » Closed (fixed)

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