SMSPeerTest Fixes (#40)

* added files to .gitignore

* removed modules.xml, as it was added to .gitignore

* added to .gitignore all patterns present in default .gitignore generated by Android Studio

* Implemented Google Library

Still everything to test

* Rewritten tests, fixed one class, added emulator exception

* Codacy improvements

* Applied suggested changes:

+ Added author tag (and standardized specs)
* Modified parseMessage() structure
* Extracted exception messages into static final fields
* Extracted RegEx string into static final field
* Used String.format()
- Removed fail() assertions

* Update README.md

* Changes:

+ Added exception types
- Removed throws clause on unchecked exceptions
* Enhanced tests
* Updated gradle

* Test Fixes:

Major fixes:
* Overridden prefix auto-detection
Small fixes:
* Syntax and typo fixes

Co-authored-by: giovannivelludo <56405609+giovannivelludo@users.noreply.github.com>
Co-authored-by: Cogno-Marco <marco.cognolato.98@gmail.com>

Author: 3 people

smslibrary/src/test/java/com/eis/smslibrary/SMSPeerTest.java

@@ -39,12 +39,12 @@ public class SMSPeerTest {
 
     private static final String HIGHER_VALID_ADDR = "3401234568";
 
-    private static PhoneNumberUtil phoneNumberUtil = PhoneNumberUtil.getInstance();
     private static String[] randomValidAddresses = new String[VALID_TEST_ITERATIONS];
     private static String randomInvalidAddress;
 
     @BeforeClass
     public static void setup() {
+        PhoneNumberUtil phoneNumberUtil = PhoneNumberUtil.getInstance();
         Set<String> availableCountries = phoneNumberUtil.getSupportedRegions();
         String[] countryCodes = availableCountries.toArray(new String[0]);
         int randomIndex = new Random().nextInt(countryCodes.length);
@@ -59,6 +59,7 @@ public static void setup() {
                     phoneNumberUtil.getExampleNumber(countryCodes[randomIndex]),
                     PhoneNumberUtil.PhoneNumberFormat.E164);
         }
+        SMSPeer.setDefaultRegion("IT");
     }
 
     @Test
@@ -87,30 +88,30 @@ public void emptyPeer_isNotCreated() {
 
     @Test
     public void shortPeer_isNotCreated() {
-        try { new SMSPeer(SHORT_ADDR);}
+        try { new SMSPeer(SHORT_ADDR); }
         catch(InvalidTelephoneNumberException e) {
             assertEquals(InvalidTelephoneNumberException.Type.INVALID_NUMBER, e.getType());
         }
     }
 
     @Test
     public void longPeer_isNotCreated() {
-        try { new SMSPeer(LONG_ADDR);}
+        try { new SMSPeer(LONG_ADDR); }
         catch(InvalidTelephoneNumberException e) {
             assertEquals(InvalidTelephoneNumberException.Type.INVALID_NUMBER, e.getType());
         }
     }
 
     @Test
     public void invalidPeer_isNotCreated() {
-        try { new SMSPeer(randomInvalidAddress);}
+        try { new SMSPeer(randomInvalidAddress); }
         catch(InvalidTelephoneNumberException e) {
             assertEquals(InvalidTelephoneNumberException.Type.INVALID_NUMBER, e.getType());
         }
     }
 
     @Test
-    public void validAddressed_areCompared() {
+    public void validAddresses_areCompared() {
         SMSPeer lowerPeer = new SMSPeer(VALID_ADDR);
         SMSPeer higherPeer = new SMSPeer(HIGHER_VALID_ADDR);
         assertTrue(higherPeer.compareTo(lowerPeer) > 0);