Fix PR Comments

Author: VladislavKostyukevich

src/main/java/main/controllers/Project/SuiteController.java

@@ -52,7 +52,7 @@ public TestSuiteDto get(String name, Integer projectId) throws AqualityException
         template.setProject_id(projectId);
         try{
             return get(template, false).get(0);
-        } catch (Exception e) {
+        } catch (IndexOutOfBoundsException e) {
             throw new AqualityException("The '%s' suite does not exist.", name);
         }
     }

src/main/java/main/model/db/imports/BaseImporter.java

@@ -83,7 +83,12 @@ private void createTestRun() throws AqualityException {
             TestRunDto template = new TestRunDto();
             template.setProject_id(testRun.getProject_id());
             template.setId(testRun.getId());
-            testRun = controllerFactory.getHandler(template).get(template, false, 1).get(0);
+            List<TestRunDto> testRuns = controllerFactory.getHandler(template).get(template, false, 1);
+            if(testRuns.size() > 0){
+                testRun = testRuns.get(0);
+            } else {
+                throw new AqualityException("Test run with %s id does not exist!", testRun.getId());
+            }
         }
         else{
             createTestRun((testRun.getBuild_name() != null && !testRun.getBuild_name().equals(""))

src/main/java/main/model/db/imports/SAXHandlers/NUnitV3Handler.java

@@ -14,6 +14,7 @@
 import java.util.*;
 
 import static main.model.db.imports.ResultStatus.*;
+import static main.model.db.imports.TestNameNodeType.*;
 
 public class NUnitV3Handler extends Handler {
     private TestSuiteDto testSuite = new TestSuiteDto();
@@ -62,7 +63,7 @@ public void startElement(String uri, String localName, String qName, Attributes
                 testRun.setAuthor(String.format("%s\\%s", attributes.getValue("user-domain"), attributes.getValue("user")));
                 break;
             case "test-suite":
-                if(attributes.getValue("type").equals("TestFixture") && testNameNodeType == TestNameNodeType.featureNameTestName) {
+                if(attributes.getValue("type").equals("TestFixture") && testNameNodeType == featureNameTestName) {
                     currentFixture = attributes.getValue("name");
                 }
                 break;
@@ -170,13 +171,15 @@ public void characters(char[] ch, int start, int length) {
     }
 
     private void setTestName(Attributes attributes) throws SAXException {
-        if(testNameNodeType == TestNameNodeType.featureNameTestName) {
-            test.setName(String.format("%s: %s", currentFixture, attributes.getValue("name")) );
-        }
-        else if(testNameNodeType == TestNameNodeType.className) {
-            test.setName(attributes.getValue("fullname"));
-        } else {
-            throw new SAXException("testNameNodeType is not correct for NUnitV3 parser.");
+        switch (testNameNodeType) {
+            case featureNameTestName:
+                test.setName(String.format("%s: %s", currentFixture, attributes.getValue("name")) );
+                break;
+            case className:
+                test.setName(attributes.getValue("fullname"));
+                break;
+            default:
+                throw new SAXException("testNameNodeType is not correct for NUnitV3 parser.");
         }
     }