Fix long method

[Rawa]

This PR splits a bunch of long methods that result in detekt linting issues.

---

This change is 

[Pururun]

Reviewed 6 of 6 files at r1, all commit messages.
Reviewable status: all files reviewed, 2 unresolved discussions (waiting on @Rawa)

---

android/app/src/main/kotlin/net/mullvad/mullvadvpn/compose/screen/ConnectScreen.kt line 111 at r1 (raw file):

    val connectViewModel: ConnectViewModel = koinViewModel()

    val state by connectViewModel.uiState.collectAsStateWithLifecycle()

I guess this will not be considered a change when we merge that other PR?

---

android/app/src/main/kotlin/net/mullvad/mullvadvpn/compose/screen/ConnectScreen.kt line 165 at r1 (raw file):

@Composable
private fun ConnectScreen(

This would break tests no?

[Rawa]

Reviewable status: all files reviewed, 2 unresolved discussions (waiting on @Pururun)

---

android/app/src/main/kotlin/net/mullvad/mullvadvpn/compose/screen/ConnectScreen.kt line 111 at r1 (raw file):

Previously, Pururun (Jonatan Rhodin) wrote…

I guess this will not be considered a change when we merge that other PR?

Correct.

---

android/app/src/main/kotlin/net/mullvad/mullvadvpn/compose/screen/ConnectScreen.kt line 165 at r1 (raw file):

Previously, Pururun (Jonatan Rhodin) wrote…

This would break tests no?

Ah yes, correct. I'll fix it.

[Pururun]

Reviewed 1 of 1 files at r2, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved

[albin-mullvad]

Reviewed 4 of 6 files at r1, 1 of 1 files at r2, 1 of 1 files at r3, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved