feat: migrate all XCTest to Swift Testing (#52)

Author: zamderax

Tests/CasbinTests/ConfigTests.swift

@@ -2,7 +2,7 @@ import Testing
 import Casbin
 import NIO
 
-@Suite("Config parsing")
+@Suite("Config parsing", .timeLimit(.minutes(1)))
 struct ConfigTests {
     @Test("load from file and get/set")
     func testGet() throws {

Tests/CasbinTests/DefaultModelTests.swift

@@ -1,57 +1,53 @@
-import XCTest
+import Testing
 import NIO
 import Casbin
 
-
-final class RbacApiTests: XCTestCase {
-    var elg = MultiThreadedEventLoopGroup(numberOfThreads: 1)
-    var pool = NIOThreadPool(numberOfThreads: 1)
-    deinit {
-        do {
-            try pool.syncShutdownGracefully()
-            try elg.syncShutdownGracefully()
-        } catch  {
-            
-        }
-    }
-    func makeEnforer(_ mfile:String,_ aFile:String? = nil) throws -> Enforcer {
-        
+@Suite("RBAC API", .timeLimit(.minutes(1)))
+struct RbacApiTests {
+    private func withEnforcer(_ mfile: String, _ aFile: String? = nil, body: (Enforcer) throws -> Void) throws {
+        let pool = NIOThreadPool(numberOfThreads: 1)
         pool.start()
+        let elg = MultiThreadedEventLoopGroup(numberOfThreads: 1)
+        defer {
+            try? elg.syncShutdownGracefully()
+            try? pool.syncShutdownGracefully()
+        }
         let fileIo = NonBlockingFileIO(threadPool: pool)
-        let m = try DefaultModel.from(file:TestsfilePath + mfile , fileIo: fileIo, on: elg.next()).wait()
-        var adapter:Adapter
+        let m = try DefaultModel.from(file: TestsfilePath + mfile, fileIo: fileIo, on: elg.next()).wait()
+        let adapter: Adapter
         if let aFile = aFile {
-            adapter = FileAdapter.init(filePath: TestsfilePath + aFile, fileIo: fileIo, eventloop: elg.next())
+            adapter = FileAdapter(filePath: TestsfilePath + aFile, fileIo: fileIo, eventloop: elg.next())
         } else {
-            adapter = MemoryAdapter.init(on: elg.next())
+            adapter = MemoryAdapter(on: elg.next())
         }
-        let e = try Enforcer.init(m: m, adapter: adapter)
-        return e
+        let e = try Enforcer(m: m, adapter: adapter)
+        e.enableLog = false
+        e.logger.logLevel = .warning
+        try body(e)
     }
-    
-    func testRoleApi() throws {
-        let e = try makeEnforer("examples/rbac_model.conf", "examples/rbac_policy.csv")
-        XCTAssertEqual(["data2_admin"], e.getRoles(for: "alice", domain: nil))
-        XCTAssertEqual([], e.getRoles(for: "bob", domain: nil))
-        XCTAssertEqual([], e.getRoles(for: "data2_admin", domain: nil))
-        XCTAssertEqual([], e.getRoles(for: "non_exists", domain: nil))
-        
-        XCTAssertEqual(false, e.hasRole(for: "alice", role: "data1_admin", domain: nil))
-        XCTAssertEqual(true, e.hasRole(for: "alice", role: "data2_admin", domain: nil))
-       _ = try e.addRole(for: "alice", role: "data1_admin", domain: nil).wait()
-        XCTAssertEqual(["data1_admin", "data2_admin"], e.getRoles(for: "alice", domain: nil).sorted())
-        XCTAssertEqual([], e.getRoles(for: "bob", domain: nil).sorted())
-        XCTAssertEqual([], e.getRoles(for: "data2_admin", domain: nil).sorted())
-        XCTAssertEqual(["read", "write"], e.getAllActions().sorted())
-        //XCTAssertEqual(["data1", "data2"], e.getAllObjects().sorted())
-        //XCTAssertEqual(["alice", "bob", "data2_admin"], e.getAllSubjects().sorted())
-        //XCTAssertEqual(["data1_admin", "data2_admin"], e.getAllRoles().sorted())
+
+    @Test("basic role APIs")
+    func roleApi() throws {
+        try withEnforcer("examples/rbac_model.conf", "examples/rbac_policy.csv") { e in
+            #expect(e.getRoles(for: "alice", domain: nil) == ["data2_admin"])
+            #expect(e.getRoles(for: "bob", domain: nil).isEmpty)
+            #expect(e.getRoles(for: "data2_admin", domain: nil).isEmpty)
+            #expect(e.getRoles(for: "non_exists", domain: nil).isEmpty)
+
+            #expect(!e.hasRole(for: "alice", role: "data1_admin", domain: nil))
+            #expect(e.hasRole(for: "alice", role: "data2_admin", domain: nil))
+            _ = try e.addRole(for: "alice", role: "data1_admin", domain: nil).wait()
+            #expect(e.getRoles(for: "alice", domain: nil).sorted() == ["data1_admin", "data2_admin"])
+            #expect(e.getRoles(for: "bob", domain: nil).sorted() == [])
+            #expect(e.getRoles(for: "data2_admin", domain: nil).sorted() == [])
+            #expect(e.getAllActions().sorted() == ["read", "write"])
+        }
     }
-    func testCoreApi_for_RoleApi_with_domain() throws {
-        let e = try makeEnforer("examples/rbac_with_domains_model.conf", "examples/rbac_with_domains_policy.csv")
-        XCTAssertEqual(["read", "write"], e.getAllActions().sorted())
-        //XCTAssertEqual(["data1", "data2"], e.getAllObjects())
-        //XCTAssertEqual(["admin",], e.getAllSubjects())
-        //XCTAssertEqual(["admin", ], e.getAllRoles())
+
+    @Test("core API with domain")
+    func coreApiWithDomain() throws {
+        try withEnforcer("examples/rbac_with_domains_model.conf", "examples/rbac_with_domains_policy.csv") { e in
+            #expect(e.getAllActions().sorted() == ["read", "write"])
+        }
     }
 }

Tests/CasbinTests/EnforcerTests.swift

@@ -1,6 +1,6 @@
 import Testing
 import Casbin
-@Suite("Role Manager Tests")
+@Suite("Role Manager Tests", .timeLimit(.minutes(1)))
 struct RoleTests {
     struct Params {
         let name1: String

Tests/CasbinTests/RbacApiTests.swift

@@ -0,0 +1,8 @@
+import NIO
+
+// Shared helpers for test file paths and small utilities
+public let TestsfilePath = #file.components(separatedBy: "TestSupport.swift")[0]
+
+@inline(__always)
+func tryBool(_ body: () throws -> Bool) -> Bool { (try? body()) ?? false }
+

Tests/CasbinTests/RoleTests.swift

@@ -1,7 +1,7 @@
 import Testing
 import Casbin
 
-@Suite("Key/Glob/IP match utilities")
+@Suite("Key/Glob/IP match utilities", .timeLimit(.minutes(1)))
 struct KeyMatchTests {
     @Test("keyMatch")
     func testKeyMatch() {
@@ -83,4 +83,3 @@ struct KeyMatchTests {
         #expect(!Util.globMatch("/prefix/subprefix/foobar", "*/foo/*"))
     }
 }
-

Tests/CasbinTests/TestSupport.swift

@@ -1,7 +1,7 @@
 import Testing
 import Casbin
 
-@Suite("Utilities: CSV + assertion escaping")
+@Suite("Utilities: CSV + assertion escaping", .timeLimit(.minutes(1)))
 struct UtilsTests {
     @Test("escapeAssertion replaces dots with underscores")
     func testEscapeAssertion() {