Replace DrupalWebTestCase browser with browser.inc
boombatower - August 19, 2009 - 07:00
| Project: | SimpleTest |
| Version: | 7.x-2.x-dev |
| Component: | Code |
| Category: | task |
| Priority: | normal |
| Assigned: | boombatower |
| Status: | needs review |
Description
Has some catch up work to do in terms of features, at least #553276: Add HTTP authentication support, not sure what else.
#1
There is a patch for the issue...so I am going to start on this.
Items to accomplish:
The current code:
<?php$this->drupalGet('node');
?>
The new format for tests will look something like this:
<?php$this->browser->get(url('node', array('absolute' => TRUE));
?>
or
<?php$this->get('node');
?>
or
<?php$this->browserGet('node');
?>
That latter two being wrappers for $browser->get() and will run url() on anything passed to them, just like drupalGet(). The browser requires an absolute url.
That or we make the browser accept none-absolute in some manor.
#2
Smart idea from webchick, leave as drupalGet() for now.
#3
Additional dependencies:
#555374: Deal with !empty($curl_options[CURLOPT_POST]) in browser.inc
#555412: Add page listener facility to browser.inc
#4
Very early patch.
#5
Move to Browser module for further development.
#6
Move to SimpleTest.
#7
Update for tonight.
#8
#9
drupalGet() and drupalPost() work!
Left:
* Convert page related assertions.
* Provide drupalHeader() equivalent and related.
* Create Drupal HEAD patch to let bot run on all tests.