Further fixes of Resolver by review issues.

yaroslavyaroslav · yaroslavyaroslav · commit 2b5f65bb1bed · 2022-11-10T21:06:49.000+07:00
diff --git a/Sources/Core/Transaction/Policies.swift b/Sources/Core/Transaction/Policies.swift
@@ -53,7 +53,5 @@ public struct Policies {
             self.maxPriorityFeePerGasPolicy = maxPriorityFeePerGasPolicy
         }
 
-    public static var auto: Policies {
-        Policies()
-    }
+    public private(set) static var auto = Policies()
 }
diff --git a/Sources/web3swift/Utils/Hooks/NonceMiddleware.swift b/Sources/web3swift/Utils/Hooks/NonceMiddleware.swift
diff --git a/Sources/web3swift/Web3/Web3+Resolver.swift b/Sources/web3swift/Web3/Web3+Resolver.swift
@@ -17,7 +17,6 @@ public class PolicyResolver {
     }
 
     public func resolveAll(for tx: inout CodableTransaction, with policies: Policies = .auto) async throws {
-        // FIXME: This should throw an appropriate error, since this condition is critical to whole function tast
         if tx.from != nil || tx.sender != nil {
             // Nonce should be resolved first - as this might be needed for some
             // tx's gas estimation

Original file line number	Diff line number	Diff line change
`@@ -53,7 +53,5 @@ public struct Policies {`
`53`	`53`	`self.maxPriorityFeePerGasPolicy = maxPriorityFeePerGasPolicy`
`54`	`54`	`}`
`55`	`55`
`56`		`- public static var auto: Policies {`
`57`		`- Policies()`
`58`		`- }`
	`56`	`+ public private(set) static var auto = Policies()`
`59`	`57`	`}`
Original file line number	Diff line number	Diff line change
`@@ -17,7 +17,6 @@ public class PolicyResolver {`
`17`	`17`	`}`
`18`	`18`
`19`	`19`	`public func resolveAll(for tx: inout CodableTransaction, with policies: Policies = .auto) async throws {`
`20`		`- // FIXME: This should throw an appropriate error, since this condition is critical to whole function tast`
`21`	`20`	`if tx.from != nil \|\| tx.sender != nil {`
`22`	`21`	`// Nonce should be resolved first - as this might be needed for some`
`23`	`22`	`// tx's gas estimation`