Fix AccountSetup view ignoring all identity providers per section b…

…ut one (#81)

# Fix `AccountSetup` view ignoring all identity providers per section
but one

## ♻️ Current situation & Problem
Currently, the AccountSetupView groups the AccountService's
`IdentityProvider`s by their section, to determine their order.
As a result of this being implemented using a dictionary, with the
providers as values, only the last provider per section is kept.
This means that, if an AccountService specifies multiple
IdentityProviders for a section, most of them (all but one) will not be
shown to the user.

## ⚙️ Release Notes 
- Fixed `@IdentityProvider`s sometimes not being included in the
`AccountSetup` view.

## 📚 Documentation
n/a

## ✅ Testing
n/a

## 📝 Code of Conduct & Contributing Guidelines 

By submitting creating this pull request, you agree to follow our [Code
of
Conduct](https://github.com/StanfordSpezi/.github/blob/main/CODE_OF_CONDUCT.md)
and [Contributing
Guidelines](https://github.com/StanfordSpezi/.github/blob/main/CONTRIBUTING.md):
- [x] I agree to follow the [Code of
Conduct](https://github.com/StanfordSpezi/.github/blob/main/CODE_OF_CONDUCT.md)
and [Contributing
Guidelines](https://github.com/StanfordSpezi/.github/blob/main/CONTRIBUTING.md).

Sources/SpeziAccount/AccountSetup.swift

@@ -6,7 +6,6 @@
 // SPDX-License-Identifier: MIT
 //
 
-import OrderedCollections
 import SpeziViews
 import SwiftUI
 
@@ -139,20 +138,21 @@ public struct AccountSetup<Header: View, Continue: View>: View {
             EmptyServicesWarning()
         } else {
             VStack {
-                let categorized = account.accountSetupComponents.reduce(into: OrderedDictionary()) { partialResult, component in
+                let components = account.accountSetupComponents.reduce(
+                    into: [AccountSetupSection: [any AnyAccountSetupComponent]]()
+                ) { dict, component in
                     guard component.configuration.isEnabled else {
                         return
                     }
-                    partialResult[component.configuration.section] = component
+                    dict[component.configuration.section, default: []].append(component)
                 }
-
-                ForEach(categorized.keys.sorted(), id: \.self) { placement in
-                    if let component = categorized[placement] {
-                        component.anyView
-
-                        if categorized.keys.last != placement {
-                            ServicesDivider()
-                        }
+                    .sorted { $0.key < $1.key }
+                    .flatMap(\.value)
+
+                ForEach(0..<components.endIndex, id: \.self) { idx in
+                    components[idx].anyView
+                    if idx < components.endIndex - 1 {
+                        ServicesDivider()
                     }
                 }
             }

Tests/SpeziAccountTests/SnapshotTesting.swift

@@ -122,7 +122,7 @@ final class SnapshotTesting: XCTestCase {
         assertSnapshot(of: view3, as: .image(layout: .device(config: .iPhone13Pro)), named: "iphone-view3")
         assertSnapshot(of: view4, as: .image(layout: .device(config: .iPhone13Pro)), named: "iphone-view4")
         assertSnapshot(of: view5, as: .image(layout: .device(config: .iPhone13Pro)), named: "iphone-view5")
-
+        
         assertSnapshot(of: view0Signup, as: .image(layout: .device(config: .iPhone13Pro)), named: "iphone-view0-signup")
         assertSnapshot(of: view1Signup, as: .image(layout: .device(config: .iPhone13Pro)), named: "iphone-view1-signup")
         assertSnapshot(of: view2Signup, as: .image(layout: .device(config: .iPhone13Pro)), named: "iphone-view2-signup")