Make sure we can store strings as per the interface

1 files changed, 37 insertions, 37 deletions

diff --git a/tests/lib/systemtag/systemtagobjectmappertest.php b/tests/lib/systemtag/systemtagobjectmappertest.php
index 5c8204f6a87..f49917a0265 100644
--- a/tests/lib/systemtag/systemtagobjectmappertest.php
+++ b/tests/lib/systemtag/systemtagobjectmappertest.php
@@ -102,10 +102,10 @@ class SystemTagObjectMapperTest extends TestCase {
 				return $result;
 			}));
 
-		$this->tagMapper->assignTags(1, 'testtype', $this->tag1->getId());
-		$this->tagMapper->assignTags(1, 'testtype', $this->tag2->getId());
-		$this->tagMapper->assignTags(2, 'testtype', $this->tag1->getId());
-		$this->tagMapper->assignTags(3, 'anothertype', $this->tag1->getId());
+		$this->tagMapper->assignTags('1', 'testtype', $this->tag1->getId());
+		$this->tagMapper->assignTags('1', 'testtype', $this->tag2->getId());
+		$this->tagMapper->assignTags('2', 'testtype', $this->tag1->getId());
+		$this->tagMapper->assignTags('3', 'anothertype', $this->tag1->getId());
 	}
 
 	public function tearDown() {
@@ -121,15 +121,15 @@ class SystemTagObjectMapperTest extends TestCase {
 
 	public function testGetTagsForObjects() {
 		$tagIdMapping = $this->tagMapper->getTagIdsForObjects(
-			[1, 2, 3, 4],
+			['1', '2', '3', '4'],
 			'testtype'
 		);
 
 		$this->assertEquals([
-			1 => [$this->tag1->getId(), $this->tag2->getId()],
-			2 => [$this->tag1->getId()],
-			3 => [],
-			4 => [],
+			'1' => [$this->tag1->getId(), $this->tag2->getId()],
+			'2' => [$this->tag1->getId()],
+			'3' => [],
+			'4' => [],
 		], $tagIdMapping);
 	}
 
@@ -140,8 +140,8 @@ class SystemTagObjectMapperTest extends TestCase {
 		);
 
 		$this->assertEquals([
-			1,
-			2,
+			'1',
+			'2',
 		], $objectIds);
 	}
 
@@ -156,29 +156,29 @@ class SystemTagObjectMapperTest extends TestCase {
 	}
 
 	public function testAssignUnassignTags() {
-		$this->tagMapper->unassignTags(1, 'testtype', [$this->tag1->getId()]);
+		$this->tagMapper->unassignTags('1', 'testtype', [$this->tag1->getId()]);
 
-		$tagIdMapping = $this->tagMapper->getTagIdsForObjects(1, 'testtype');
+		$tagIdMapping = $this->tagMapper->getTagIdsForObjects('1', 'testtype');
 		$this->assertEquals([
 			1 => [$this->tag2->getId()],
 		], $tagIdMapping);
 
-		$this->tagMapper->assignTags(1, 'testtype', [$this->tag1->getId()]);
-		$this->tagMapper->assignTags(1, 'testtype', $this->tag3->getId());
+		$this->tagMapper->assignTags('1', 'testtype', [$this->tag1->getId()]);
+		$this->tagMapper->assignTags('1', 'testtype', $this->tag3->getId());
 
-		$tagIdMapping = $this->tagMapper->getTagIdsForObjects(1, 'testtype');
+		$tagIdMapping = $this->tagMapper->getTagIdsForObjects('1', 'testtype');
 
 		$this->assertEquals([
-			1 => [$this->tag1->getId(), $this->tag2->getId(), $this->tag3->getId()],
+			'1' => [$this->tag1->getId(), $this->tag2->getId(), $this->tag3->getId()],
 		], $tagIdMapping);
 	}
 
 	public function testReAssignUnassignTags() {
 		// reassign tag1
-		$this->tagMapper->assignTags(1, 'testtype', [$this->tag1->getId()]);
+		$this->tagMapper->assignTags('1', 'testtype', [$this->tag1->getId()]);
 
 		// tag 3 was never assigned
-		$this->tagMapper->unassignTags(1, 'testtype', [$this->tag3->getId()]);
+		$this->tagMapper->unassignTags('1', 'testtype', [$this->tag3->getId()]);
 
 		$this->assertTrue(true, 'No error when reassigning/unassigning');
 	}
@@ -187,13 +187,13 @@ class SystemTagObjectMapperTest extends TestCase {
 	 * @expectedException \OCP\SystemTag\TagNotFoundException
 	 */
 	public function testAssignNonExistingTags() {
-		$this->tagMapper->assignTags(1, 'testtype', [100]);
+		$this->tagMapper->assignTags('1', 'testtype', [100]);
 	}
 
 	public function testAssignNonExistingTagInArray() {
 		$caught = false;
 		try {
-			$this->tagMapper->assignTags(1, 'testtype', [100, $this->tag3->getId()]);
+			$this->tagMapper->assignTags('1', 'testtype', [100, $this->tag3->getId()]);
 		} catch (TagNotFoundException $e) {
 			$caught = true;
 		}
@@ -201,12 +201,12 @@ class SystemTagObjectMapperTest extends TestCase {
 		$this->assertTrue($caught, 'Exception thrown');
 
 		$tagIdMapping = $this->tagMapper->getTagIdsForObjects(
-			[1],
+			['1'],
 			'testtype'
 		);
 
 		$this->assertEquals([
-			1 => [$this->tag1->getId(), $this->tag2->getId()],
+			'1' => [$this->tag1->getId(), $this->tag2->getId()],
 		], $tagIdMapping, 'None of the tags got assigned');
 	}
 
@@ -214,13 +214,13 @@ class SystemTagObjectMapperTest extends TestCase {
 	 * @expectedException \OCP\SystemTag\TagNotFoundException
 	 */
 	public function testUnassignNonExistingTags() {
-		$this->tagMapper->unassignTags(1, 'testtype', [100]);
+		$this->tagMapper->unassignTags('1', 'testtype', [100]);
 	}
 
 	public function testUnassignNonExistingTagsInArray() {
 		$caught = false;
 		try {
-			$this->tagMapper->unassignTags(1, 'testtype', [100, $this->tag1->getId()]);
+			$this->tagMapper->unassignTags('1', 'testtype', [100, $this->tag1->getId()]);
 		} catch (TagNotFoundException $e) {
 			$caught = true;
 		}
@@ -233,14 +233,14 @@ class SystemTagObjectMapperTest extends TestCase {
 		);
 
 		$this->assertEquals([
-			1 => [$this->tag1->getId(), $this->tag2->getId()],
+			'1' => [$this->tag1->getId(), $this->tag2->getId()],
 		], $tagIdMapping, 'None of the tags got unassigned');
 	}
 
 	public function testHaveTagAllMatches() {
 		$this->assertTrue(
 			$this->tagMapper->haveTag(
-				[1],
+				['1'],
 				'testtype',
 				$this->tag1->getId(),
 				true
@@ -250,7 +250,7 @@ class SystemTagObjectMapperTest extends TestCase {
 
 		$this->assertTrue(
 			$this->tagMapper->haveTag(
-				[1, 2],
+				['1', '2'],
 				'testtype',
 				$this->tag1->getId(),
 				true
@@ -260,7 +260,7 @@ class SystemTagObjectMapperTest extends TestCase {
 
 		$this->assertFalse(
 			$this->tagMapper->haveTag(
-				[1, 2],
+				['1', '2'],
 				'testtype',
 				$this->tag2->getId(),
 				true
@@ -270,7 +270,7 @@ class SystemTagObjectMapperTest extends TestCase {
 
 		$this->assertFalse(
 			$this->tagMapper->haveTag(
-				[2],
+				['2'],
 				'testtype',
 				$this->tag2->getId(),
 				true
@@ -280,7 +280,7 @@ class SystemTagObjectMapperTest extends TestCase {
 
 		$this->assertFalse(
 			$this->tagMapper->haveTag(
-				[3],
+				['3'],
 				'testtype',
 				$this->tag2->getId(),
 				true
@@ -292,7 +292,7 @@ class SystemTagObjectMapperTest extends TestCase {
 	public function testHaveTagAtLeastOneMatch() {
 		$this->assertTrue(
 			$this->tagMapper->haveTag(
-				[1],
+				['1'],
 				'testtype',
 				$this->tag1->getId(),
 				false
@@ -302,7 +302,7 @@ class SystemTagObjectMapperTest extends TestCase {
 
 		$this->assertTrue(
 			$this->tagMapper->haveTag(
-				[1, 2],
+				['1', '2'],
 				'testtype',
 				$this->tag1->getId(),
 				false
@@ -312,7 +312,7 @@ class SystemTagObjectMapperTest extends TestCase {
 
 		$this->assertTrue(
 			$this->tagMapper->haveTag(
-				[1, 2],
+				['1', '2'],
 				'testtype',
 				$this->tag2->getId(),
 				false
@@ -322,7 +322,7 @@ class SystemTagObjectMapperTest extends TestCase {
 
 		$this->assertFalse(
 			$this->tagMapper->haveTag(
-				[2],
+				['2'],
 				'testtype',
 				$this->tag2->getId(),
 				false
@@ -332,7 +332,7 @@ class SystemTagObjectMapperTest extends TestCase {
 
 		$this->assertFalse(
 			$this->tagMapper->haveTag(
-				[3],
+				['3'],
 				'testtype',
 				$this->tag2->getId(),
 				false
@@ -346,7 +346,7 @@ class SystemTagObjectMapperTest extends TestCase {
 	 */
 	public function testHaveTagNonExisting() {
 		$this->tagMapper->haveTag(
-			[1],
+			['1'],
 			'testtype',
 			100
 		);