fix:'clean_session' in app.config is not passed to the client

[congzhou09]

"app.config['MQTT_CLEAN_SESSION'] = False" does not work currently. The reason is that the param is not passed to "self.client".

The git commit log shows this issue appeared since the commit hashed by "df8fcaae08dd43eb4fd8850f8380e8f8415a00b9".

[Sohaib90]

Hey @congzhou09

Thank you for the PR. Yes I can see where the problem is. Would be nice if you could also edit the test cases to include a test case so that the PR is more complete. What do you think?

[congzhou09]

Hey @congzhou09

Thank you for the PR. Yes I can see where the problem is. Would be nice if you could also edit the test cases to include a test case so that the PR is more complete. What do you think?

Roger that.

[congzhou09]

Hello @Sohaib90 . I notice that the test case for MQTT_CLEAN_SESSION already exists in "tests/test_flaskmqtt.py:66". However, since the assertion value(True) is the same as the field's default value(True), it didn't detect the issue. So I have modified the assertion value to False.

[congzhou09]

Hello @Sohaib90 . Are there any revise needed for this PR?

[Sohaib90]

@congzhou09 I see that the checks have passed, let me go through the PR in detail during this weekend and then I will merge it if no further issues are found :)