fix: Add call to Message Bus Connect()

[lenny-goodell]

PR Checklist

Please check if your PR fulfills the following requirements:

• The commit message follows our guidelines: https://wiki.edgexfoundry.org/display/FA/Contributor%27s+Guide
• Tests for the changes have been added (for bug fixes / features)
• Docs have been added / updated (for bug fixes / features)

PR Type

What kind of change does this PR introduce?

• Bugfix
• Feature
• Code style update (formatting, local variables)
• Refactoring (no functional changes, no api changes)
• Build related changes
• CI related changes
• Documentation content changes
• Other... Please describe:

What is the current behavior?

Core data fails to publish to MQTT MessageBus due to not being connected.

Issue Number: #2466

What is the new behavior?

Core data successfully publishes to MQTT MessageBus.

Does this PR introduce a breaking change?

• Yes
• No

Are there any specific instructions or things that should be known prior to reviewing?

Other information

[codecov-io]

Codecov Report

Merging #2467 into master will decrease coverage by 0.03%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master    #2467      +/-   ##
==========================================
- Coverage   38.25%   38.21%   -0.04%     
==========================================
  Files         153      153              
  Lines       12583    12594      +11     
==========================================
  Hits         4813     4813              
- Misses       7513     7524      +11     
  Partials      257      257              

Impacted Files	Coverage Δ
internal/core/data/init.go	0.00% <0.00%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5d710f1...ae5bb09. Read the comment docs.

[AnthonyMBonafide]

Could we add logic for calling the Disconnect method on the client as well. During testing these changes I noticed that the MQTT server is reporting that the client did not properly disconnect.

[lenny-goodell]

Could we add logic for calling the Disconnect method on the client as well. During testing these changes I noticed that the MQTT server is reporting that the client did not properly disconnect.

Great catch. I will add this. THX!

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities (and 0 Security Hotspots to review)
0 Code Smells

No Coverage information
0.0% Duplication

[lenny-goodell]

@AnthonyMBonafide , disconnect code has been added.

[AnthonyMBonafide]

LGTM! I verified these changes manually with a locally running MQTT Mosquitto server.