try to install redis

Author: ktoso

.github/workflows/pull_request.yml

@@ -15,8 +15,9 @@ jobs:
     name: Unit tests
     uses: swiftlang/github-workflows/.github/workflows/swift_package_test.yml@main
     with:
-      # since we don't have systemctl, we run the redis server manually in the background
-      linux_pre_build_command: apt-get update -y && apt-get install redis -y && redis-server &
       linux_exclude_swift_versions: "[{\"swift_version\": \"5.8\"}]"
+      # since we don't have systemctl, we run the redis server manually in the background
+      linux_pre_build_command: apt-get update -y && apt-get install redis
       linux_env_vars: REDIS_URL=redis
+      linux_build_command: bash -c 'redis-server &; swift test'
       enable_windows_checks: false

Tests/RediStackIntegrationTests/Commands/SortedSetCommandsTests.swift

@@ -328,7 +328,7 @@ final class SortedSetCommandsTests: RediStackIntegrationTestCase {
         XCTAssertEqual(elements.count, 6)
 
         let values = try RedisConnection._mapSortedSetResponse(elements, scoreIsFirst: false)
-            .map { (value, _) in return Int(fromRESP: value) }
+            .map { (value, _) in Int(fromRESP: value) }
 
         XCTAssertEqual(values[0], 9)
         XCTAssertEqual(values[1], 8)
@@ -352,7 +352,7 @@ final class SortedSetCommandsTests: RediStackIntegrationTestCase {
         XCTAssertEqual(elements.count, 6)
 
         let values = try RedisConnection._mapSortedSetResponse(elements, scoreIsFirst: false)
-            .map { (_, score) in return score }
+            .map { (_, score) in score }
 
         XCTAssertEqual(values[0], 1.0)
         XCTAssertEqual(values[1], 2.0)
@@ -376,7 +376,7 @@ final class SortedSetCommandsTests: RediStackIntegrationTestCase {
         XCTAssertEqual(elements.count, 6)
 
         let values = try RedisConnection._mapSortedSetResponse(elements, scoreIsFirst: false)
-            .map { (_, score) in return score }
+            .map { (_, score) in score }
 
         XCTAssertEqual(values[0], 3.0)
         XCTAssertEqual(values[1], 2.0)

Tests/RediStackTests/ChannelHandlers/RedisByteDecoderTests.swift

@@ -155,8 +155,8 @@ extension RedisByteDecoderTests {
 
         var arraysMatch = true
 
-        left.enumerated().forEach {
-            let (offset, decodedElement) = $0
+        for item in left.enumerated() {
+            let (offset, decodedElement) = item
 
             switch (decodedElement, right[offset]) {
             case (let .bulkString(decoded), let .bulkString(expected)): arraysMatch = decoded == expected