fix

Author: VinceG

src/Printers.php

@@ -19,7 +19,7 @@ public function __construct()
 
     public function all()
     {
-        return $this->printers;
+        return collect($this->printers);
     }
 
     public function get($name)
@@ -29,8 +29,12 @@ public function get($name)
 
     protected function buildPrinters()
     {
-        $raw = shell_exec("lpstat -p");
-        
+        $raw = shell_exec("lpstat -p & > /dev/null");
+
+        if(gettype($raw) === 'NULL') {
+            return $this;
+        }
+
         $this->printers = collect(explode("\n", $raw))
             ->reject(function ($value) {
                 return empty($value) || stripos($value, 'unknown') !== false || stripos($value, 'looking') !== false;

tests/PrintersTest.php

@@ -6,7 +6,7 @@
 
 use Printing\Printers;
 use Printing\TestCase;
-use Printing\Exceptions\InvalidOptionException;
+use Tightenco\Collect\Support\Collection;
 
 class PrintersTest extends TestCase
 {
@@ -16,7 +16,7 @@ class PrintersTest extends TestCase
     public function it_returns_all_printers()
     {
         $printers = (new Printers)->all();
-        $this->assertNotEmpty($printers);
+        $this->assertTrue($printers instanceof Collection);
     }
 
     /**
@@ -25,8 +25,14 @@ public function it_returns_all_printers()
     public function it_can_find_a_printer()
     {
         $class = new Printers;
-        $name = $class->all()->first()['name'];
-        
-        $this->assertNotEmpty($class->get($name));
+        $first = $class->all()->first();
+
+        if(is_null($first)) {
+            $this->assertEmpty($class->all()->toArray());
+        } else {
+            $name = $first['name'];
+            
+            $this->assertNotEmpty($class->get($name));
+        }
     }
 }