Problem/Motivation

Starting May 29, 2016 - 20:10 the automated tests for this module (7.x-2.x-dev version) started failing the GMap.LGFATestCase.

As a result of this, all contributed patches starting failing on what seems to be a module issue.

Proposed resolution

- Identify what was changed on my 29th.
- Update the test accordingly.

CommentFileSizeAuthor
#5 repair_the_failing-2778721-5.patch436 bytesneograph734
#3 repair_the_failing-2778721-3.patch540 bytesneograph734

Comments

Neograph734 created an issue. See original summary.

neograph734’s picture

neograph734
he/him
Location Netherlands
commented
Title: Repair the failing GMap.LGFATestCase » Repair the failing testGmapSaveConfig test
neograph734’s picture

neograph734
he/him
Location Netherlands
commented
Status: Active » Needs review
StatusFileSize
new repair_the_failing-2778721-3.patch540 bytes

This was caused by the 'administer fields' permission that was introduced with Drupal 7.50 (changelog). The test has been updated to also grant this permission to the privileged_user in the test.

Status: Needs review » Needs work

The last submitted patch, 3: repair_the_failing-2778721-3.patch, failed testing.

neograph734’s picture

neograph734
he/him
Location Netherlands
commented
Status: Needs work » Needs review
StatusFileSize
new repair_the_failing-2778721-5.patch436 bytes

Let's try this...

neograph734’s picture

neograph734
he/him
Location Netherlands
commented
Title: Repair the failing testGmapSaveConfig test » [Patch Blocker] - Repair the failing testGmapSaveConfig test
Status: Needs review » Reviewed & tested by the community
mustanggb’s picture

mustanggb commented

Nice find.

mustanggb’s picture

mustanggb commented
Priority: Major » Critical

So this is a blocker for all other patches in the issue queue, bumping priority accordingly.

podarok’s picture

podarok
🇺🇦 he/him/his
Primary language Ukrainian
Location Ukraine
commented

Thanks for the fix
Committing

  • podarok committed cb8a3de on 7.x-2.x authored by Neograph734 
    Issue #2778721 by Neograph734: [Patch Blocker] - Repair the failing...
podarok’s picture

podarok
🇺🇦 he/him/his
Primary language Ukrainian
Location Ukraine
commented
Status: Reviewed & tested by the community » Fixed
mustanggb’s picture

mustanggb commented

Thanks!

Status: Fixed » Closed (fixed)

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