fix:添加保存

backend/src/main/java/io/metersphere/api/jmeter/APIBackendListenerClient.java

@@ -3,16 +3,20 @@ package io.metersphere.api.jmeter;
 import io.metersphere.api.service.APIReportService;
 import io.metersphere.api.service.APITestService;
 import io.metersphere.base.domain.ApiTestReport;
+import io.metersphere.base.domain.Notice;
 import io.metersphere.commons.constants.APITestStatus;
 import io.metersphere.commons.constants.ApiRunMode;
 import io.metersphere.commons.utils.CommonBeanFactory;
 import io.metersphere.commons.utils.LogUtil;
+import io.metersphere.notice.service.MailService;
+import io.metersphere.notice.service.NoticeService;
 import org.apache.commons.lang3.StringUtils;
 import org.apache.jmeter.assertions.AssertionResult;
 import org.apache.jmeter.samplers.SampleResult;
 import org.apache.jmeter.visualizers.backend.AbstractBackendListenerClient;
 import org.apache.jmeter.visualizers.backend.BackendListenerContext;
 
+import javax.annotation.Resource;
 import java.io.Serializable;
 import java.util.*;
 
@@ -113,9 +117,12 @@ public class APIBackendListenerClient extends AbstractBackendListenerClient impl
             report = apiReportService.getRunningReport(testResult.getTestId());
         }
         apiReportService.complete(testResult, report);
-
         queue.clear();
         super.teardownTest(context);
+        NoticeService noticeService = CommonBeanFactory.getBean(NoticeService.class);
+        List<Notice> notice = noticeService.queryNotice(testResult.getTestId());
+        MailService mailService = CommonBeanFactory.getBean(MailService.class);
+        mailService.sendHtml(report.getId(), notice, report.getStatus(), "api");
     }
 
     private RequestResult getRequestResult(SampleResult result) {

backend/src/main/java/io/metersphere/api/service/APITestService.java

@@ -223,10 +223,10 @@ public class APITestService {
             apiTest.setUserId(request.getUserId());
         }
         String reportId = apiReportService.create(apiTest, request.getTriggerMode());
-        if (request.getTriggerMode().equals("SCHEDULE")) {
+        /*if (request.getTriggerMode().equals("SCHEDULE")) {
             List<Notice> notice = noticeService.queryNotice(request.getId());
             mailService.sendHtml(reportId,notice,"api");
-        }
+        }*/
         changeStatus(request.getId(), APITestStatus.Running);
         jMeterService.run(request.getId(), null, is);
         return reportId;

backend/src/main/java/io/metersphere/notice/service/MailService.java

@@ -9,6 +9,7 @@ import io.metersphere.commons.utils.LogUtil;
 import io.metersphere.dto.LoadTestDTO;
 import io.metersphere.service.SystemParameterService;
 import io.metersphere.service.UserService;
+import org.springframework.mail.MailException;
 import org.springframework.mail.javamail.JavaMailSenderImpl;
 import org.springframework.mail.javamail.MimeMessageHelper;
 import org.springframework.stereotype.Service;
@@ -30,7 +31,7 @@ public class MailService {
     @Resource
     private SystemParameterService systemParameterService;
 
-    public void sendHtml(String id, List<Notice> notice,String type) {
+    public void sendHtml(String id, List<Notice> notice, String status, String type) {
         JavaMailSenderImpl javaMailSender = new JavaMailSenderImpl();
         List<SystemParameter> paramList = systemParameterService.getParamList(ParamConstants.Classify.MAIL.getValue());
         javaMailSender.setDefaultEncoding("UTF-8");
@@ -58,15 +59,12 @@ public class MailService {
         javaMailSender.setJavaMailProperties(props);
         MimeMessage mimeMessage = javaMailSender.createMimeMessage();
         String testName = "";
-        String state = "";
         if (type.equals("api")) {
             APIReportResult reportResult = apiAndPerformanceHelper.getApi(id);
             testName = reportResult.getTestName();
-            state = reportResult.getStatus();
         } else if (type.equals("performance")) {
             LoadTestDTO performanceResult = apiAndPerformanceHelper.getPerformance(id);
             testName = performanceResult.getName();
-            state = performanceResult.getStatus();
         }
         String html1 = "<!DOCTYPE html>\n" +
                 "<html lang=\"en\">\n" +
@@ -115,7 +113,7 @@ public class MailService {
                 LogUtil.error("Recipient information is empty");
             }
 
-            if(state.equals("success")){
+            if (status.equals("Success")) {
                 users = successEmailList.toArray(new String[successEmailList.size()]);
                 helper.setText(html2, true);
             } else {
@@ -124,11 +122,15 @@ public class MailService {
 
             }
             helper.setTo(users);
-            javaMailSender.send(mimeMessage);
 
         } catch (MessagingException e) {
             e.printStackTrace();
         }
+        try {
+            javaMailSender.send(mimeMessage);
+        } catch (MailException e) {
+            e.printStackTrace();
+        }
     }
 
 

backend/src/main/java/io/metersphere/notice/service/NoticeService.java

@@ -16,30 +16,54 @@ import java.util.List;
 public class NoticeService {
     @Resource
     private NoticeMapper noticeMapper;
+
     public void saveNotice(NoticeRequest noticeRequest) {
+        Notice notice = new Notice();
         NoticeExample example = new NoticeExample();
         example.createCriteria().andTestIdEqualTo(noticeRequest.getTestId());
         List<Notice> notices = noticeMapper.selectByExample(example);
-        if (notices != null) {
+        if (notices.size() > 0) {
             noticeMapper.deleteByExample(example);
-            noticeRequest.getNotices().forEach(notice -> {
+            noticeRequest.getNotices().forEach(n -> {
-                for(String x:notice.getNames()){
+                if (n.getNames().length > 0) {
-                    notice.setEvent(notice.getEvent());
+                    for (String x : n.getNames()) {
-                    notice.setEmail(notice.getEmail());
+                        notice.setEvent(n.getEvent());
-                    notice.setEnable(notice.getEnable());
+                        notice.setEmail(n.getEmail());
+                        notice.setEnable(n.getEnable());
                         notice.setTestId(noticeRequest.getTestId());
                         notice.setName(x);
                         noticeMapper.insert(notice);
                     }
+                } else {
+                    notice.setEvent(n.getEvent());
+                    notice.setEmail(n.getEmail());
+                    notice.setEnable(n.getEnable());
+                    notice.setTestId(noticeRequest.getTestId());
+                    notice.setName("");
+                    noticeMapper.insert(notice);
+                }
             });
         } else {
-            noticeRequest.getNotices().forEach(notice -> {
+            noticeRequest.getNotices().forEach(n -> {
-                notice.setEvent(noticeRequest.getEvent());
+                if (n.getNames().length > 0) {
+                    for (String x : n.getNames()) {
+                        notice.setEvent(n.getEvent());
+                        notice.setEmail(n.getEmail());
+                        notice.setEnable(n.getEnable());
                         notice.setTestId(noticeRequest.getTestId());
+                        notice.setName(x);
                         noticeMapper.insert(notice);
+                    }
+                } else {
+                    notice.setEvent(n.getEvent());
+                    notice.setEmail(n.getEmail());
+                    notice.setEnable(n.getEnable());
+                    notice.setTestId(noticeRequest.getTestId());
+                    notice.setName("");
+                    noticeMapper.insert(notice);
+                }
             });
         }
-
     }
 
     public List<Notice> queryNotice(String id) {

backend/src/main/java/io/metersphere/performance/service/PerformanceTestService.java

@@ -239,7 +239,7 @@ public class PerformanceTestService {
         startEngine(loadTest, engine, request.getTriggerMode());
         if (request.getTriggerMode().equals("SCHEDULE")) {
             List<Notice> notice = noticeService.queryNotice(request.getId());
-            mailService.sendHtml(engine.getReportId(),notice,"performance");
+            mailService.sendHtml(engine.getReportId(), notice, "success", "performance");
         }
         return engine.getReportId();
     }

frontend/src/business/components/common/components/MsScheduleEdit.vue

@@ -71,6 +71,7 @@
                   </template>
                 </el-table-column>
               </el-table>
+              <el-button  type="primary" @click="saveNotice">{{$t('commons.save')}}</el-button>
             </template>
           </el-tab-pane>
         </el-tabs>
@@ -206,9 +207,11 @@
             return false;
           }
         });
+      },
+      saveNotice(){
         let param = this.buildParam();
         this.result = this.$post("notice/save", param, () => {
-
+          this.$success(this.$t('commons.save_success'));
         })
       },
       close() {