Merge pull request #239 from dumityty/delete-all-runs

Add link to delete all run from the main paginated list page
perftools · May 29, 2018 · b9abf78 · b9abf78
2 parents bfefcf8 + 223b15c
commit b9abf78
Show file tree

Hide file tree

Showing 4 changed files with 48 additions and 0 deletions.
diff --git a/src/Xhgui/Controller/Run.php b/src/Xhgui/Controller/Run.php
@@ -124,6 +124,21 @@ public function delete()
         $this->app->redirect($redirect);
     }
 
+    public function deleteAll()
+    {
+        $request = $this->app->request();
+
+        // Delete all profile runs.
+        $delete = $this->profiles->truncate();
+
+        $this->app->flash('success', 'Deleted all profiles');
+
+        $referrer = $request->getReferrer();
+        // In case route is accessed directly the referrer is not set.
+        $redirect = isset($referrer) ? $referrer : $this->app->urlFor('home');
+        $this->app->redirect($redirect);
+    }
+
     public function url()
     {
         $request = $this->app->request();

diff --git a/src/routes.php b/src/routes.php
@@ -34,6 +34,10 @@
     $di['runController']->delete();
 })->name('run.delete');
 
+$app->get('/run/delete_all', function () use ($di, $app) {
+    $di['runController']->deleteAll();
+})->name('run.deleteAll');
+
 $app->get('/url/view', function () use ($di, $app) {
     $app->controller = $di['runController'];
     $app->controller->url();

diff --git a/src/templates/runs/list.twig b/src/templates/runs/list.twig
@@ -10,6 +10,10 @@
 
 {% if runs|length or has_search %}
 <div class="searchbar clearfix">
+    <a href="{{ url('run.deleteAll') }}" class="pull-right btn btn-small delete-all" title="Delete all">
+        <i class="icon-trash"></i> Delete all
+    </a>
+
     <a href="#" class="pull-right btn btn-small search-expand" title="Show search form">
         <i class="icon-search"></i> Search
     </a>

diff --git a/tests/Controller/RunTest.php b/tests/Controller/RunTest.php
@@ -192,4 +192,29 @@ public function testDelete()
         $result = $this->profiles->getAll();
         $this->assertCount(4, $result['results']);
     }
+
+    public function testDeleteAll()
+    {
+        loadFixture($this->profiles, XHGUI_ROOT_DIR . '/tests/fixtures/results.json');
+
+        Environment::mock(array(
+          'SCRIPT_NAME' => 'index.php',
+          'PATH_INFO' => '/run/delete_all',
+        ));
+
+        $this->app->expects($this->once())
+          ->method('urlFor')
+          ->with('home');
+
+        $this->app->expects($this->once())
+          ->method('redirect');
+
+        $result = $this->profiles->getAll();
+        $this->assertCount(5, $result['results']);
+
+        $this->runs->deleteAll();
+
+        $result = $this->profiles->getAll();
+        $this->assertCount(0, $result['results']);
+    }
 }