fix a Hashable risk

Author: winddpan

Sources/CodableWrapper/Codec.swift

@@ -52,6 +52,14 @@ extension Codec: CustomStringConvertible, CustomDebugStringConvertible {
     }
 }
 
-extension Codec: Equatable where Value: Equatable { }
+extension Codec: Equatable where Value: Equatable {
+    public static func == (lhs: Codec<Value>, rhs: Codec<Value>) -> Bool {
+        lhs.wrappedValue == rhs.wrappedValue
+    }
+}
 
-extension Codec: Hashable where Value: Hashable { }
+extension Codec: Hashable where Value: Hashable {
+    public func hash(into hasher: inout Hasher) {
+        hasher.combine(wrappedValue)
+    }
+}

Sources/CodableWrapper/CodecCore/CodecConstruct.swift

@@ -7,7 +7,7 @@
 
 import Foundation
 
-enum CodecConstructKey: Hashable {
+enum CodecConstructKey {
     case noNested(String)
     case nested([String])
 
@@ -25,7 +25,7 @@ enum CodecConstructKey: Hashable {
     }
 }
 
-final class CodecConstruct<Value>: Hashable {
+final class CodecConstruct<Value> {
     var codingKeys: [CodecConstructKey] = []
     var transformer: AnyTransfromTypeTunk?
     var storedValue: Value?
@@ -53,13 +53,4 @@ final class CodecConstruct<Value>: Hashable {
         self.codingKeys = codingKeys
         self.transformer = transformer
     }
-
-    func hash(into hasher: inout Hasher) {
-        hasher.combine(self.codingKeys)
-        hasher.combine(self.transformer?.hashValue)
-    }
-
-    static func == (lhs: CodecConstruct, rhs: CodecConstruct) -> Bool {
-        return lhs.hashValue == rhs.hashValue
-    }
 }

Tests/CodableWrapperTests/ExtensionTest.swift

@@ -0,0 +1,43 @@
+//
+//  ExtensionTest.swift
+//
+//
+//  Created by winddpan on 2023/1/5.
+//
+
+import CodableWrapper
+import XCTest
+
+final class ExtensionTest: XCTestCase {
+    struct HashableModel: Codable, Hashable {
+        @Codec var value1: String?
+    }
+
+    struct NavtiveHashableModel: Codable, Hashable {
+        var value1: String?
+    }
+
+    struct NavtiveHashableModel2: Codable, Hashable {
+        var value1: String?
+    }
+
+    override func setUpWithError() throws {
+        // Put setup code here. This method is called before the invocation of each test method in the class.
+    }
+
+    override func tearDownWithError() throws {
+        // Put teardown code here. This method is called after the invocation of each test method in the class.
+    }
+
+    func testHashable() throws {
+        let a = HashableModel(value1: "abc")
+        let b = HashableModel(value1: "abc")
+
+        let c = NavtiveHashableModel(value1: "abc")
+        let d = NavtiveHashableModel2(value1: "abc")
+
+        XCTAssertEqual(a.hashValue, b.hashValue)
+        XCTAssertEqual(a.hashValue, c.hashValue)
+        XCTAssertEqual(c.hashValue, d.hashValue)
+    }
+}