No need to stay on the import form.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

rszrama created an issue. See original summary.

mitrpaka’s picture

Status: Active » Needs review
FileSize
542 bytes
mglaman’s picture

Great! Made a comment on the PR. It'd be great to add a test assertion that the collection route is the current page after submitting the form.

  public function testCurrencyImport() {
    $this->drupalGet('admin/commerce/config/currency/import');
    $edit = [
      'currency_codes[]' => ['CHF'],
    ];
    $this->submitForm($edit, 'Import');
    // INSERT ROUTE CHECK HERE
    $currency = Currency::load('CHF');
mitrpaka’s picture

CurrencyTest::testCurrencyImport test extended to check that he collection route is the current page after submitting the form. PR updated accordingly.

mglaman’s picture

Status: Needs review » Fixed

Committed in acae42b

Status: Fixed » Closed (fixed)

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