Reroll of 108

src/Commands/Send.php

@@ -83,6 +83,16 @@ protected function configure() {
    * {@inheritdoc}
    */
   protected function execute(InputInterface $input, OutputInterface $output) {
+    // Check for any open time logs first.
+    if ($open = $this->repository->getActive()) {
+      $details = $this->connector->ticketDetails($open->getTicketId(), $open->getConnectorId());
+      $output->writeLn(sprintf('<error>Please stop open time logs first: %s [<info>%d</info>]</error>',
+        $details->getTitle(),
+        $open->getTicketId()
+      ));
+      return;
+    }
+
     // Find any untagged items needing review, use an arbitrarily early date.
     $review = $this->reviewer->getSummary(static::ALL, TRUE);
     if (count($review) > 2) {

tests/Commands/SendTest.php

@@ -0,0 +1,38 @@
+<?php
+/**
+ * @file
+ * Contains \Larowlan\Tl\Tests\Commands\SendTest.php
+ */
+
+namespace Larowlan\Tl\Tests\Commands;
+
+use Larowlan\Tl\Tests\TlTestBase;
+use Larowlan\Tl\Ticket;
+
+/**
+ * @coversDefaultClass \Larowlan\Tl\Commands\Send
+ * @group Commands
+ */
+class SendTest extends TlTestBase {
+
+  /**
+   * @covers ::execute
+   */
+  public function testSend() {
+    $this->setupConnector();
+    // Start time log on a ticket.
+    $this->executeCommand('start', [
+      'issue_number' => 1234,
+      'comment' => 'Foo bar',
+    ]);
+    $this->getRepository()->tag(9, 1234);
+    $this->assertTicketIsOpen(1234, 'Foo bar');
+    $this->getMockConnector()->expects($this->any())
+      ->method('ticketDetails')
+      ->with(1234)
+      ->willReturn(new Ticket('Running tests', 1234));
+    $output = $this->executeCommand('send');
+    $this->assertEquals('Please stop open time logs first: Running tests [1234]', trim($output->getDisplay()));
+  }
+
+}