Minor tweaks to bin/mail send-test

Summary: Clean up some arg handling stuff.

Test Plan: Used this while debugging.

Reviewers: btrahan

Reviewed By: btrahan

Subscribers: epriestley

Differential Revision: https://secure.phabricator.com/D10314

Author: epriestley

src/applications/metamta/management/PhabricatorMailManagementSendTestWorkflow.php

@@ -79,6 +79,13 @@ public function execute(PhutilArgumentParser $args) {
     $tos = $args->getArg('to');
     $ccs = $args->getArg('cc');
 
+    if (!$tos && !$ccs) {
+      throw new PhutilArgumentUsageException(
+        pht(
+          'Specify one or more users to send mail to with `--to` and '.
+          '`--cc`.'));
+    }
+
     $names = array_merge($tos, $ccs);
     $users = id(new PhabricatorPeopleQuery())
       ->setViewer($viewer)
@@ -103,14 +110,17 @@ public function execute(PhutilArgumentParser $args) {
     }
 
     $subject = $args->getArg('subject');
+    if ($subject === null) {
+      $subject = pht('No Subject');
+    }
+
     $tags = $args->getArg('tag');
     $attach = $args->getArg('attach');
     $is_bulk = $args->getArg('bulk');
 
     $console->writeErr("%s\n", pht('Reading message body from stdin...'));
     $body = file_get_contents('php://stdin');
 
-
     $mail = id(new PhabricatorMetaMTAMail())
       ->addTos($tos)
       ->addCCs($ccs)