We have enabled automated branch and patch testing for the module.
Currently this passes for php7 but php5.5 / 5.6 still fail.

Original report
According to comments made by @aspilicious in #2624410: Create settings for the url processor to change the facet's alias (comment 11 & 13). it should be possible to enable testing on drupal.org.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

borisson_ created an issue. See original summary.

  • borisson_ committed d32ab64 on 8.x-1.x
    Issue #2638330: Make sure all unit tests get detected on a case...
borisson_’s picture

borisson_’s picture

borisson_’s picture

Status: Active Β» Needs work

Currently the testsuite passes on php7(https://www.drupal.org/pift-ci-job/124864).

It doesn't pass on:
- php5.5 + sqlite (https://www.drupal.org/pift-ci-job/125166)
- php5.5 + mysql (https://www.drupal.org/pift-ci-job/125216)
- php5.6 + mysql (https://www.drupal.org/pift-ci-job/125222)

borisson_’s picture

During the debugging of this issue; I also noticed that not all tests pass when ran trough the UI, even on php7. The test fails are different but they need more work.

borisson_’s picture

I spent a couple of hours trying to debug the php5 fails but I didn't really get any closer to a solution. I set up the drupalci vagrant box locally and I can reproduce the php5 fails there.

We're expecting the result of the test (UrlIntegrationTest:97) to be "http://localhost/checkout/search-api-test-fulltext?y%5B0%5D=facet%7C%7Citem (y[0]=facet||item).

The 2 pipes are the seperator used in DummyQuery - a url processor defined in a test module. We've set the y as filter key earlier in the test.

    $edit = [
      'filterKey' => 'y',
      'urlProcessor' => 'dummy_query',
    ];
    $this->drupalPostForm(NULL, $edit, $this->t('Save'));

The result is "http://localhost/checkout/search-api-test-fulltext?f%5B0%5D=facet%3Aitem" (f[0]=facet:item). This is as if there is the save of the form in the test didn't work. But we have a specific test for saving the form that does work (it is green - also on php5).

I don't understand why this test doesn't work. It should.

borisson_’s picture

Status: Needs work Β» Needs review
FileSize
2.91 KB

So I wrote out a lot more specific in the test what we're actually trying to test. However there are still unexplainable things going on for php5.
At least the test is more explicit in what it's doing.

borisson_’s picture

Issue summary: View changes
borisson_’s picture

FileSize
1.12 KB
2.91 KB
borisson_’s picture

FileSize
2.32 KB
2.91 KB

So, I was doubting that the actual saving of this might've been the issue. That's why I expanded the FacetSourceTest. See attached patch.

This verifies that the saving of the facet source is not what's going wrong.

borisson_’s picture

FileSize
5.23 KB

Uploading the right patch now :)

borisson_’s picture

FileSize
2.21 KB
6.18 KB
borisson_’s picture

Looks like the failures in travis are different to the failures here, because this issue didn't give me enough headaches already as it was.

borisson_’s picture

FileSize
6.21 KB

Rebase.

Status: Needs review Β» Needs work

The last submitted patch, 15: enable_testing_on-2638330-15.patch, failed testing.

borisson_’s picture

Status: Needs work Β» Needs review
FileSize
395 bytes
6.24 KB
borisson_’s picture

FileSize
7.92 KB
13.54 KB

We relied on some search api views that had the same path, see #2654860: Test views have the same path.. We now ship our own view (just one, not 2).

  • borisson_ committed 981941f on 8.x-1.x
    Issue #2638330 by borisson_: Enable testing on drupal.org for patches
    
borisson_’s picture

Status: Needs review Β» Fixed
borisson_’s picture

Many, many thanks to @Nick_vh for helping me out in debugging this issue.

The last submitted patch, 8: enable_testing_on-2638330-8.patch, failed testing.

The last submitted patch, 8: enable_testing_on-2638330-8.patch, failed testing.

The last submitted patch, 10: enable_testing_on-2638330-10.patch, failed testing.

The last submitted patch, 11: enable_testing_on-2638330-10.patch, failed testing.

The last submitted patch, 12: enable_testing_on-2638330-12.patch, failed testing.

The last submitted patch, 13: enable_testing_on-2638330-13.patch, failed testing.

The last submitted patch, 15: enable_testing_on-2638330-15.patch, failed testing.

The last submitted patch, 17: enable_testing_on-2638330-17.patch, failed testing.

Status: Fixed Β» Needs work

The last submitted patch, 18: enable_testing_on-2638330-18.patch, failed testing.

borisson_’s picture

Status: Needs work Β» Fixed

Status: Fixed Β» Closed (fixed)

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