Fix SPM warnings (#4)

Author: mickael-menu

.github/workflows/checks.yml

@@ -92,10 +92,12 @@ jobs:
         run: |
           xcodebuild build -scheme TestApp -destination "platform=$platform,name=$device"
 
-  # Warning: This cannot actually test the state of the current commit, but
-  # will check that the latest branch/tag set in the Podspecs are valid.
   int-cocoapods:
     name: Integration (CocoaPods)
+    # Warning: This job cannot actually test the state of the current commit,
+    # but will check that the latest branch/tag set in the Podspecs are valid.
+    # That's the reason for running only with push events.
+    if: github.event_name == 'push'
     runs-on: macos-latest
     defaults:
       run:

Package.swift

@@ -35,7 +35,6 @@ let package = Package(
             dependencies: ["Fuzi", "SwiftSoup", "Zip"],
             path: "Sources/Shared",
             exclude: [
-                "Info.plist",
                 // Support for ZIPFoundation is not yet achieved.
                 "Toolkit/Archive/ZIPFoundation.swift",
             ],
@@ -51,7 +50,6 @@ let package = Package(
             name: "R2SharedTests",
             dependencies: ["R2Shared"],
             path: "Tests/SharedTests",
-            exclude: ["Info.plist"],
             resources: [
                 .copy("Fixtures"),
             ]
@@ -67,7 +65,6 @@ let package = Package(
                 "R2Shared"
             ],
             path: "Sources/Streamer",
-            exclude: ["Info.plist"],
             resources: [
                 .copy("Assets"),
             ]
@@ -76,7 +73,6 @@ let package = Package(
             name: "R2StreamerTests",
             dependencies: ["R2Streamer"],
             path: "Tests/StreamerTests",
-            exclude: ["Info.plist"],
             resources: [
                 .copy("Fixtures"),
             ]
@@ -91,7 +87,6 @@ let package = Package(
             ],
             path: "Sources/Navigator",
             exclude: [
-                "Info.plist",
                 "EPUB/Scripts",
             ],
             resources: [
@@ -102,8 +97,7 @@ let package = Package(
         .testTarget(
             name: "R2NavigatorTests",
             dependencies: ["R2Navigator"],
-            path: "Tests/NavigatorTests",
-            exclude: ["Info.plist"]
+            path: "Tests/NavigatorTests"
         ),
 
         .target(
@@ -112,14 +106,12 @@ let package = Package(
                 "Fuzi",
                 "R2Shared"
             ],
-            path: "Sources/OPDS",
-            exclude: ["Info.plist"]
+            path: "Sources/OPDS"
         ),
         .testTarget(
             name: "ReadiumOPDSTests",
             dependencies: ["ReadiumOPDS"],
             path: "Tests/OPDSTests",
-            exclude: ["Info.plist"],
             resources: [
                 .copy("Samples"),
             ]
@@ -134,7 +126,6 @@ let package = Package(
                 .product(name: "SQLite", package: "SQLite.swift"),
             ],
             path: "Sources/LCP",
-            exclude: ["Info.plist"],
             resources: [
                 .process("Resources")
             ]
@@ -145,7 +136,6 @@ let package = Package(
         //     name: "ReadiumLCPTests",
         //     dependencies: ["ReadiumLCP"],
         //     path: "Tests/LCPTests",
-        //     exclude: ["Info.plist"],
         //     resources: [
         //         .copy("Fixtures"),
         //     ]

Tests/LCPTests/Info.plist

@@ -1,36 +1,27 @@
-//
-//  r2_navigator_swiftTests.swift
-//  r2-navigator-swiftTests
-//
-//  Created by Alexandre Camilleri on 8/23/17.
-//  Copyright © 2017 Readium. All rights reserved.
-//
-
 import XCTest
 @testable import R2Navigator
 
-class r2_navigator_swiftTests: XCTestCase {
-    
-    override func setUp() {
-        super.setUp()
+class NavigatorTests: XCTestCase {
+
+    override func setUpWithError() throws {
         // Put setup code here. This method is called before the invocation of each test method in the class.
     }
-    
-    override func tearDown() {
+
+    override func tearDownWithError() throws {
         // Put teardown code here. This method is called after the invocation of each test method in the class.
-        super.tearDown()
     }
-    
-    func testExample() {
+
+    func testExample() throws {
         // This is an example of a functional test case.
         // Use XCTAssert and related functions to verify your tests produce the correct results.
     }
-    
-    func testPerformanceExample() {
+
+    func testPerformanceExample() throws {
         // This is an example of a performance test case.
         self.measure {
             // Put the code you want to measure the time of here.
         }
     }
-    
+
 }
+