The module doesn't contain schema definition for module's settings, without the schema definition the tests cannot run successfully.

CommentFileSizeAuthor
#3 2786029_l10n_client-settings-schema-3.patch627 bytessegi
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

segi created an issue. See original summary.

Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy at Acquia commented

Yay, I think you have this on your machine, can you post it? :) Thanks!

segi’s picture

segi CreditAttribution: segi at Cheppers commented
Status: Active » Needs review
FileSize
2786029_l10n_client-settings-schema-3.patch627 bytes

Here it is.

  • Gábor Hojtsy committed e2a8691 on 7.x-1.x 
    Issue #2786029 by segi: Schema definition is missing for module settings
Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy at Acquia commented
Status: Needs review » Fixed

Thanks, looks good. Committed/pushed.

Status: Fixed » Needs work

The last submitted patch, 3: 2786029_l10n_client-settings-schema-3.patch, failed testing.

segi’s picture

segi CreditAttribution: segi at Cheppers commented

Gabor, you committed the schema to the wrong branch.

  • Gábor Hojtsy committed 6381b64 on 7.x-1.x 
    Revert "Issue #2786029 by segi: Schema definition is missing for module...

  • Gábor Hojtsy committed e7b22d6 on 8.x-1.x 
    Issue #2786029 by segi: Schema definition is missing for module settings
Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy at Acquia commented
Status: Needs work » Fixed

Wups, true, sorry. Fixed for good now.

Status: Fixed » Closed (fixed)

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