Resolves issue #12

Author: peter279k

src/SystemCtl.php

@@ -11,6 +11,7 @@
 use icanhazstring\SystemCtl\Unit\Timer;
 use icanhazstring\SystemCtl\Unit\Socket;
 use icanhazstring\SystemCtl\Unit\Scope;
+use icanhazstring\SystemCtl\Unit\Slice;
 use icanhazstring\SystemCtl\Unit\UnitInterface;
 
 /**
@@ -263,6 +264,38 @@ public function getScopes(?string $unitPrefix = null): array
         }, $units);
     }
 
+    /**
+     * @param string $name
+     *
+     * @return Slice
+     */
+    public function getSlice(string $name): Slice
+    {
+        $units = $this->listUnits($name, [Slice::UNIT]);
+
+        $unitName = $this->searchForUnitInUnits($name, $units);
+
+        if (is_null($unitName)) {
+            throw UnitNotFoundException::create(Slice::UNIT, $name);
+        }
+
+        return new Slice($unitName, $this->getCommandDispatcher());
+    }
+
+    /**
+     * @param null|string $unitPrefix
+     *
+     * @return Slice[]
+     */
+    public function getSlices(?string $unitPrefix = null): array
+    {
+        $units = $this->listUnits($unitPrefix, [Slice::UNIT]);
+
+        return array_map(function ($unitName) {
+            return new Slice($unitName, $this->getCommandDispatcher());
+        }, $units);
+    }
+
     /**
      * Restart the daemon to reload specs and new units
      *

src/Unit/Slice.php

@@ -0,0 +1,24 @@
+<?php
+
+namespace icanhazstring\SystemCtl\Unit;
+
+/**
+ * Class Slice
+ *
+ * @package icanhazstring\SystemCtl\Unit
+ */
+class Slice extends AbstractUnit
+{
+    /**
+     * @var string
+     */
+    public const UNIT = 'slice';
+
+    /**
+     * @inheritdoc
+     */
+    protected function getUnitSuffix(): string
+    {
+        return static::UNIT;
+    }
+}

test/Integration/Unit/UnitTest.php

@@ -14,6 +14,7 @@
 use icanhazstring\SystemCtl\Unit\Timer;
 use icanhazstring\SystemCtl\Unit\Socket;
 use icanhazstring\SystemCtl\Unit\Scope;
+use icanhazstring\SystemCtl\Unit\Slice;
 
 /**
  * Class UnitTest
@@ -149,6 +150,35 @@ public function testScopeCommandsIfProcessIsUnsuccessFulShouldRaiseException():
         $scope->start();
     }
 
+    public function testSliceCommandsIfProcessIsSuccessfulShouldReturnTrue()
+    {
+        $command = $this->prophesize(CommandInterface::class);
+        $command->isSuccessful()->willReturn(true);
+
+        $commandDispatcher = $this->createCommandDispatcherStub();
+        $commandDispatcher->dispatch(Argument::cetera())->willReturn($command);
+
+        $slice = new Slice('AwesomeSlice', $commandDispatcher->reveal());
+
+        $this->assertTrue($slice->start());
+        $this->assertTrue($slice->stop());
+        $this->assertTrue($slice->enable());
+        $this->assertTrue($slice->disable());
+        $this->assertTrue($slice->reload());
+        $this->assertTrue($slice->restart());
+    }
+
+    public function testSliceCommandsIfProcessIsUnsuccessFulShouldRaiseException()
+    {
+        $commandDispatcher = $this->createCommandDispatcherStub();
+        $commandDispatcher->dispatch(Argument::cetera())->willThrow(CommandFailedException::class);
+
+        $slice = new Slice('AwesomeSlice', $commandDispatcher->reveal());
+
+        $this->expectException(CommandFailedException::class);
+        $slice->start();
+    }
+
     public function testDeviceCommandsIfProcessIsSuccessfulShouldReturnTrue()
     {
         $command = $this->prophesize(CommandInterface::class);

test/Unit/SystemCtlTest.php

@@ -15,6 +15,7 @@
 use icanhazstring\SystemCtl\Unit\Timer;
 use icanhazstring\SystemCtl\Unit\Socket;
 use icanhazstring\SystemCtl\Unit\Scope;
+use icanhazstring\SystemCtl\Unit\Slice;
 
 /**
  * Class SystemCtlTest
@@ -178,6 +179,25 @@ public function itShouldCallCommandDispatcherWithListUnitsAndUnitPrefixOnScopeGe
         self::assertEquals($unitName, $scope->getName());
     }
 
+    /**
+     * @test
+     */
+    public function itShouldCallCommandDispatcherWithListUnitsAndUnitPrefixOnSliceGetting()
+    {
+        $unitName = 'testSlice';
+        $output = ' testSlice.slice     Active running';
+        $commandDispatcherStub = $this->buildCommandDispatcherStub();
+        $commandDispatcherStub
+            ->dispatch(...['list-units', $unitName . '*'])
+            ->willReturn($this->buildCommandStub($output));
+
+        $systemctl = (new SystemCtl())->setCommandDispatcher($commandDispatcherStub->reveal());
+
+        $slice = $systemctl->getSlice($unitName);
+        $this->assertInstanceOf(Slice::class, $slice);
+        $this->assertEquals($unitName, $slice->getName());
+    }
+
     /**
      * @test
      */
@@ -229,6 +249,23 @@ public function itShouldThrowAnExceptionIfNoScopeCouldBeFound(): void
         $systemctl->getScope($unitName);
     }
 
+    /**
+     * @test
+     */
+    public function itShouldThrowAnExceptionIfNoSliceCouldBeFound()
+    {
+        $unitName = 'testSlice';
+        $commandDispatcherStub = $this->buildCommandDispatcherStub();
+        $commandDispatcherStub
+            ->dispatch(...['list-units', $unitName . '*'])
+            ->willReturn($this->buildCommandStub(''));
+
+        $systemctl = (new SystemCtl())->setCommandDispatcher($commandDispatcherStub->reveal());
+
+        $this->expectException(UnitNotFoundException::class);
+        $systemctl->getSlice($unitName);
+    }
+
     /**
      * @test
      */

test/Unit/Unit/AbstractUnitTest.php

@@ -58,6 +58,9 @@ public function itShouldReturnCorrectNameDataProvider(): array
             [
                 'name' => 'test1.scope',
             ],
+            [
+                'name' => 'test1.slice',
+            ],
             [
                 'name' => 'test1.mount',
             ],
@@ -127,6 +130,10 @@ public function itDetectsMultiInstanceUnitsCorrectlyDataProvider(): array
                 'name' => 'test1.scope',
                 'isMultiInstance' => false,
             ],
+            [
+                'name' => 'test1.slice',
+                'isMultiInstance' => false,
+            ],
         ];
     }
 
@@ -183,6 +190,10 @@ public function itDetectsMultiInstanceInstanceNamesCorrectlyDataProvider(): arra
                 'name' => 'test1.scope',
                 'instanceName' => null,
             ],
+            [
+                'name' => 'test1.slice',
+                'instanceName' => null,
+            ],
         ];
     }
 

test/Unit/Utils/OutputFetcherTest.php

@@ -44,12 +44,14 @@ public function itDeterminesTheCorrectAmountOfUnitsDataProvider(): array
   nonservice.socket            active running
   nonservice.device            active running
   nonservice.scope             active running
+  nonservice.slice             active running
   superservice.mount           active running
   awesomeservice.mount         active running
   nonservice.timer             active running
   nonservice.socket            active running
   nonservice.device            active running
   nonservice.scope             active running
+  nonservice.slice             active running
   superservice.service         active running
   awesomeservice.service       active running
 ● [email protected]   loaded failed failed
@@ -81,6 +83,11 @@ public function itDeterminesTheCorrectAmountOfUnitsDataProvider(): array
                 'suffix' => 'scope',
                 'amount' => 2,
             ],
+            [
+                'output' => $output,
+                'suffix' => 'slice',
+                'amount' => 2,
+            ],
             [
                 'output' => $output,
                 'suffix' => 'mount',
@@ -120,12 +127,14 @@ public function itOnlyExtractsTheUnitNamesDataProvider(): array
   a-socket.socket          Active running
   a-device.device          Active running
   a-scope.scope          Active running
+  a-slice.slice          Active running
   super.mount        Active running
   awesome.mount      Active running
   nonservice.timer          Active running
   nonservice.socket          Active running
   nonservice.device          Active running
   nonservice.scope          Active running
+  nonservice.slice          Active running
   [email protected]      Active running
   [email protected]    Active running
 ● [email protected] loaded failed failed
@@ -175,6 +184,14 @@ public function itOnlyExtractsTheUnitNamesDataProvider(): array
                     'nonservice',
                 ],
             ],
+            [
+                'output' => $output,
+                'suffix' => 'slice',
+                'units'  => [
+                    'a-slice',
+                    'nonservice',
+                ],
+            ],
             [
                 'output' => $output,
                 'suffix' => 'mount',