So this is only a backport of an existing active issue. Typically I wait until the newer version is committed but this one is running into some strange testing issues. As a result I'm adding a D6 patch here to see if the same test issues persist. This patch is not complete as it doesn't yet include any tests of its own functional changes.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

eshta’s picture

eshta’s picture

Updated tests to be more like d7.

eshta’s picture

This adds a test for the configured endpoint.

eshta’s picture

A tweak to the tests to more closely match the d7 fix.

Status: Needs review » Needs work

The last submitted patch, 4: mollom.endpoint.backport.4.patch, failed testing.

eshta’s picture

Status: Needs work » Needs review
FileSize
12.26 KB

Fixed test failures (I hope).

Status: Needs review » Needs work

The last submitted patch, 6: mollom.endpoint.backport.6.patch, failed testing.

eshta’s picture

Status: Needs work » Needs review
FileSize
12.32 KB

Adding more info to troubleshoot test failures (cause they work locally).

Status: Needs review » Needs work

The last submitted patch, 8: mollom.endpoint.backport.8.patch, failed testing.

eshta’s picture

Status: Needs work » Needs review
FileSize
12.3 KB

Checking with those bots again.

  • eshta committed 54af545 on 6.x-2.x
    Issue #2389943 by eshta: Allow Mollom API endpoint to be configurable.
    
eshta’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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