From 7fbd9b8de4d59ba63ae4c7651a1f3d0e308532ed Mon Sep 17 00:00:00 2001
From: Shuhei Tanuma <chobieee@gmail.com>
Date: Thu, 24 Dec 2015 15:35:28 +0900
Subject: [PATCH] (PHP) improve indirect buffer test

---
 tests/generate_code.bat         |   1 -
 tests/generate_code.sh          |   1 -
 tests/monsterdata_indirect.json |   6 ------
 tests/monsterdata_indirect.mon  | Bin 156 -> 0 bytes
 tests/monsterdata_test.json     |   3 +++
 tests/monsterdata_test.mon      | Bin 352 -> 384 bytes
 tests/phpTest.php               |  26 ++++++++++++++++++++------
 7 files changed, 23 insertions(+), 14 deletions(-)
 delete mode 100644 tests/monsterdata_indirect.json
 delete mode 100644 tests/monsterdata_indirect.mon

diff --git a/tests/generate_code.bat b/tests/generate_code.bat
index c46d10af6..75ac1433e 100644
--- a/tests/generate_code.bat
+++ b/tests/generate_code.bat
@@ -1,3 +1,2 @@
 ..\flatc.exe -c -j -n -g -b -p --php -s --gen-mutable --no-includes monster_test.fbs monsterdata_test.json
 ..\flatc.exe -b --schema monster_test.fbs
-..\flatc.exe -b .\monster_test.fbs .\monsterdata_indirect.json
\ No newline at end of file
diff --git a/tests/generate_code.sh b/tests/generate_code.sh
index de736da5c..91661f30d 100644
--- a/tests/generate_code.sh
+++ b/tests/generate_code.sh
@@ -1,3 +1,2 @@
 ../flatc --cpp --java --csharp --go --binary --python --js --php --gen-mutable --no-includes monster_test.fbs monsterdata_test.json
 ../flatc --binary --schema monster_test.fbs
-../flatc --binary monster_test.fbs monsterdata_indirect.json
\ No newline at end of file
diff --git a/tests/monsterdata_indirect.json b/tests/monsterdata_indirect.json
deleted file mode 100644
index 297c1721f..000000000
--- a/tests/monsterdata_indirect.json
+++ /dev/null
@@ -1,6 +0,0 @@
-{
-  "name": "Gob",
-  "enemy": {
-    "name": "Awk"
-  }
-}
\ No newline at end of file
diff --git a/tests/monsterdata_indirect.mon b/tests/monsterdata_indirect.mon
deleted file mode 100644
index f6262f32d7ef8ca17f248065de056846ffb8b24f..0000000000000000000000000000000000000000
GIT binary patch
literal 0
HcmV?d00001

literal 156
zcmZ=@U|{g|_X}oVFk|2YvO$0aLLreHNNiN58IT(Q#5Q1IGX|JSOfyjRA@e|HfYdVs
Ov155Qn0C)kVgLYoBLz(W

diff --git a/tests/monsterdata_test.json b/tests/monsterdata_test.json
index 7ed39a8de..89278755e 100755
--- a/tests/monsterdata_test.json
+++ b/tests/monsterdata_test.json
@@ -37,6 +37,9 @@
     "test1",
     "test2"
   ],
+  enemy: {
+    name: "Fred"
+  },
   testarrayofbools:[
     true, false, true
   ],
diff --git a/tests/monsterdata_test.mon b/tests/monsterdata_test.mon
index eff1e66ca2a5b60af52b6deeffcf0eaba91dd00d..69d1d7538e406a7d46bc5d85bbb565ea95e960ca 100644
GIT binary patch
delta 145
zcmaFB)WAGJ#JGilgMop8jll#+S^zOCgAIcNgA0)50mLBS!w|rbz>vWZ!Vtkwz)-;u
zGtu2p7bNioh@SxQ86a)};us(XX>>eUc^8BWmo)Z)u*Sr>%JM*Q24)~;1j1ea|NmzJ
QvO(CbC^ZF0O*}pm06JwJZ~y=R

delta 98
zcmZo*e!w(A#JGZigMop8jllp&ngB5?g9U>Pg9DHS11=063=s?o3_c723>gds3?UQU
l4FzuiWlsR{1|ZG=Vuy(ng&9>QE>&h{2J#pg7$-iT2>_pL56%Dp

diff --git a/tests/phpTest.php b/tests/phpTest.php
index 027780cc1..ab41121ce 100644
--- a/tests/phpTest.php
+++ b/tests/phpTest.php
@@ -30,6 +30,10 @@ function main()
 
     // We set up the same values as monsterdata.json:
     $str = $fbb->createString("MyMonster");
+    $name = $fbb->createString('Fred');
+    \MyGame\Example\Monster::startMonster($fbb);
+    \MyGame\Example\Monster::addName($fbb, $name);
+    $enemy = \MyGame\Example\Monster::endMonster($fbb);
 
     $inv = \MyGame\Example\Monster::CreateInventoryVector($fbb, array(0, 1, 2, 3, 4));
 
@@ -62,6 +66,7 @@ function main()
     \MyGame\Example\Monster::AddTest($fbb, $mon2);
     \MyGame\Example\Monster::AddTest4($fbb, $test4);
     \MyGame\Example\Monster::AddTestarrayofstring($fbb, $testArrayOfString);
+    \MyGame\Example\Monster::AddEnemy($fbb, $enemy);
     \MyGame\Example\Monster::AddTestbool($fbb, false);
     $mon = \MyGame\Example\Monster::EndMonster($fbb);
 
@@ -75,7 +80,7 @@ function main()
 //    testUnicode($assert);
 
 
-    testIndirectBuffer($assert);
+    testAnyBuffer($assert);
 
     echo 'FlatBuffers php test: completed successfully' . PHP_EOL;
 }
@@ -135,6 +140,10 @@ function test_buffer(Assert $assert, Google\FlatBuffers\ByteBuffer $bb) {
     $assert->strictEqual($monster->GetTestarrayofstringLength(), 2);
     $assert->strictEqual($monster->GetTestarrayofstring(0), 'test1');
     $assert->strictEqual($monster->GetTestarrayofstring(1), 'test2');
+
+    $fred = $monster->getEnemy();
+    $assert->Equal('Fred', $fred->getName());
+
     $assert->strictEqual($monster->GetTestbool(), false);
 }
 
@@ -590,15 +599,20 @@ function testByteBuffer(Assert $assert) {
     $assert->Equal(0x0D0C0B0A, $uut->readLittleEndian(0, 4, true));
 
 }
-function testIndirectBuffer(Assert $assert)
+
+function testAnyBuffer(Assert $assert)
 {
-    $js = json_decode(file_get_contents('monsterdata_indirect.json'), true);
-    $data = file_get_contents('monsterdata_indirect.mon');
+//    PHP needs double quote. for now, use Fred directly
+//    $js = json_decode(file_get_contents('monsterdata_test.json'), true);
+    $data = file_get_contents('monsterdata_test.mon');
     $bb = Google\FlatBuffers\ByteBuffer::wrap($data);
     $mons = \MyGame\Example\Monster::getRootAsMonster($bb);
-    $assert->Equal($js["name"], $mons->getName());
-    $assert->Equal($js["enemy"]["name"], $mons->getEnemy()->getName());
+    $indirect_monster = new \MyGame\Example\Monster();
+    $assert->Equal($mons->getTestType(), \MyGame\Example\Any::Monster);
+    $mons->getTest($indirect_monster);
+    $assert->Equal("Fred", $indirect_monster->getName());
 }
+
 class Assert {
     public function ok($result, $message = "") {
         if (!$result){