CVS info incorrect for contrib modules

FYI #666258: Testing PIFR on 7.x-2.x branch is the empty patch for testing I put in XML sitemap 7.x-2.x.

Discovered the problem is that all projects not Drupal core need the 'modules' information from the CVS repository settings (in contribs case, its the /contributions path). Patch that uses the modules information for non-core projects. Also fixes a few things in the tests, but as boombatower said on IRC they're expected to be broken right now. Just matched some things up to how they actually are on drupal.org with the test data.

Tested out pift_cron_queue_batch_build_projects locally with 'duplicate' projects for core and XML sitemap contrib. This is the data that returned:

define('PIFT_PID', 29);

array(
  2 => 
  array (
    'client_identifier' => '29',
    'name' => 'Drupal',
    'repository_type' => 'cvs',
    'repository_url' => ':pserver:anonymous:anonymous@cvs.drupal.org:/cvs/drupal/drupal/',
    'link' => 'http://mysql.drupal6dev.local/content/drupal',
  ),
  3 => 
  array (
    'client_identifier' => '30',
    'name' => 'XML sitemap',
    'repository_type' => 'cvs',
    'repository_url' => ':pserver:anonymous:anonymous@cvs.drupal.org:/cvs/drupal-contrib/contributions/modules/xmlsitemap/',
    'link' => 'http://mysql.drupal6dev.local/content/xml-sitemap',
  ),
);

So everything looks good I think.

Follow-up un-break things patch.

This version adds an rtrim([cvsurl], '/'). Either this or #10 should work.

Yeah, sorry about that. Marking back for deployment of #11.