add some tests for sandbox

swooletw · Dec 13, 2018 · 781773d · 781773d
1 parent b996e36
commit 781773d
Show file tree

Hide file tree

Showing 2 changed files with 25 additions and 8 deletions.
diff --git a/src/Server/Sandbox.php b/src/Server/Sandbox.php
@@ -323,14 +323,6 @@ public function getSnapshot()
         return Context::getApp();
     }
 
-    /**
-     * Remove current request.
-     */
-    public function removeRequest()
-    {
-        return Context::removeData('_request');
-    }
-
     /**
      * Get current request.
      */

diff --git a/tests/Server/SandboxTest.php b/tests/Server/SandboxTest.php
@@ -99,6 +99,31 @@ public function testGetApplication()
         $this->assertTrue($sandbox->getSnapshot() instanceof Container);
     }
 
+    public function testGetCachedSnapshot()
+    {
+        $container = m::mock(Container::class);
+        $snapshot = m::mock(Container::class);
+        $snapshot->shouldReceive('offsetGet')
+            ->with('foo')
+            ->once()
+            ->andReturn($result = 'bar');
+
+        $sandbox = new Sandbox;
+        $sandbox->setBaseApp($container);
+        $sandbox->setSnapshot($snapshot);
+
+        $this->assertTrue($sandbox->getApplication() instanceof Container);
+        $this->assertEquals($result, $sandbox->getApplication()->foo);
+    }
+
+    public function testRunWithoutSnapshot()
+    {
+        $this->expectException(SandboxException::class);
+
+        $sandbox = new Sandbox;
+        $sandbox->run($request = m::mock(Request::class));
+    }
+
     public function testSetRequest()
     {
         $request = m::mock(Request::class);