BUG: refs #0358. Fix folder.create api method and add a test for it

Author: zachmullen

modules/api/controllers/components/ApiComponent.php

@@ -673,7 +673,7 @@ function folderCreate($args)
     $modelLoader = new MIDAS_ModelLoader();
     $folderModel = $modelLoader->loadModel('Folder');
     $name = $args['name'];
-    $description = $args['description'];
+    $description = isset($args['description']) ? $args['description'] : '';
 
     $uuid = isset($args['uuid']) ? $args['uuid'] : '';
     $record = false;
@@ -720,7 +720,7 @@ function folderCreate($args)
       $policyGroup = $folder->getFolderpolicygroup();
       $policyUser = $folder->getFolderpolicyuser();
       $folderpolicygroupModel = $modelLoader->loadModel('Folderpolicygroup');
-      $folderpolicyuserModel = $modelLoader->loadModel('Folderpolicygroup');
+      $folderpolicyuserModel = $modelLoader->loadModel('Folderpolicyuser');
       foreach($policyGroup as $policy)
         {
         $folderpolicygroupModel->createPolicy($policy->getGroup(), $new_folder, $policy->getPolicy());

modules/api/tests/controllers/ApiCallMethodsTest.php

@@ -174,6 +174,27 @@ public function testCommunityList()
     //TODO test that a private community is not returned (requires another community in the data set)
     }
 
+  /** Test creating a folder */
+  public function testFolderCreate()
+    {
+    $usersFile = $this->loadData('User', 'default');
+    $userDao = $this->User->load($usersFile[0]->getKey());
+
+    $this->resetAll();
+    $this->params['token'] = $this->_loginAsAdministrator();
+    $this->params['method'] = 'midas.folder.create';
+    $this->params['name'] = 'testFolderCreate';
+    $this->params['parentid'] = $userDao->getPublicfolderId();
+    $resp = $this->_callJsonApi();
+    $this->_assertStatusOk($resp);
+
+    // Make sure folder was created correctly
+    $this->assertNotEmpty($resp->data->uuid);
+    $this->assertEquals($userDao->getPublicfolderId(), $resp->data->parent_id);
+    $this->assertEquals('testFolderCreate', $resp->data->name);
+    $this->assertEquals('', $resp->data->description);
+    }
+
   /** Test listing of child folders */
   public function testFolderChildren()
     {