correctly override auto

Author: swhitty

SwiftDraw/Sources/CommandLine/CommandLine.Configuration.swift

@@ -210,11 +210,14 @@ extension CommandLine {
 
     static func parseInsets(from value: String??) throws -> Insets? {
         guard let value = value,
-              let value = value,
-              value != "auto" else {
+              let value = value else {
             return nil
         }
 
+        guard value != "auto" else {
+            return Insets()
+        }
+
         var scanner = XMLParser.Scanner(text: value)
         let top = try scanner.scanInset()
         _ = try scanner.scanString(",")

SwiftDraw/Tests/CommandLine/CommandLine.ArgumentsTests.swift

@@ -42,7 +42,7 @@ final class CommandLineArgumentsTests: XCTestCase {
         XCTAssertEqual(modifiers, [.ultralightInsets: "a", .blackInsets: "b", .hideUnsupportedFilters: nil, .legacy: nil])
 
         modifiers = try CommandLine.parseModifiers(from: ["--ultralight-insets", "a", "--black-insets", "b", "--hide-unsupported-filters", "--legacy"])
-        XCTAssertEqual(modifiers, [.ultralightInsets: "a", .blackInsets: "b", .hideUnsupportedFilters: nil, .legacy: nil])"
+        XCTAssertEqual(modifiers, [.ultralightInsets: "a", .blackInsets: "b", .hideUnsupportedFilters: nil, .legacy: nil])
 
     }
 

SwiftDraw/Tests/CommandLine/CommandLine.ConfigurationTests.swift

@@ -90,9 +90,8 @@ final class CommandLineConfigurationTests: XCTestCase {
     }
 
     func testParseInsets() throws {
-        XCTAssertEqual(
+        XCTAssertNil(
             try CommandLine.parseInsets(from: nil),
-            .init()
         )
         XCTAssertEqual(
             try CommandLine.parseInsets(from: "auto"),