fix(ble_gatt_server): Deinit services before device deinit

[finger563]

Description

• Ensure BleGattServer::client_ connection info is cleared, so that it does not improperly think that it is still connected when it may not be.
• Update order of calls in BleGattServer::deinit to deinit the services before the stack
• Update esp-nimble-cpp submodule to fix bug in NimBLEDevice::deinit(true).
• Update ble_gatt_server/example to run BleGattServer::deinit
• Update BleGattServer so that init() and deinit() can be called multiple times in a row without causing a crash
• Update ble_gatt_server/example to test calling deinit multiple times (once manually and once via destructor)

Motivation and Context

Ensures deinit works properly and does not cause any crashes.

• fix(NimBLEDevice): clear all before port_deinit to prevent crash esp-nimble-cpp#3
• fix(NimBLEDevice): clear all before port_deinit to prevent crash h2zero/esp-nimble-cpp#197

How has this been tested?

Building and running the updated example, showing no crash.

Screenshots (if appropriate, e.g. schematic, board, console logs, lab pictures):

After the change (fixed):

no clients were ever connected, then deinit:

client was connected, then deinit:

Before the change:

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• Documentation Update
• Hardware (schematic, board, system design) change
• Software change

Checklist:

• My change requires a change to the documentation.
• I have added / updated the documentation related to this change via either README or WIKI

Software

• I have added tests to cover my changes.
• I have updated the .github/workflows/build.yml file to add my new test to the automated cloud build github action.
• All new and existing tests passed.
• My code follows the code style of this project.

[github-actions]

✅Static analysis result - no issues found! ✅