Adding test suites

.gitignore

@@ -1,4 +1,7 @@
-vendor/
 composer.lock
+vendor/
 material/
-lazarus/backup/
+lazarus/backup/
+bin/*
+!bin/.gitkeep
+phpunit.xml

.travis.yml

@@ -0,0 +1,20 @@
+language: php
+
+php:
+  - 5.6
+  - 7.0
+  - hhvm
+
+before_install:
+  - travis_retry composer self-update
+
+install:
+  - travis_retry composer install --no-interaction --prefer-source --dev
+
+script:
+  - bin/phing
+
+matrix:
+  allow_failures:
+    - php: hhvm
+  fast_finish: true

README.md

@@ -122,6 +122,14 @@ cd lazarus/
 lazbuild phpgui.lpr
 ```
 
+### Test
+
+First install the dependencies, and after you can run:
+
+```bash
+bin/phing
+```
+
 ## TO-DO
 
 The "Issues" page from this repository is being used for TO-DO management, just search for the "to-do" tag.

build.xml

@@ -0,0 +1,28 @@
+<?xml version="1.0" encoding="utf-8"?>
+<project name="php-gui" basedir="." default="main">
+    <property name="examples" value="examples" />
+    <property name="source" value="src" />
+    <property name="bindir" value="bin" />
+
+    <fileset id="php-files" dir="${project.basedir}">
+        <include name="${project.source}/**/*.php"/>
+    </fileset>
+
+    <target name="main" description="Start analyzing our application">
+        <echo msg="Start Build" />
+
+        <phingCall target="phpunit" />
+        <phingCall target="phpcs" />
+
+        <echo msg="Finished Build" />
+    </target>
+
+    <target name="phpunit" description="Run unit tests using PHPUnit and generates junit.xml and clover.xml">
+        <exec passthru="true" command="${bindir}/phpunit" checkreturn="true"/>
+    </target>
+
+    <target name="phpcs" description="Coding Standards Analysis">
+        <exec passthru="true" command="${bindir}/phpcs --standard=PSR2 ${source}" checkreturn="true" />
+        <exec passthru="true" command="${bindir}/phpcs --standard=PSR2 ${examples}" checkreturn="true" />
+    </target>
+</project>

composer.json

@@ -13,9 +13,18 @@
     "require": {
         "react/child-process": "^0.4.0"
     },
+    "require-dev" : {
+        "phing/phing": "2.*",
+        "phpunit/phpunit": "4.5.*",
+        "squizlabs/php_codesniffer": "2.0.*@dev"
+    },
+    "config" : {
+        "bin-dir" : "bin/"
+    },
     "autoload": {
         "psr-4": {
-            "Gui\\": "src/"
+            "Gui\\": "src/",
+            "Test\\": "test/"
         }
     }
 }

examples/01-basic/example.php

@@ -25,13 +25,15 @@
         $button->setValue('Ouch ' . $button->getCounter() . 'x');
         $button->setCounter($button->getCounter() + 1);
 
-        if ($button->getCounter() == 20) {
+        if ($button->getCounter() >= 20) {
             $text = 'Please, stop! You already clicked ' . preg_replace('/[^0-9]/', '', $button->getValue()) . ' times';
             $input->setValue($text);
-        } else if ($button->getCounter() == 30) {
+        }
+
+        if ($button->getCounter() == 30) {
             $button->setVisible(false);
         }
     });
 });
 
-$application->run();
+$application->run();

phpunit.xml.dist

@@ -0,0 +1,31 @@
+<?xml version="1.0" encoding="UTF-8"?>
+
+<phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+         xsi:noNamespaceSchemaLocation="http://schema.phpunit.de/4.1/phpunit.xsd"
+         backupGlobals="false"
+         colors="true"
+         convertErrorsToExceptions="true"
+         convertNoticesToExceptions="true"
+         convertWarningsToExceptions="true"
+         syntaxCheck="true"
+         stopOnFailure="false"
+         bootstrap="vendor/autoload.php">
+    <testsuites>
+        <testsuite name="Test Suite">
+            <directory>test/</directory>
+        </testsuite>
+    </testsuites>
+    <filter>
+        <whitelist addUncoveredFilesFromWhitelist="true">
+            <directory suffix=".php">src</directory>
+            <exclude>
+                <directory suffix="Interface.php">src</directory>
+                <directory suffix="Aware.php">src</directory>
+                <directory suffix="Trait.php">src</directory>
+            </exclude>
+        </whitelist>
+        <blacklist>
+            <directory>vendor</directory>
+        </blacklist>
+    </filter>
+</phpunit>

src/Application.php

@@ -81,10 +81,10 @@ public function run()
         if (OsDetector::isMacOS()) {
             $processName = './phpgui-i386-darwin';
             $processPath = __DIR__ . '/../lazarus/phpgui-i386-darwin.app/Contents/MacOS/';
-        } else if (OsDetector::isUnix()) {
+        } elseif (OsDetector::isUnix()) {
             $processName = './phpgui-x86_64-linux';
             $processPath = __DIR__ . '/../lazarus/';
-        } else if (OsDetector::isWindows()) {
+        } elseif (OsDetector::isWindows()) {
             // @TODO: Windows binary
         } else {
             throw new RuntimeException('Operational System not identified by PHP-GUI.');
@@ -95,10 +95,10 @@ public function run()
         $this->receiver = $receiver = new Receiver($this);
         $this->sender = new Sender($this, $receiver);
 
-        $this->loop->addTimer(0.001, function($timer) use ($process, $application, $receiver) {
+        $this->loop->addTimer(0.001, function ($timer) use ($process, $application, $receiver) {
             $process->start($timer->getLoop());
 
-            $process->stdout->on('data', function($data) use ($receiver) {
+            $process->stdout->on('data', function ($data) use ($receiver) {
                 $receiver->onData($data);
             });
 
@@ -128,4 +128,4 @@ public function waitCommand($method, $params)
 
         return $this->sender->waitReturn($message);
     }
-}
+}

src/Components/Object.php

@@ -40,7 +40,7 @@ public function __construct($defaultAttributes = null, $application = null)
                     'lazarusObjectId' => $this->lazarusObjectId,
                 ]
             ],
-            function($result) use ($object) {
+            function ($result) use ($object) {
                 // Ok, object created
             }
         );
@@ -88,7 +88,7 @@ protected function set($name, $value)
                 $name,
                 $value
             ],
-            function($result) {
+            function ($result) {
                 // Ok, the property changed
             }
         );
@@ -134,7 +134,7 @@ public function on($eventName, $eventHandler)
         $this->application->sendCommand('setObjectEventListener', [
             $this->lazarusObjectId,
             $eventName
-        ], function($result) {
+        ], function ($result) {
             // Ok, the event listener created
         });
 
@@ -332,4 +332,4 @@ public function getLazarusClass()
     {
         return $this->lazarusClass;
     }
-}
+}

src/Ipc/CommandMessage.php

@@ -15,4 +15,4 @@ public function __construct($method, $params, $callback = false)
         $this->params = $params;
         $this->callback = $callback;
     }
-}
+}

src/Ipc/MessageInterface.php

@@ -4,4 +4,4 @@
 
 interface MessageInterface
 {
-}
+}

src/Ipc/Receiver.php

@@ -84,4 +84,4 @@ public function onData($data)
             }
         }
     }
-}
+}

src/Ipc/Sender.php

@@ -102,4 +102,4 @@ private function output($json)
     {
         echo 'Sent: ' . $json . PHP_EOL;
     }
-}
+}

test/ApplicationTest.php

@@ -0,0 +1,17 @@
+<?php
+
+namespace Test;
+
+use Gui\Application;
+
+class ApplicationTest extends \PHPUnit_Framework_TestCase
+{
+    public function testGetNextObjectId()
+    {
+        $application = new Application();
+
+        $this->assertEquals(0, $application->getNextObjectId());
+        $this->assertEquals(1, $application->getNextObjectId());
+        $this->assertEquals(2, $application->getNextObjectId());
+    }
+}