The reject is:

--- 503,526 ----
            'line' => $method_info->getStartLine(),
            'function' => $class . '->' . $method . '()',
          );
+ 
+         // Run test in each browser.
+         foreach ($info['browsers'] as $browser) {
+           $this->browser = $browser;
+ 
+           $completion_check_id = DrupalTestCase::insertAssert($this->testId, $class, FALSE, t('The test did not complete due to a fatal error.'), 'Completion check', $caller);
+           try {
+             $this->setUp();
+             $this->$method();
+             // Finish up.
+           }
+           catch (Exception $e) {
+             $this->exceptionHandler($e);
+           }
+           $this->tearDown();
+           // Remove the completion check record.
+           DrupalTestCase::deleteAssert($completion_check_id);
          }
        }
      }

This hunk fails with both 7.12 and the latest dev of Drupal.

And, thanks for this module, it's exactly what I was looking for.

Comments

alberto56’s picture

By the way, the module still works with Firefox.

alberto56’s picture

Status: Active » Needs review
StatusFileSize
new3.82 KB

So here is a patch that can be applied to the selenium module. It updates the patch included in the selenium module, called D7-core-selenium.patch, so that that patch can apply cleanly to the latest dev version of Drupal 7.

alberto56’s picture

Title: Core patch: 1 out of 3 hunks FAILED with Drupal 7.12 and dev » Update core patch for latest dev version of Drupal, to avod Hunk Failed error
ygerasimov’s picture

Very important point is missing in patch #2 that we expect $this->setUp() to throw exception. I have added try - catch logic. See attached patch.

alberto56’s picture

Note that this patch is not necessary with the 7.x-3.x branch, only 7.x-2.x