add user views

LimeSurvey · Nov 6, 2017 · e88b2c9 · e88b2c9
1 parent b1190ab
commit e88b2c9
Show file tree

Hide file tree

Showing 5 changed files with 15 additions and 5 deletions.
diff --git a/application/views/admin/user/editusers.php b/application/views/admin/user/editusers.php
@@ -3,7 +3,7 @@
 /* @var $dataProvider CActiveDataProvider */
 
 // DO NOT REMOVE This is for automated testing to validate we see that page
-echo viewHelper::getViewTestTag('modifyUser');
+echo viewHelper::getViewTestTag('usersIndex');
 
 ?>
 <div class="container-fluid">

diff --git a/application/views/admin/user/modifyuser.php b/application/views/admin/user/modifyuser.php
@@ -3,7 +3,7 @@
 * @var User $oUser
 */
 // DO NOT REMOVE This is for automated testing to validate we see that page
-echo viewHelper::getViewTestTag('usersIndex');
+echo viewHelper::getViewTestTag('modifyUser');
 ?>
 
 <div class="pagetitle h3">

diff --git a/application/views/admin/user/setuserpermissions.php b/application/views/admin/user/setuserpermissions.php
@@ -1,7 +1,12 @@
 <?php
 /**
+ * @var $this AdminController
  * Set user permissions
  */
+
+// DO NOT REMOVE This is for automated testing to validate we see that page
+echo viewHelper::getViewTestTag('setUserPermissions');
+
 ?>
 
 <!-- set user permissions -->

diff --git a/tests/controllers/AdminViewsTest.php b/tests/controllers/AdminViewsTest.php
@@ -92,7 +92,12 @@ public function testSettingsViews($name, $view)
      */
     public function testUsersViews($name,$view){
         // use Admin user
-        $view['route'] = ReplaceFields($view['route'],['{UID}'=>1]);
+        $uid = 1;
+        // non-adminuser for some views
+        if(in_array($name,['setUserPermissions','setUserTemplates'])){
+            $uid = 2;
+        }
+        $view['route'] = ReplaceFields($view['route'],['{UID}'=>$uid]);
         $this->findViewTag($name, $view);
     }
 }
diff --git a/tests/data/views/adminUsersViews.php b/tests/data/views/adminUsersViews.php
@@ -12,6 +12,6 @@
     ['modifyUser', ['route'=>'user/sa/modifyuser/uid/{UID}']],
 
     // needs POST implementation
-    //['modifyUser', ['route'=>'user/sa/setuserpermissions/uid/']],
-    setusertemplates
+    ['setUserPermissions', ['route'=>'user/sa/setuserpermissions/uid/{UID}']],
+    ['setUserTemplates', ['route'=>'user/sa/setusertemplates/uid/{UID}']],
 ];