TestingParty08: drupal_http_request redirects need a test

#296307: TestingParty08: drupal_http_request basic auth needs a test already sets up a dummy module for drupal_http_request. That patch can probably be used as a starting point for the current issue.

Few very minor things:

• Missing $Id$ at top of module file.
• Added comment: Implementation of setUp(). to added setUp() method.
• Core now uses uses spaces on both sides of dot.
• Removed trailing white-space and on blank lines.
• Parameters to function usually not split up on separate lines when they are small.

Test looks good, ran test and it passes.

This patch includes the test by flobruit and nickske from: #296307: TestingParty08: drupal_http_request basic auth needs a test which I have marked as a duplicate.

the .module should be:

// $Id$

/**
 * Implementation of hook_menu().
 */
function drupal_http_request_test_menu() {
  $items = array();

  $items['drupal_http_request_test'] = array(
    'page callback' => 'drupal_http_request_test_basic_auth_page',
    'access callback' => TRUE,
    'type' => MENU_CALLBACK,
  );

  $items['drupal_http_request_test/%'] = array(
    'title' => t('Redirect'),
    'page callback' => 'drupal_http_request_test',
    'access arguments' => array('access content'),
    'type' => MENU_CALLBACK,
  );

  return $items;
}

function drupal_http_request_test_basic_auth_page() {
  $output = t('$_SERVER[\'PHP_AUTH_USER\'] is @username.', array('@username' => $_SERVER['PHP_AUTH_USER']));
  $output .= t('$_SERVER[\'PHP_AUTH_PW\'] is @password.', array('@password' => $_SERVER['PHP_AUTH_PW']));
  return $output;
}

function drupal_http_request_test($code) {
  if ($code != 200) {
    header("Location: " . url('drupal_http_request_test/200', array('absolute' => TRUE)), TRUE, $code);
    exit;
  }
  return '';
}

Sure, that is cleaner. Missing 'page arguments', but otherwise it works.

* I'm not sure we should create a dedicated module for this test. It feels like it would be better to create a more generic system_test.module or something along these lines.

* We don't use underscores in URLs. Use dashes instead, and consider to shorten the URLs.

Extended tests with parameter "$retry" set to "0".

Added redirect to invalid URL tests.

Tests look thorough, pass, and now use a standard system_test.module.

Looks like good work.

Test results:

19 passes, 0 fails, 0 exceptions

Excellent. Thanks for incorporating my feedback so quickly. Committed to CVS HEAD. Thanks ...

I get unresponsive script errors whenever I run this test in FF3 on OSX. Didn't get a chance to look more closely at it yet.

I looked in the simpletest table in the database. This is the last row to get inserted before it times out:

"The query passed to the page is correctly represented by drupal_get_detination()."

It's very strange, because that's the last assertion in common.test. I tried commenting its parent function, testDrupalGetDestination(), out but no dice.

Runs fine FF3 SUSE 11.0 (linux).

I'll try digging in and debugging this tonight. Good to know Linux is unaffected. I'll see if I can scrounge up some Mac users to test.

http://drupal.org/node/250047#comment-996555

Oh, god, that test was *silly*.

  $this->assertEqual($redirect_301->redirect_code, 200, t('drupal_http_request follows the 301 redirect.'));

Come on! That's a "if there is a bug, print that there is none".

@webchick: what you were experiencing are notices in drupal_http_request that broke the batch API somehow.

This patch fixes both drupal_http_request and the test.

Committed. Love it when tests find bugs. Yay! :)

Sweet!

This fix was wrong and should be reverted. The real fix to the bug is here: http://drupal.org/node/293529#comment-960905

Let's this issue die peacefully. I'll answer in the other one.