Improving the groups and adding mbstring Required tests

test/PHPMailer/PHPMailerTest.php

@@ -1346,6 +1346,8 @@ EOT;
     /**
      * @requires extension mbstring
      * @requires function idn_to_ascii
+     * 
+     * @group mbstringExtRequired
      */
     public function testGivenIdnAddress_addAddress_returns_true()
     {

test/PHPMailer/ParseAddressesTest.php

@@ -33,6 +33,7 @@ final class ParseAddressesTest extends TestCase
      * with the Mbstring extension available.
      *
      * @requires extension mbstring
+     * @group mbstringExtRequired
      *
      * @dataProvider dataAddressSplitting
      *
@@ -64,6 +65,7 @@ final class ParseAddressesTest extends TestCase
      *
      * @requires extension imap
      * @requires extension mbstring
+     * @group mbstringExtRequired
      *
      * @dataProvider dataAddressSplitting
      *
@@ -93,7 +95,7 @@ final class ParseAddressesTest extends TestCase
      * Test RFC822 address splitting using the PHPMailer native implementation
      * without the Mbstring extension.
      *
-     * @group nombstring
+     * @group mbstringExtDisabled
      * @dataProvider dataAddressSplitting
      *
      * @param string $addrstr  The address list string.
@@ -128,7 +130,7 @@ final class ParseAddressesTest extends TestCase
      *
      * @requires extension imap
      *
-     * @group nombstring
+     * @group mbstringExtDisabled
      * @dataProvider dataAddressSplitting
      *
      * @param string $addrstr  The address list string.

test/PHPMailer/PunyencodeAddressTest.php

@@ -28,6 +28,8 @@ final class PunyencodeAddressTest extends TestCase
      *
      * @requires extension mbstring
      * @requires function idn_to_ascii
+     * 
+     * @group mbstringExtRequired
      *
      * @dataProvider dataPunyencodeAddressConversion
      *

test/PHPMailer/ReplyToGetSetClearTest.php

@@ -270,6 +270,8 @@ final class ReplyToGetSetClearTest extends PreSendTestCase
      *
      * @requires extension mbstring
      * @requires function idn_to_ascii
+     * 
+     * @group mbstringExtRequired
      */
     public function testEnqueueAndAddIdnAddress()
     {
@@ -308,6 +310,8 @@ final class ReplyToGetSetClearTest extends PreSendTestCase
      *
      * @requires extension mbstring
      * @requires function idn_to_ascii
+     * 
+     * @group mbstringExtRequired
      */
     public function testNoDuplicateReplyToAddresses()
     {
@@ -404,7 +408,7 @@ final class ReplyToGetSetClearTest extends PreSendTestCase
      * an 8bit character is passed and either the MbString or the Intl extension are
      * not available.
      *
-     * @group nombstring
+     * @group mbstringExtDisabled
      *
      * @covers \PHPMailer\PHPMailer\PHPMailer::addAnAddress
      */
@@ -424,6 +428,8 @@ final class ReplyToGetSetClearTest extends PreSendTestCase
      *
      * @requires extension mbstring
      * @requires function idn_to_ascii
+     * 
+     * @group mbstringExtRequired
      */
     public function testClearReplyTos()
     {

test/PHPMailer/SetFromTest.php

@@ -191,7 +191,7 @@ final class SetFromTest extends TestCase
      * containing an 8bit character is passed and either the MbString or the Intl extension are
      * not available.
      *
-     * @group nombstring
+     * @group mbstringExtDisabled
      */
     public function testSetFromFailsOn8BitCharInDomainWithoutOptionalExtensions()
     {