Merge pull request #79 from captbaritone/issue/78-support-for-https-urls

HTTPS URL support for `grabUrlsFromLastEmail` with tests

src/Module/MailCatcher.php

@@ -279,15 +279,16 @@ public function grabFromLastEmailTo(string $address, string $regex): string
      */
     public function grabUrlsFromLastEmail(): array
     {
+        $regex = '#\bhttps?://[^,\s()<>]+(?:\([\w\d]+\)|([^,[:punct:]\s]|/))#';
         $email = $this->lastMessage();
 
         $message = Message::from($email->getSource());
 
         $text = $message->getTextContent();
-        preg_match_all('#\bhttp?://[^,\s()<>]+(?:\([\w\d]+\)|([^,[:punct:]\s]|/))#', $text, $text_matches);
+        preg_match_all($regex, $text, $text_matches);
 
         $html = $message->getHtmlContent();
-        preg_match_all('#\bhttp?://[^,\s()<>]+(?:\([\w\d]+\)|([^,[:punct:]\s]|/))#', $html, $html_matches);
+        preg_match_all($regex, $html, $html_matches);
 
         return array_merge($text_matches[0], $html_matches[0]);
     }

tests/acceptance/MailcatcherCest.php

@@ -126,6 +126,17 @@ public function test_grab_from_last_email_to(AcceptanceTester $I)
      * @example ["http://localhost.com/index.php?auth&token=123"]
      * @example ["http://localhost.com/index.php?auth&id=12&token=123"]
      * @example ["http://example.com/list.php?page=56"]
+     *
+     * @example ["https://localhost"]
+     * @example ["https://localhost/"]
+     * @example ["https://localhost.com"]
+     * @example ["https://localhost.com/"]
+     * @example ["https://localhost.com/index.html"]
+     * @example ["https://localhost.com/index.php"]
+     * @example ["https://localhost.com/index.php?token=123"]
+     * @example ["https://localhost.com/index.php?auth&token=123"]
+     * @example ["https://localhost.com/index.php?auth&id=12&token=123"]
+     * @example ["https://example.com/list.php?page=56"]
      */
     public function test_grab_urls_from_last_email(
         AcceptanceTester $I,