-
Notifications
You must be signed in to change notification settings - Fork 2.6k
Create Mqtt5 Example #364
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Create Mqtt5 Example #364
Conversation
Signed-off-by: nmy6452 <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you so much for your contribution. It is much appreciated!
@@ -0,0 +1,135 @@ | |||
/* | |||
* Copyright 2016-2018 the original author or authors. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Update all the copyright date for all .java files to 2025.
/** | ||
* Starts the Spring Context and will initialize the Spring Integration message flow. | ||
* | ||
* @author nmy6452 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please use your name if using the @author tag.
* @since 5.2 | ||
* | ||
*/ | ||
public class BrokerRunning extends TestWatcher { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please use the Mosquitto test container as shown here: https://github.com/spring-projects/spring-integration/blob/main/spring-integration-mqtt/src/test/java/org/springframework/integration/mqtt/MqttDslTests.java
import java.util.concurrent.CountDownLatch; | ||
import java.util.concurrent.TimeUnit; | ||
|
||
import org.junit.ClassRule; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Update tests to use JUnit 5 . Thanks!
|
||
# Overview | ||
|
||
This sample demonstrates basic functionality of the **Spring Integration MQTT Adapters**. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also add your project to the list of available projects in the README for main sample project.
@@ -0,0 +1,392 @@ | |||
<?xml version="1.0" encoding="UTF-8"?> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You don't have to add the pom.xml file. This will be done by the gradle tooling.
I have to close this PR because a DCO requirement in that we can't use your handle for the acceptance of the DCO. You will notice I did add a few review comments, so if you can update the new PR with these changes that would be great! Again thank you for the contribution! |
This is the result of your work on Issue #323.