Fixes #10969: Verify that closed issues are retrieved by mc_project_g…

…et_issues

Following a bug report which hints that mc_project_get_issues may be ignoring
closed issues, two tests are introduced:

- verify that a newly created issue is retrieved by mc_project_get_issues
  with the correct id;
- verify that a newly created issue with closed status is retrieved by
  mc_project_get_issues;

tests/soap/FilterTest.php

@@ -0,0 +1,103 @@
+<?php
+# MantisBT - a php based bugtracking system
+
+# MantisBT is free software: you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation, either version 2 of the License, or
+# (at your option) any later version.
+#
+# MantisBT is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with MantisBT.  If not, see <http://www.gnu.org/licenses/>.
+
+/**
+ * @package Tests
+ * @subpackage UnitTests
+ * @copyright Copyright (C) 2002 - 2009  MantisBT Team - mantisbt-dev@lists.sourceforge.net
+ * @link http://www.mantisbt.org
+ */
+
+require_once 'SoapBase.php';
+
+/**
+ * Test fixture for filter related webservice method.
+ */
+class FilterTest extends SoapBase {
+	/**
+	 * A test case that tests the following:
+	 * 1. Retrieving all the project's issues
+	 * 2. Creating an issue
+	 * 3. Retrieving all the project's issues
+	 * 4. Verifying that one extra issue is found in the results
+	 * 5. Verifying that the first returned issue is the one we have submitted
+	 */
+	public function testGetProjectIssues() {
+
+		$initialIssues = $this->getProjectIssues();
+
+		$issueToAdd = $this->getIssueToAdd( 'FilterTest.getProjectIssues' );
+
+		$issueId = $this->client->mc_issue_add(
+			$this->userName,
+			$this->password,
+			$issueToAdd);
+
+		$projectIssues = $this->getProjectIssues();
+
+		$this->assertEquals( 1, count( $projectIssues ) - count( $initialIssues ), "count(projectIssues) - count(initialIssues)");
+		$this->assertEquals( $issueId, $projectIssues[0]->id, "issueId");
+
+		$this->client->mc_issue_delete(
+			$this->userName,
+			$this->password,
+			$issueId);
+	}
+
+	/**
+	 * A test case that tests the following:
+	 * 1. Retrieving all the project's issues
+	 * 2. Creating an issue with status = closed and resolution = fixed
+	 * 3. Retrieving all the project's issues
+	 * 4. Verifying that one extra issue is found in the results
+	 */
+	public function testGetProjectClosedIssues() {
+
+		$initialIssues = $this->getProjectIssues();
+
+		$issueToAdd = $this->getIssueToAdd( 'FilterTest.testGetProjectClosedIssues' );
+		$issueToAdd['status'] = 'closed';
+		$issueToAdd['resolution'] = 'fixed';
+
+		$issueId = $this->client->mc_issue_add(
+			$this->userName,
+			$this->password,
+			$issueToAdd);
+
+		$projectIssues = $this->getProjectIssues();
+
+		$this->assertEquals( 1, count( $projectIssues ) - count( $initialIssues ), "count(projectIssues) - count(initialIssues)");
+
+		$this->client->mc_issue_delete(
+			$this->userName,
+			$this->password,
+			$issueId);
+	}
+
+	/**
+	 *
+	 * @return Array the project issues
+	 */
+	private function getProjectIssues() {
+
+		return $this->client->mc_project_get_issues(
+			$this->userName,
+			$this->password,
+			$this->getProjectId(),
+			0,
+			50);
+	}
+}