From 9113aa33ce0f1ebafcefc5218b44b21ec8c1eb2d Mon Sep 17 00:00:00 2001
From: pkowalczyk <11366345+PabloKowalczyk@users.noreply.github.com>
Date: Sat, 21 Apr 2018 11:07:28 +0200
Subject: [PATCH 1/4] Test autoload from src and bin perspective.

---
 crunz | 22 +++++++++++++++++++---
 1 file changed, 19 insertions(+), 3 deletions(-)

diff --git a/crunz b/crunz
index 684a0063..dd6dd052 100755
--- a/crunz
+++ b/crunz
@@ -13,7 +13,7 @@
 |
 */
 
-$autoloadFile = \implode(
+$autoloadSrc = \implode(
     DIRECTORY_SEPARATOR,
     [
         __DIR__,
@@ -22,12 +22,28 @@ $autoloadFile = \implode(
     ]
 );
 
-require $autoloadFile;
+$autoloadBin = \implode(
+    DIRECTORY_SEPARATOR,
+    [
+        __DIR__,
+        '..',
+        '..',
+        'autoload.php'
+    ]
+);
+
+$hasBinAutoloader = \file_exists($autoloadBin);
+
+if ($hasBinAutoloader) {
+    require_once $autoloadBin;
+} else {
+    require_once $autoloadSrc;
+}
 
 define('CRUNZ_ROOT', __DIR__);
 
 // Setting the base directory as an environment variable
-setbase(getroot($autoloadFile));
+setbase(getroot($hasBinAutoloader ? $autoloadBin : $autoloadSrc));
 
 // Setting the handler for catching/logging fatal and parse errors. 
 Crunz\ErrorHandler::getInstance()->set();

From 23030174781d0dd7efaf9ac7a006c89ba7be740f Mon Sep 17 00:00:00 2001
From: pkowalczyk <11366345+PabloKowalczyk@users.noreply.github.com>
Date: Sat, 21 Apr 2018 12:07:10 +0200
Subject: [PATCH 2/4] Added Vagrant as a development environment.

---
 .gitignore  |  1 +
 Vagrantfile | 63 +++++++++++++++++++++++++++++++++++++++++++++++++++++
 2 files changed, 64 insertions(+)
 create mode 100644 Vagrantfile

diff --git a/.gitignore b/.gitignore
index 24e4a14f..59dbfc3f 100644
--- a/.gitignore
+++ b/.gitignore
@@ -6,3 +6,4 @@ composer.lock
 *.log
 .idea/
 var/
+.vagrant/
diff --git a/Vagrantfile b/Vagrantfile
new file mode 100644
index 00000000..ea63b6df
--- /dev/null
+++ b/Vagrantfile
@@ -0,0 +1,63 @@
+# -*- mode: ruby -*-
+# vi: set ft=ruby :
+
+$script = <<SCRIPT
+
+export DEBIAN_FRONTEND=noninteractive;
+
+PHP_VERSION="5.6";
+
+## Add Onrej PPA for php 5.6 - 7.2
+add-apt-repository -y -u ppa:ondrej/php;
+
+apt-get install git \
+    make \
+    "php$PHP_VERSION" \
+    "php$PHP_VERSION-cli" \
+    "php$PHP_VERSION-common" \
+    "php$PHP_VERSION-zip" \
+    "php$PHP_VERSION-mbstring" \
+    "php$PHP_VERSION-opcache" \
+    "php$PHP_VERSION-xml" \
+    "php$PHP_VERSION-intl" -y
+
+SCRIPT
+
+$composer = <<SCRIPT
+
+COMPOSER_VERSION="1.6.4";
+BIN_DIR="/home/ubuntu/bin";
+COMPOSER_FILE="$BIN_DIR/composer";
+
+mkdir -p "$BIN_DIR" &&
+
+wget -q -O "$COMPOSER_FILE" "https://getcomposer.org/download/$COMPOSER_VERSION/composer.phar" &&
+
+chmod +x "$COMPOSER_FILE";
+
+SCRIPT
+
+Vagrant.configure("2") do |config|
+    config.vm.box = "ubuntu/xenial64"
+
+    config.vm.network :private_network, ip: "192.168.3.25"
+
+    config.vm.synced_folder ".", "/vagrant",
+        mount_options: ["actimeo=1"],
+        nfs: true,
+        linux__nfs_options: ["rw", "no_subtree_check", "all_squash", "async"]
+
+    if Vagrant::Util::Platform.windows? then
+        config.winnfsd.uid = 1000
+        config.winnfsd.gid = 1000
+    end
+
+    config.vm.provider "virtualbox" do |vb|
+        vb.memory = "1024"
+        vb.cpus = 2
+        vb.name = "crunz-dev"
+    end
+
+    config.vm.provision "shell", inline: $script
+    config.vm.provision "shell", inline: $composer
+end

From f67e104748d93d9efbb64e0a437b087d089a9f4f Mon Sep 17 00:00:00 2001
From: pkowalczyk <11366345+PabloKowalczyk@users.noreply.github.com>
Date: Sat, 21 Apr 2018 12:07:32 +0200
Subject: [PATCH 3/4] Added NotImplemented exception.

---
 src/Exception/NotImplementedException.php | 7 +++++++
 1 file changed, 7 insertions(+)
 create mode 100644 src/Exception/NotImplementedException.php

diff --git a/src/Exception/NotImplementedException.php b/src/Exception/NotImplementedException.php
new file mode 100644
index 00000000..352f0ef2
--- /dev/null
+++ b/src/Exception/NotImplementedException.php
@@ -0,0 +1,7 @@
+<?php
+
+namespace Crunz\Exception;
+
+class NotImplementedException extends CrunzException
+{
+}

From 8d94acf84a110cd94d9dd9f5a9b7cda3fdff0f51 Mon Sep 17 00:00:00 2001
From: pkowalczyk <11366345+PabloKowalczyk@users.noreply.github.com>
Date: Sat, 21 Apr 2018 12:08:14 +0200
Subject: [PATCH 4/4] Fixed changing user.

---
 src/Event.php       | 11 ++++++++---
 tests/EventTest.php | 40 ++++++++++++++++++++++++++++++++++++++++
 2 files changed, 48 insertions(+), 3 deletions(-)

diff --git a/src/Event.php b/src/Event.php
index 57c8fd9a..31a8302b 100644
--- a/src/Event.php
+++ b/src/Event.php
@@ -5,6 +5,7 @@
 use Closure;
 use Carbon\Carbon;
 use Cron\CronExpression;
+use Crunz\Exception\NotImplementedException;
 use GuzzleHttp\Client as HttpClient;
 use Symfony\Component\Process\Process;
 use SuperClosure\Serializer;
@@ -210,7 +211,7 @@ public function buildCommand()
         
         if ($this->cwd) {           
             if($this->user) {
-                $command .= $this->sudo();
+                $command .= $this->sudo($this->user);
             }
 
             // Support changing drives in Windows
@@ -221,7 +222,7 @@ public function buildCommand()
         }
     
         if ($this->user) {
-           $this->sudo();
+            $command .= $this->sudo($this->user);
         }
         
         $command .= $this->isClosure() ? $this->serializeClosure($this->command) : $this->command;
@@ -238,7 +239,7 @@ public function buildCommand()
      */
     protected function sudo($user)
     {
-        return 'sudo -u' . $user . ' ';
+        return "sudo -u {$user} ";
     }
 
     /**
@@ -741,6 +742,10 @@ public function timezone($timezone)
      */
     public function user($user)
     {
+        if ($this->isWindows()) {
+            throw new NotImplementedException('Changing user on Windows is not implemented.');
+        }
+
         $this->user = $user;
 
         return $this;
diff --git a/tests/EventTest.php b/tests/EventTest.php
index fa9bb01e..7c02df82 100644
--- a/tests/EventTest.php
+++ b/tests/EventTest.php
@@ -240,6 +240,46 @@ public function onDoNotRunTaskEveryMinute()
         $this->assertSame('0 8 * * * *', $event->getExpression());
     }
 
+    /** @test */
+    public function settingUserPrependSudoToCommand()
+    {
+        $event = new Event($this->id, 'php -v');
+
+        $event->user('john');
+
+        $this->assertSame('sudo -u john php -v', $event->buildCommand());
+    }
+
+    /** @test */
+    public function customUserAndCwd()
+    {
+        if ($this->isWindows()) {
+            $this->markTestSkipped('Required Unix-based OS.');
+        }
+
+        $event = new Event($this->id, 'php -i');
+
+        $event->user('john');
+        $event->in('/var/test');
+
+        $this->assertSame('sudo -u john cd /var/test; sudo -u john php -i', $event->buildCommand());
+    }
+
+    /** @test */
+    public function notImplementedUserChangeOnWindows()
+    {
+        if (!$this->isWindows()) {
+            $this->markTestSkipped('Required Windows OS.');
+        }
+
+        $this->expectException(\Crunz\Exception\NotImplementedException::class);
+        $this->expectExceptionMessage('Changing user on Windows is not implemented.');
+
+        $event = new Event($this->id, 'php -i');
+
+        $event->user('john');
+    }
+
     private function isWindows()
     {
         return DIRECTORY_SEPARATOR === '\\';