Add tests for issue creation (including metadata fields)

Oh, how much I'd love to commit this. Sadly, this is weird to my eyes:

A)

+  /**
+   * Create an Issue node.
+   *
+   * @param stdClass $project
+   *   A project node.
+   * @param $edit
+   *   An array of form values, passed to drupalPost. Optional
+   * @return
+   *   An issue node.
+   */
+  function createIssue($project, $edit = array()) {
+    if (empty($edit)) {
+      $project = $this->createProject();
+      $edit['title'] = $this->randomName(8);
+      $edit['body'] = $this->randomName(64);
+      $edit['component'] = 'Code';
+      $edit['category'] = 'bug';
+    }

It doesn't say that $project is optional. Just because $edit is optional doesn't mean that if you don't define it, you should clobber the $project the caller passed in and use your own. I think we should probably just remove that $project = $this->createProject(); line entirely and retain the behavior that $project is required for this function. Thoughts?

B) testProjectCreation() needs to be renamed to testProjectIssueCreation() it seems.

C) Code comment about "Test project node form fields" is wrong.

Thanks!
-Derek

Fixes A-C. mikey_p: do you agree with (A)? If so, set this RTBC and it'll go in. The tests all pass locally with this.

@dmitrig01: I don't think you cvs up'ed project.test from HEAD, since your project.test patch here isn't needed anymore and conflicts. I'm on BART now. When I get to the conf I'll clean this up and commit. ;)

cheers,
-Derek

Cleaned up and reconciled the patches in #6 with what's already in HEAD:

http://drupal.org/cvs?commit=357142
http://drupal.org/cvs?commit=357150

Back to active here, since we really need assertions on the issue metadata fields (Category, Priority, Component, etc).

Thanks y'all, this is so exciting that we're making such great progress!!

Yay, but... ;)

A) assertIssueMetadata() needs a PHPdoc comment

B) the assertions from function testProjectIssueMetadata() could just be merged into testProjectIssueCreation()

C) I can never remember WTF += vs. array_merge() does. Maybe a reminder comment somewhere? ;)

Thanks!

Committed a modified version of this based on some nice in-person debugging here at DCSF (DamZ's Field API talk)...