Don't show linter warnings

checkout · Oct 24, 2023 · aebfc18 · aebfc18
1 parent 0b349fd
commit aebfc18
Show file tree

Hide file tree

Showing 8 changed files with 26 additions and 9 deletions.
diff --git a/.github/scripts/lintEditedFiles.sh b/.github/scripts/lintEditedFiles.sh
@@ -13,7 +13,7 @@ if EDITED_FILES=$(git diff HEAD origin/main --name-only --diff-filter=d | grep "
   if [ -z "$EDITED_FILES" ]; then
     echo "No edited .swift files found."
   else
-    swiftlint lint $EDITED_FILES | sed -E 's/^(.*):([0-9]+):([0-9]+): (warning|error|[^:]+): (.*)/::\4 title=Lint error,file=\1,line=\2,col=\3::\5\n\1:\2:\3/'
+    swiftlint lint $EDITED_FILES | sed -E -n 's/^(.*):([0-9]+):([0-9]+): error: (.*)/::error file=\1,line=\2,col=\3::\4\n\1:\2:\3/p'
   fi
 else
   echo "No changes in .swift files found."

diff --git a/Source/UI/SecurityCodeComponent/DefaultSecurityCodeFormStyle+Extension.swift b/Source/UI/SecurityCodeComponent/DefaultSecurityCodeFormStyle+Extension.swift
@@ -31,7 +31,6 @@ extension DefaultSecurityCodeFormStyle {
                                                                       normalColor: .clear,
                                                                       focusColor: .clear,
                                                                       errorColor: .clear))
-
-
+
   }
 }
diff --git a/Source/UI/SecurityCodeComponent/SecurityCodeComponent.swift b/Source/UI/SecurityCodeComponent/SecurityCodeComponent.swift
@@ -52,7 +52,7 @@ extension SecurityCodeComponent {
 
 extension SecurityCodeComponent: SecurityCodeViewDelegate {
   func update(securityCode: String) {
-    guard securityCode.isEmpty else {
+    guard !securityCode.isEmpty else {
       isSecurityCodeValid(false)
       return
     }

diff --git a/Source/UI/SecurityCodeComponent/SecurityCodeComponentConfiguration.swift b/Source/UI/SecurityCodeComponent/SecurityCodeComponentConfiguration.swift
@@ -25,15 +25,15 @@ public struct SecurityCodeComponentConfiguration {
   let environment: Environment
   public var style: SecurityCodeComponentStyle
   public var cardScheme: Card.Scheme?
-  
+
   public init(apiKey: String,
               environment: Environment,
               style: SecurityCodeComponentStyle? = nil,
               cardScheme: Card.Scheme? = nil) {
     self.apiKey = apiKey
     self.environment = environment
     self.cardScheme = cardScheme
-    
+
     if let style = style {
       self.style = style
     } else {

diff --git a/iOS Example Frame SPM/iOS Example Frame SPM.xcodeproj/project.pbxproj b/iOS Example Frame SPM/iOS Example Frame SPM.xcodeproj/project.pbxproj
@@ -30,6 +30,8 @@
 		168CEC952AC5C71700BB52B0 /* SecurityCodeViewController.swift in Sources */ = {isa = PBXBuildFile; fileRef = 168CEC942AC5C71700BB52B0 /* SecurityCodeViewController.swift */; };
 		16900D442AE6BA38009A7CE9 /* SecurityCodeComponentUITests.swift in Sources */ = {isa = PBXBuildFile; fileRef = 16900D432AE6BA38009A7CE9 /* SecurityCodeComponentUITests.swift */; };
 		16900D4B2AE70F90009A7CE9 /* Frames in Frameworks */ = {isa = PBXBuildFile; productRef = 16900D4A2AE70F90009A7CE9 /* Frames */; };
+		16900D4D2AE73FF4009A7CE9 /* Frames in Frameworks */ = {isa = PBXBuildFile; productRef = 16900D4C2AE73FF4009A7CE9 /* Frames */; };
+		16900D4F2AE741B7009A7CE9 /* Frames in Frameworks */ = {isa = PBXBuildFile; productRef = 16900D4E2AE741B7009A7CE9 /* Frames */; };
 		169DF1482A7BFB1B00891DF0 /* CardSchemeFormatSnapshotTests.swift in Sources */ = {isa = PBXBuildFile; fileRef = 169DF1472A7BFB1B00891DF0 /* CardSchemeFormatSnapshotTests.swift */; };
 		16C3F8402A7927ED00690639 /* SnapshotTesting in Frameworks */ = {isa = PBXBuildFile; productRef = 16C3F83F2A7927ED00690639 /* SnapshotTesting */; };
 		16C3F8422A7956EA00690639 /* CardValidationSnapshotTests.swift in Sources */ = {isa = PBXBuildFile; fileRef = 16C3F8412A7956EA00690639 /* CardValidationSnapshotTests.swift */; };
@@ -206,6 +208,7 @@
 			isa = PBXFrameworksBuildPhase;
 			buildActionMask = 2147483647;
 			files = (
+				16900D4F2AE741B7009A7CE9 /* Frames in Frameworks */,
 				16C3F8402A7927ED00690639 /* SnapshotTesting in Frameworks */,
 			);
 			runOnlyForDeploymentPostprocessing = 0;
@@ -214,6 +217,7 @@
 			isa = PBXFrameworksBuildPhase;
 			buildActionMask = 2147483647;
 			files = (
+				16900D4D2AE73FF4009A7CE9 /* Frames in Frameworks */,
 			);
 			runOnlyForDeploymentPostprocessing = 0;
 		};
@@ -472,6 +476,7 @@
 			name = "iOS Example Frame Regression Tests";
 			packageProductDependencies = (
 				16C3F83F2A7927ED00690639 /* SnapshotTesting */,
+				16900D4E2AE741B7009A7CE9 /* Frames */,
 			);
 			productName = "iOS Example Frame Regression Tests";
 			productReference = 16857B712A65F15F005CAE39 /* iOS Example Frame Regression Tests.xctest */;
@@ -492,6 +497,7 @@
 			);
 			name = "iOS Example FrameUITests";
 			packageProductDependencies = (
+				16900D4C2AE73FF4009A7CE9 /* Frames */,
 			);
 			productName = "iOS Example FrameUITests";
 			productReference = 16AE74C42A5C1EBB0031F794 /* iOS Example FrameUITests.xctest */;
@@ -1237,6 +1243,16 @@
 			package = 16900D492AE70F90009A7CE9 /* XCRemoteSwiftPackageReference "frames-ios" */;
 			productName = Frames;
 		};
+		16900D4C2AE73FF4009A7CE9 /* Frames */ = {
+			isa = XCSwiftPackageProductDependency;
+			package = 16900D492AE70F90009A7CE9 /* XCRemoteSwiftPackageReference "frames-ios" */;
+			productName = Frames;
+		};
+		16900D4E2AE741B7009A7CE9 /* Frames */ = {
+			isa = XCSwiftPackageProductDependency;
+			package = 16900D492AE70F90009A7CE9 /* XCRemoteSwiftPackageReference "frames-ios" */;
+			productName = Frames;
+		};
 		16C3F83F2A7927ED00690639 /* SnapshotTesting */ = {
 			isa = XCSwiftPackageProductDependency;
 			package = 16C3F83E2A7927ED00690639 /* XCRemoteSwiftPackageReference "swift-snapshot-testing" */;

diff --git a/iOS Example Frame/iOS Example Frame/ViewController/HomeViewController.swift b/iOS Example Frame/iOS Example Frame/ViewController/HomeViewController.swift
@@ -21,7 +21,7 @@ class HomeViewController: UIViewController {
   @IBOutlet private weak var defaultButton: UIButton!
   @IBOutlet private weak var theme1Button: UIButton!
   @IBOutlet weak var securityCodeComponentButton: UIButton!
-  
+
   private var notificationCenter: NotificationCenter = .default
   private lazy var checkoutAPIService = Frames.CheckoutAPIService(publicKey: Factory.apiKey, environment: .sandbox)
 

diff --git a/iOS Example Frame/iOS Example Frame/ViewController/SecurityCodeViewController.swift b/iOS Example Frame/iOS Example Frame/ViewController/SecurityCodeViewController.swift
@@ -51,7 +51,7 @@ final class SecurityCodeViewController: UIViewController {
 
   override func viewDidLoad() {
     super.viewDidLoad()
-    
+
     defaultSecurityCodeComponent.accessibilityIdentifier = "DefaultSecurityCodeComponent"
     defaultPayButton.accessibilityIdentifier = "DefaultPayButton"
     customSecurityCodeComponent.accessibilityIdentifier = "CustomSecurityCodeComponent"

diff --git a/iOS Example Frame/iOS Example FrameUITests/SecurityCodeComponentUITests.swift b/iOS Example Frame/iOS Example FrameUITests/SecurityCodeComponentUITests.swift
@@ -49,7 +49,9 @@ final class SecurityCodeComponentUITests: XCTestCase {
         XCTAssertFalse(testData.payButton.isEnabled)
       }
       app.enterText(testData.text, into: testData.component)
-      XCTAssertEqual(testData.payButton.isEnabled, testData.isPayButtonEnabled)
+      XCTAssertEqual(testData.payButton.isEnabled,
+                     testData.isPayButtonEnabled,
+                     "Failed at component: \(testData.component), text: \(testData.text)")
     }
 
     // TODO: Add tokenisation test