Merge pull request #17 from nextcloud/fix_15

Delete job if tag not found

1 files changed, 17 insertions, 3 deletions

diff --git a/tests/lib/BackgroundJob/RetentionJobTest.php b/tests/lib/BackgroundJob/RetentionJobTest.php
index 628affb..004079f 100644
--- a/tests/lib/BackgroundJob/RetentionJobTest.php
+++ b/tests/lib/BackgroundJob/RetentionJobTest.php
@@ -20,7 +20,7 @@
  * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  *
  */
-namespace OCA\Files_Retention\Tests\BackgroundJob\RententionJobTest;
+namespace OCA\Files_Retention\Tests\BackgroundJob;
 
 use OCA\Files_Retention\BackgroundJob\RetentionJob;
 use OCA\Files_Retention\Constants;
@@ -36,11 +36,13 @@ use OCP\Files\IRootFolder;
 use OCP\SystemTag\ISystemTagManager;
 use OCP\SystemTag\ISystemTagObjectMapper;
 use OCP\IUser;
+use OCP\SystemTag\TagNotFoundException;
+use Test\TestCase;
 
 /**
  * @group DB
  */
-class RetentionJobTest extends \Test\TestCase {
+class RetentionJobTest extends TestCase {
 
 	/** @var ISystemTagManager|\PHPUnit_Framework_MockObject_MockObject */
 	private $tagManager;
@@ -203,7 +205,7 @@ class RetentionJobTest extends \Test\TestCase {
 		$this->retentionJob->run(['tag' => 42]);
 	}
 
-	public function testNoSuchTag() {
+	public function testInvalidTag() {
 		$this->tagManager->expects($this->once())
 			->method('getTagsByIds')
 			->will($this->throwException(new \InvalidArgumentException()));
@@ -215,6 +217,18 @@ class RetentionJobTest extends \Test\TestCase {
 		$this->retentionJob->run(['tag' => 42]);
 	}
 
+	public function testNoSuchTag() {
+		$this->tagManager->expects($this->once())
+			->method('getTagsByIds')
+			->will($this->throwException(new TagNotFoundException()));
+
+		$this->jobList->expects($this->once())
+			->method('remove')
+			->with($this->equalTo($this->retentionJob), $this->equalTo(['tag' => 42]));
+
+		$this->retentionJob->run(['tag' => 42]);
+	}
+
 	public function testNoSuchRetention() {
 		// Tag exists
 		$this->tagManager->expects($this->once())