[Improvement][Unit Test] Fix flaky unit test in org.apache.dolphinscheduler.plugin.alert.http.HttpSenderTest

[SbloodyS]

Search before asking

• I had searched in the issues and found no similar feature requirement.

Description

https://github.com/apache/dolphinscheduler/runs/8226755468?check_suite_focus=true

Are you willing to submit a PR?

• Yes I am willing to submit a PR!

Code of Conduct

• I agree to follow this project's Code of Conduct

[EricGao888]

We need to decouple the external system in UT, e.g. connection to ''www.baidu.com'' here:

dolphinscheduler/dolphinscheduler-alert/dolphinscheduler-alert-plugins/dolphinscheduler-alert-http/src/test/java/org/apache/dolphinscheduler/plugin/alert/http/HttpSenderTest.java

Lines 30 to 41 in e101a5c

	@Test
	public void sendTest() {
	Map<String, String> paramsMap = new HashMap<>();
	paramsMap.put(HttpAlertConstants.NAME_URL, "http://www.baidu.com");
	paramsMap.put(HttpAlertConstants.NAME_REQUEST_TYPE, "POST");
	paramsMap.put(HttpAlertConstants.NAME_HEADER_PARAMS, "{\"Content-Type\":\"application/json\"}");
	paramsMap.put(HttpAlertConstants.NAME_BODY_PARAMS, "{\"number\":\"13457654323\"}");
	paramsMap.put(HttpAlertConstants.NAME_CONTENT_FIELD, "content");
	HttpSender httpSender = new HttpSender(paramsMap);
	AlertResult alertResult = httpSender.send("Fault tolerance warning");
	Assert.assertEquals("true", alertResult.getStatus());
	}

[SbloodyS]

We need to decouple the external system in UT, e.g. connection to ''www.baidu.com'' here:

dolphinscheduler/dolphinscheduler-alert/dolphinscheduler-alert-plugins/dolphinscheduler-alert-http/src/test/java/org/apache/dolphinscheduler/plugin/alert/http/HttpSenderTest.java

Lines 30 to 41 in e101a5c

	@Test
	public void sendTest() {
	Map<String, String> paramsMap = new HashMap<>();
	paramsMap.put(HttpAlertConstants.NAME_URL, "http://www.baidu.com");
	paramsMap.put(HttpAlertConstants.NAME_REQUEST_TYPE, "POST");
	paramsMap.put(HttpAlertConstants.NAME_HEADER_PARAMS, "{\"Content-Type\":\"application/json\"}");
	paramsMap.put(HttpAlertConstants.NAME_BODY_PARAMS, "{\"number\":\"13457654323\"}");
	paramsMap.put(HttpAlertConstants.NAME_CONTENT_FIELD, "content");
	HttpSender httpSender = new HttpSender(paramsMap);
	AlertResult alertResult = httpSender.send("Fault tolerance warning");
	Assert.assertEquals("true", alertResult.getStatus());
	}

That's right.

[huahuacheng]

I'm interested. May I take this one? QAQ

[SbloodyS]

I'm interested. May I take this one? QAQ

Sure. I've assigned to you. Looking forward to your contribution.

[EricGao888]

I'm interested. May I take this one? QAQ

Sure. I've assigned to you. Looking forward to your contribution.

+ 1

[github-actions]

Thank you for your feedback, we have received your issue, Please wait patiently for a reply.

• In order for us to understand your request as soon as possible, please provide detailed information、version or pictures.
• If you haven't received a reply for a long time, you can join our slack and send your question to channel #troubleshooting

[github-actions]

This issue has been automatically marked as stale because it has not had recent activity for 30 days. It will be closed in next 7 days if no further activity occurs.