Currently the test are kind of a mess with lots of legacy functions testing the pre-context functionality. Need to go through, cut out the old crap, rewrite functions to handle contexts and generally clean up everything in the tests.
Currently the test are kind of a mess with lots of legacy functions testing the pre-context functionality. Need to go through, cut out the old crap, rewrite functions to handle contexts and generally clean up everything in the tests.
Comments
Comment #1
dave reidFirst step is changing drupalGetSitemap():
1. To use contexts instead of the hard-coded $language parameter.
2. Remove the $regenerate parameter in favor of calling $this->regenerateSitemap() manually.
3. Remove the check for if response was 200 and be more like drupalGet() and not care about the response. Now that we're checking any watchdog messages in tearDown() we'll find any responses that 404'd automatically.
Comment #2
dave reidFirst round of cleaning up (#1) committed:
http://drupal.org/cvs?commit=364454
http://drupal.org/cvs?commit=364456
http://drupal.org/cvs?commit=364460
http://drupal.org/cvs?commit=364462
Comment #3
dave reidNext step: Adding an xmlsitemap_test.module to actually test the context system and adding/editing/deleting sitemaps.
Comment #4
dave reidComment #5
dave reid