Skip to content

cloud: fix fallback provisioning network state handling#394

Merged
simensrostad merged 1 commit intomainfrom
fix-fallback-provisioning
Sep 26, 2025
Merged

cloud: fix fallback provisioning network state handling#394
simensrostad merged 1 commit intomainfrom
fix-fallback-provisioning

Conversation

@simensrostad
Copy link
Copy Markdown
Contributor

The fallback provisioning logic incorrectly determined the next state when provisioning failed. The network_connected variable was not being updated properly, causing incorrect state transitions.

This fix ensures the network connection status is checked accurately before determining whether to retry provisioning or transition to an offline state.

Added corresponding unit tests to verify the correct behavior of the fallback provisioning logic under different network conditions.

@simensrostad simensrostad force-pushed the fix-fallback-provisioning branch 2 times, most recently from 9991e28 to a92e6f0 Compare September 26, 2025 12:00
The fallback provisioning logic incorrectly determined the next state
when provisioning failed. The network_connected variable was not being
updated properly, causing incorrect state transitions.

This fix ensures the network connection status is checked accurately
before determining whether to retry provisioning or transition to an
offline state.

Added corresponding unit tests to verify the correct behavior of the
fallback provisioning logic under different network conditions.

Signed-off-by: Simen S. Røstad <simen.rostad@nordicsemi.no>
@simensrostad simensrostad force-pushed the fix-fallback-provisioning branch from a92e6f0 to cc285d6 Compare September 26, 2025 12:12
@sonarqubecloud
Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
47.5% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

@simensrostad simensrostad merged commit f99ac2f into main Sep 26, 2025
9 of 10 checks passed
@simensrostad simensrostad deleted the fix-fallback-provisioning branch September 26, 2025 12:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant