Support from Acquia helps fund testing for Drupal Acquia logo

Comments

naveenvalecha created an issue. See original summary.

naveenvalecha’s picture

Status: Active » Needs review
FileSize
870 bytes

Add test coverage for GoogleCurrencyConverterForm validateFinal Method

Status: Needs review » Needs work

The last submitted patch, 2: 2772235-2.patch, failed testing.

naveenvalecha’s picture

naveenvalecha’s picture

Status: Needs work » Needs review
FileSize
3.49 KB
2.64 KB

Status: Needs review » Needs work

The last submitted patch, 5: 2772235-5.patch, failed testing.

The last submitted patch, 5: 2772235-5.patch, failed testing.

naveenvalecha’s picture

Status: Needs work » Needs review
FileSize
3.67 KB
824 bytes

Added test coverage for block configuration form as subformstate issue gets committed #2537732: PluginFormInterface must have access to the complete $form_state (introduce SubFormState for embedded forms)

Status: Needs review » Needs work

The last submitted patch, 8: 2772235-8.patch, failed testing.

naveenvalecha’s picture

Status: Needs work » Needs review
FileSize
4.16 KB

The patch introduces random failures, not sure why its happening.

Rerunning the tests on core issue as well.

Status: Needs review » Needs work

The last submitted patch, 10: 2772235-10.patch, failed testing.

naveenvalecha’s picture

naveenvalecha’s picture

Status: Needs work » Fixed

Committed #8 to 8.x-1.x

Status: Fixed » Closed (fixed)

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