Merged OS-39-create-user-ticket-comment-classes into master

server/api-test/user/signup.rb

@@ -1,12 +0,0 @@
-describe '/user/signup' do
-  it 'should create user in database' do
-      response = request('/user/signup', {
-          'email' => 'steve@jobs.com',
-          'password' => 'custom'
-      })
-
-      userRow = $database.getRow('users', response['data']['userId'])
-
-      (userRow['email']).should.equal('steve@jobs.com')
-  end
-end

server/controllers/user/signup.php

@@ -19,7 +19,7 @@ class SignUpController extends Controller {
         $userInstance = new User();
         $userInstance->setProperties(array(
             'email' => $email,
-            'password' => User::hashPassword($password)
+            'password' => Hashing::hashPassword($password)
         ));
 
         return $userInstance->store();

server/database.sh

@@ -1,9 +0,0 @@
-#!/bin/bash
-
-# DELETE ALL TABLES
-TABLES=$(mysql -u os_dev -pos_dev os_dev -e "SHOW TABLES IN os_dev;" | awk '{ print $1}' | grep -v '^Tables')
-
-for t in $TABLES
-do
- mysql -u os_dev -pos_dev os_dev -e "DROP TABLE $t"
-done

server/index.php

@@ -10,6 +10,11 @@ RedBean::setup('mysql:host='. $mysql_host .';dbname=' . $mysql_database, $mysql_
 \Slim\Slim::registerAutoLoader();
 $app = new \Slim\Slim();
 
+// LOAD LIBRARIES
+include_once 'libs/Controller.php';
+include_once 'libs/ControllerGroup.php';
+include_once 'libs/Hashing.php';
+
 // LOAD MODELS
 spl_autoload_register(function ($class) {
     $classPath = "models/{$class}.php";
@@ -20,9 +25,6 @@ spl_autoload_register(function ($class) {
 });
 
 // LOAD CONTROLLERS
-include_once 'libs/Controller.php';
-include_once 'libs/ControllerGroup.php';
-
 foreach (glob('controllers/*.php') as $controller) {
     include $controller;
 }

server/libs/FileUploader.php

@@ -0,0 +1,18 @@
+<?php
+class FileUploader {
+    private static $instance = null;
+
+    public static function getInstance() {
+        if (self::$instance === null) {
+            self::$instance = new FileUploader();
+        }
+
+        return self::$instance;
+    }
+
+    private function __construct() {}
+
+    public function upload() {
+        // TODO: Implement file upload features
+    }
+}

server/libs/Hashing.php

@@ -0,0 +1,10 @@
+<?php
+class Hashing {
+    public static function hashPassword($password) {
+        return password_hash($password, PASSWORD_DEFAULT);
+    }
+
+    public static function verifyPassword($password, $hash) {
+        return password_verify($password, $hash);
+    }
+}

server/models/Comment.php

@@ -0,0 +1,19 @@
+<?php
+
+class Comment extends DataStore {
+    const TABLE = 'comments';
+
+    public static function getProps() {
+        return array(
+            'content',
+            'file',
+            'ticket',
+            'author',
+            'date'
+        );
+    }
+
+    protected function getDefaultProps() {
+        return array();
+    }
+}

server/models/DataStore.php

@@ -4,7 +4,7 @@ use RedBeanPHP\Facade as RedBean;
 abstract class DataStore {
     protected $_bean;
 
-    abstract protected function getDefaultProperties();
+    abstract protected function getDefaultProps();
 
     public static function getDataStore($value, $property = 'id') {
         $bean = RedBean::findOne(static::TABLE, static::validateProp($property) . ' =:value', array(
@@ -21,7 +21,7 @@ abstract class DataStore {
         }
         else {
             $this->_bean = RedBean::dispense(static::TABLE);
-            $defaultProperties = $this->getDefaultProperties();
+            $defaultProperties = $this->getDefaultProps();
 
             foreach ($defaultProperties as $PROP => $VALUE) {
                 $this->_bean[$PROP] = $VALUE;
@@ -29,15 +29,9 @@ abstract class DataStore {
         }
     }
 
-    protected static function deleteDataStore($dataStore) {
-        if ($dataStore instanceof DataStore) {
-            RedBean::trash($dataStore->getBeanInstance());
-            unset($dataStore);
-            return true;
-        }
-        else {
-            return false;
-        }
+    public function delete() {
+        RedBean::trash($this->getBeanInstance());
+        unset($this);
     }
 
     public function getBeanInstance() {

server/models/Ticket.php

@@ -0,0 +1,26 @@
+<?php
+
+class Ticket extends DataStore {
+    const TABLE = 'tickets';
+
+    public static function getProps() {
+        return array(
+            'ticketId',
+            'title',
+            'content',
+            'language',
+            'department',
+            'file',
+            'date',
+            'unread',
+            'closed',
+            'author',
+            'owner',
+            'ownComments'
+        );
+    }
+
+    protected function getDefaultProps() {
+        return array();
+    }
+}

server/models/User.php

@@ -3,40 +3,33 @@
 class User extends DataStore {
     const TABLE = 'users';
 
-    public static function hashPassword($password) {
-        return password_hash($password, PASSWORD_DEFAULT);
-    }
-
-    public static function verifyPassword($password, $hash) {
-        return password_verify($password, $hash);
-    }
-
     public static function authenticate($userEmail, $userPassword) {
-        $user = static::getUser($userEmail, 'email');
+        $user = User::getUser($userEmail, 'email');
 
-        return ($user && static::verifyPassword($userPassword, $user->password)) ? $user : null;
+        return ($user && Hashing::verifyPassword($userPassword, $user->password)) ? $user : null;
     }
 
     public static function getProps() {
         return array(
             'email',
-            'password'
+            'password',
+            'name',
+            'verificationToken',
+            'ownTickets'
         );
     }
 
-    public function getDefaultProperties() {
-        return array();
+    public function getDefaultProps() {
+        return array(
+            'ownTickets' => []
+        );
+    }
+
+    public function addTicket($ticket) {
+        $this->ownTickets[] = $ticket;
     }
 
     public static function getUser($value, $property = 'id') {
         return parent::getDataStore($value, $property);
     }
-
-    public static function deleteUser($user) {
-        parent::deleteDataStore($user);
-    }
-
-    public function showUserDetails() {
-        return $this->_user;
-    }
 }

server/tests/models/DataStoreTest.php

@@ -17,16 +17,12 @@ class DataStoreMock extends DataStore {
         );
     }
 
-    public function getDefaultProperties() {
+    public function getDefaultProps() {
         return array(
             'prop1' => 0,
             'prop2' => 'hello'
         );
     }
-
-    public static function deleteDataStore($dataStore) {
-        return parent::deleteDataStore($dataStore);
-    }
 }
 
 class DataStoreTest extends PHPUnit_Framework_TestCase {
@@ -78,9 +74,8 @@ class DataStoreTest extends PHPUnit_Framework_TestCase {
     }
 
     public function testDeleteDataStore() {
-        $beanInstance = $this->instance->getBeanInstance();
-        DataStoreMock::deleteDataStore($this->instance);
+        $this->instance->delete();
 
-        $this->assertTrue(RedBean::get('trash')->hasBeenCalledWithArgs($beanInstance));
+        $this->assertTrue(RedBean::get('trash')->hasBeenCalledWithArgs($this->instance->getBeanInstance()));
     }
 }

tests/clean_db.sh

@@ -0,0 +1,9 @@
+#!/bin/bash
+
+# DELETE ALL TABLES
+TABLES=$(mysql -u root development -e "SHOW TABLES IN os_dev;" | awk '{ print $1}' | grep -v '^Tables')
+
+for t in $TABLES
+do
+ mysql -u root development -e "DROP TABLE $t"
+done

tests/init.rb

@@ -5,6 +5,7 @@ require 'uri'
 require 'mysql'
 require 'json'
 require './libs.rb'
+require './scripts.rb'
 
 # TESTS
 require './user/signup.rb'

tests/run-tests.sh

@@ -0,0 +1,2 @@
+./clean_db.sh
+bacon init.rb

tests/scripts.rb

@@ -7,5 +7,6 @@ class Scripts
 
         if response['status'] === 'fail'
             raise "Could not create user"
+        end
     end
 end

tests/user/login.rb

@@ -22,3 +22,4 @@ describe '/user/login' do
     it 'should fail if already logged in' do
 
     end
+end

tests/user/signup.rb

@@ -0,0 +1,12 @@
+describe '/user/signup' do
+    it 'should create user in database' do
+        response = request('/user/signup', {
+          'email' => 'steve@jobs.com',
+          'password' => 'custom'
+        })
+
+        userRow = $database.getRow('users', response['data']['userId'])
+
+        (userRow['email']).should.equal('steve@jobs.com')
+    end
+end