Merge pull request #22 from aquality-automation/feature/Nunit3Add_Possibility_to_use_fullname_attribute_as_test_name

Feature/nunit3 add possibility to use fullname attribute as test name

Author: VladislavKostyukevich

CHANGELOG.md

@@ -0,0 +1,22 @@
+# CHANGELOG
+
+## 0.3.2 (unreleased)
+
+Features:
+
+  - Import: NUnit V3 add possibility to import with fullname test name  -> [View Issue](https://github.com/aquality-automation/aquality-tracking-ui/issues/27)
+
+Bugfixes:
+
+  - Import: Import to latest test Run not working when importing more than one file  -> [View Issue](https://github.com/aquality-automation/aquality-tracking-ui/issues/29)
+
+## 0.3.1 (2019-08-30)
+
+Features:
+
+  - Add ability to set Default Email Pattern
+  - Add regex search to Results Searcher
+
+Bugfixes:
+
+  - Fix Fail Reason Search

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

@@ -46,6 +46,17 @@ public List<TestSuiteDto> get(TestSuiteDto template) throws AqualityException {
         return get(template, false);
     }
 
+    public TestSuiteDto get(String name, Integer projectId) throws AqualityException {
+        TestSuiteDto template = new TestSuiteDto();
+        template.setName(name);
+        template.setProject_id(projectId);
+        try{
+            return get(template, false).get(0);
+        } catch (IndexOutOfBoundsException e) {
+            throw new AqualityException("The '%s' suite does not exist.", name);
+        }
+    }
+
     @Override
     public boolean delete(TestSuiteDto template) throws AqualityException {
         if(baseUser.isManager() || baseUser.getProjectUserBySuiteId(template.getId()).isManager()){

src/main/java/main/controllers/Project/TestRunController.java

@@ -65,6 +65,18 @@ public List<TestRunStatisticDto> get(TestRunStatisticDto template) throws Aquali
         return testRunStatisticDao.searchAll(template);
     }
 
+    public TestRunDto getLastSuiteTestRun(Integer suiteId, Integer projectId) throws AqualityException {
+        TestRunDto template = new TestRunDto();
+        template.setTest_suite_id(suiteId);
+        template.setProject_id(projectId);
+        List<TestRunDto> testRuns = get(template, false, 1);
+        if(testRuns.size() > 0){
+            return testRuns.get(0);
+        }
+
+        throw new AqualityException("There are no test runs for suite with '%s' id!", suiteId);
+    }
+
     @Override
     public boolean delete(TestRunDto template) throws AqualityException {
         if(baseUser.isManager() || baseUser.getProjectUser(template.getProject_id()).isEditor()){

src/main/java/main/exceptions/AqualityException.java

@@ -8,4 +8,8 @@ public class AqualityException extends Exception {
     public AqualityException(String error) {
         super(error);
     }
+
+    public AqualityException(String error, Object... args) {
+        super(String.format(error, args));
+    }
 }

src/main/java/main/exceptions/AqualitySQLException.java

@@ -17,6 +17,8 @@ private static String getErrorMessage(String sqlcode){
                 return "You are trying to create duplicate entity.";
             case "42000":
                 return "You Regular expression is not valid!";
+            case "40001":
+                return "You are trying to edit entity which is locked. Please retry the operation.";
             default:
                 return String.format("Unknown SQL Error: %s", sqlcode);
         }

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

@@ -80,7 +80,15 @@ void addLogToImport(String log) throws AqualityException {
 
     private void createTestRun() throws AqualityException {
         if(testRun.getId() != null){
-            this.testRun = controllerFactory.getHandler(testRun).get(testRun, false, 1).get(0);
+            TestRunDto template = new TestRunDto();
+            template.setProject_id(testRun.getProject_id());
+            template.setId(testRun.getId());
+            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/HandlerFactory.java

@@ -28,7 +28,10 @@ Handler getHandler(File file, String type, TestNameNodeType testNameNodeType) th
             case NUnit_v2:
                 return new NUnitV2(file);
             case NUnit_v3:
-                return new NUnitV3(file);
+                if(testNameNodeType == null){
+                    throw new AqualityException("testNameNode is required");
+                }
+                return new NUnitV3(file, testNameNodeType);
             default:
                 throw new AqualityException(String.format("Import Type '%s' is not implemented", type));
         }

src/main/java/main/model/db/imports/ImportHandlers/NUnitV3.java

@@ -3,6 +3,7 @@
 import main.exceptions.AqualityException;
 import main.model.db.imports.Handler;
 import main.model.db.imports.SAXHandlers.NUnitV3Handler;
+import main.model.db.imports.TestNameNodeType;
 import main.model.dto.TestDto;
 import main.model.dto.TestResultDto;
 import main.model.dto.TestRunDto;
@@ -17,8 +18,8 @@ public class NUnitV3 extends Handler {
 
     private NUnitV3Handler handler;
 
-    public NUnitV3(File file) throws AqualityException {
-        handler = new NUnitV3Handler();
+    public NUnitV3(File file, TestNameNodeType testNameNodeType) throws AqualityException {
+        handler = new NUnitV3Handler(testNameNodeType);
         try {
             this.parser.parse(file, handler);
         } catch (SAXException | IOException e) {

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

@@ -3,6 +3,7 @@
 import main.model.db.imports.ResultStatus;
 import main.exceptions.AqualityException;
 import main.model.db.imports.Handler;
+import main.model.db.imports.TestNameNodeType;
 import main.model.dto.*;
 import org.xml.sax.Attributes;
 import org.xml.sax.SAXException;
@@ -13,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();
@@ -28,9 +30,11 @@ public class NUnitV3Handler extends Handler {
     private Boolean isReasonStarted = false;
     private Boolean isTestCaseStarted = false;
     private Integer assertionNumber = 0;
+    private TestNameNodeType testNameNodeType;
 
-    public NUnitV3Handler() throws AqualityException {
+    public NUnitV3Handler(TestNameNodeType testNameNodeType) throws AqualityException {
         super();
+        this.testNameNodeType = testNameNodeType;
         result.setFail_reason("$blank");
     }
 
@@ -59,13 +63,13 @@ 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")) {
+                if(attributes.getValue("type").equals("TestFixture") && testNameNodeType == featureNameTestName) {
                     currentFixture = attributes.getValue("name");
                 }
                 break;
             case "test-case":
                 isTestCaseStarted = true;
-                test.setName(String.format("%s: %s", currentFixture, attributes.getValue("name")) );
+                setTestName(attributes);
                 try {
                     result.setStart_date(convertToDate(attributes.getValue("start-time")));
                     result.setFinish_date(convertToDate(attributes.getValue("end-time")));
@@ -166,6 +170,18 @@ public void characters(char[] ch, int start, int length) {
         }
     }
 
+    private void setTestName(Attributes attributes) throws SAXException {
+        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.");
+        }
+    }
 
     private Date convertToDate(String dateString) throws ParseException {
         DateFormat format = new SimpleDateFormat("yyyy-MM-dd HH:mm:ss");

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

@@ -3,5 +3,6 @@
 public enum TestNameNodeType {
     className,
     testName,
-    descriptionNode
+    descriptionNode,
+    featureNameTestName
 }