Merge pull request #88 from jevonmao/release_1.5.1

Project clean up & accesibility

Author: jevonmao

Package.swift

@@ -131,5 +131,6 @@ let package = Package(
             exclude: [],
             resources: [.process("__Snapshots__")]
         ),
+        
     ] + permissionsTargets
 )

Sources/CorePermissionsSwiftUI/Model/Mocks/MockHealthManager.swift

@@ -8,7 +8,6 @@
 import Foundation
 #if !os(tvOS) && PERMISSIONSWIFTUI_HEALTH
 import HealthKit
-
 protocol HealthManager {
     func authorizationStatus(for type: HKObjectType) -> HKAuthorizationStatus
     func requestAuthorization(toShare typesToShare: Set<HKSampleType>?, read typesToRead: Set<HKObjectType>?, completion: @escaping (Bool, Error?) -> Void)

Sources/CorePermissionsSwiftUI/SwiftUI/Shared/AllowButtonSection.swift

@@ -18,5 +18,6 @@ struct AllowButtonSection: View {
                 .buttonStatusColor(for: allowButtonStatus)
         })
         .layoutPriority(-1)
+        .accessibility(identifier: "Allow button")
     }
 }

Sources/CorePermissionsSwiftUI/SwiftUI/Shared/ExitButtonSection.swift

@@ -38,6 +38,7 @@ struct ExitButtonSection: View {
                         .padding(4)
                 )
         })
+        .accessibility(identifier: "Exit button")
     }
 }
 

Sources/CorePermissionsSwiftUI/SwiftUI/Shared/HeaderView.swift

@@ -27,6 +27,7 @@ struct HeaderView: View {
                             .frame(maxWidth:.infinity, alignment: .leading)
                             //Negative padding make the bottom title "shift up" a little
                             .padding(.vertical, -5)
+                            .accessibility(identifier: "Alert header")
                     }
 
                 HStack {
@@ -37,6 +38,7 @@ struct HeaderView: View {
                             .minimumScaleFactor(0.85)
                             .allowsTightening(true)
                             .layoutPriority(1)
+                            .accessibility(identifier: "Main title")
 
 
                     Spacer()
@@ -66,6 +68,7 @@ struct HeaderView: View {
                     //Allow scacling down to half the original size on smaller screens
                     .minimumScaleFactor(0.5)
                     .textHorizontalAlign(.leading)
+                    .accessibility(identifier: "Modal header description")
             }
 
         }

Sources/CorePermissionsSwiftUI/SwiftUI/Shared/PermissionSectionCell.swift

@@ -59,18 +59,23 @@ struct PermissionSectionCell: View {
                 .font(.system(size: screenSizeConstant))
                 .frame(width: screenSizeConstant)
                 .padding(.horizontal, 5)
+                .accessibility(identifier: "Permission icon")
+            
             VStack(alignment: .leading) {
                 Text(currentPermission.title)
                     .font(.system(size: fontSizeConstant))
                     .bold()
                     .lineLimit(1)
                     .minimumScaleFactor(0.7)
                     .layoutPriority(1)
+                    .accessibility(identifier: "Permission title")
+                
                 Text(currentPermission.description)
                     .font(.system(size: smallFontSizeConstant))
                     .lineLimit(3)
                     .foregroundColor(Color(.systemGray2))
                     .minimumScaleFactor(0.5)
+                    .accessibility(identifier: "Permission description")
 
             }
             .padding(.horizontal, 3)

Tests/PermissionsSwiftUITests/UITests.swift

@@ -0,0 +1,24 @@
+//
+//  UITests.swift
+//  
+//
+//  Created by Jevon Mao on 6/15/21.
+//
+
+import Foundation
+import XCTest
+
+final class UITests: XCTestCase {
+    var app: XCUIApplication!
+
+    override func setUp() {
+        continueAfterFailure = false
+        app = XCUIApplication()
+        app.launchArguments = ["testing"]
+        app.launch()
+    }
+    
+    func testTitle() {
+        //XCTAssertTrue(app.staticTexts["Hello World!"].exists)
+    }
+}