Lint + generate tests

Author: ctreffs

Sources/Assimp/AiMatKey.swift

@@ -74,7 +74,7 @@ extension AiMatKey {
         case GLTF_PBRMETALLICROUGHNESS_METALLIC_FACTOR = "$mat.gltf.pbrMetallicRoughness.metallicFactor"
         case GLTF_PBRMETALLICROUGHNESS_ROUGHNESS_FACTOR = "$mat.gltf.pbrMetallicRoughness.roughnessFactor"
         case GLTF_ALPHAMODE = "$mat.gltf.alphaMode"
-        case GLTF_ALPHACUTOFF =  "$mat.gltf.alphaCutoff"
+        case GLTF_ALPHACUTOFF = "$mat.gltf.alphaCutoff"
         case GLTF_PBRSPECULARGLOSSINESS = "$mat.gltf.pbrSpecularGlossiness"
         case GLTF_PBRSPECULARGLOSSINESS_GLOSSINESS_FACTOR = "$mat.gltf.pbrMetallicRoughness.glossinessFactor"
         case GLTF_UNLIT = "$mat.gltf.unlit"

Sources/Assimp/AiMaterial.swift

@@ -82,7 +82,6 @@ public struct AiMaterial {
      - aiGetMaterialUVTransform
      - aiGetMaterialXXX
      */
-
     public func getMaterialProperty(_ key: AiMatKey) -> AiMaterialProperty? {
         withUnsafePointer(to: self.material) { matPtr -> AiMaterialProperty? in
             let matPropPtr = UnsafeMutablePointer<UnsafePointer<aiMaterialProperty>?>.allocate(capacity: MemoryLayout<aiMaterialProperty>.stride)

Sources/Assimp/AiMesh.swift

@@ -269,7 +269,7 @@ extension AiMesh: Equatable {
             lhs.textureCoords == rhs.textureCoords &&
             lhs.vertices == rhs.vertices
 
-        //FIXME: lhs.colors == rhs.colors &&
+        // FIXME: lhs.colors == rhs.colors &&
 
     }
 }

Sources/Assimp/AiNode.swift

@@ -25,7 +25,7 @@ public struct AiNode {
     ///
     /// Implementations should be able to handle tokens such as whitespace, tabs, line feeds, quotation marks, ampersands etc.
     ///
-    ///Sometimes assimp introduces new nodes not present in the source file into the hierarchy (usually out of necessity because sometimes the source hierarchy format is simply not compatible).
+    /// Sometimes assimp introduces new nodes not present in the source file into the hierarchy (usually out of necessity because sometimes the source hierarchy format is simply not compatible).
     ///
     /// Their names are surrounded by
     /// `<>`

Sources/Assimp/AiPostProcessStep.swift

@@ -31,8 +31,8 @@ public struct AiPostProcessStep: OptionSet {
     /// Full, weight-based skinning is expensive while animating nodes is extremely cheap,
     /// so this step is offered to clean up the data in that regard.
     ///
-    ///Use #AI_CONFIG_PP_DB_THRESHOLD to control this.
-    ///Use #AI_CONFIG_PP_DB_ALL_OR_NONE if you want bones removed if and only if all bones within the scene qualify for removal.
+    /// Use #AI_CONFIG_PP_DB_THRESHOLD to control this.
+    /// Use #AI_CONFIG_PP_DB_ALL_OR_NONE if you want bones removed if and only if all bones within the scene qualify for removal.
     public static let debone = AiPostProcessStep(rawValue: aiProcess_Debone.rawValue)
 
     /// This step searches all meshes for degenerate primitives and converts them to proper lines or points.

Sources/Assimp/String+aiString.swift

@@ -15,7 +15,7 @@ extension String {
             }
             guard let boundMemory: UnsafePointer<CChar> = bytesPtr.baseAddress?.bindMemory(to: CChar.self,
                                                                                            capacity: Int(aiString.length)) else {
-                                                                                            return nil
+                return nil
             }
 
             let stringBuffer = UnsafeBufferPointer<CChar>(start: boundMemory,
@@ -36,7 +36,7 @@ extension String {
                                                    count: length)
 
         let codeUnits: [UTF8.CodeUnit] = bufferPtr
-            //.map { $0 > 0 ? $0 : Int8(0x20) } // this replaces all invalid characters with blank space
+            // .map { $0 > 0 ? $0 : Int8(0x20) } // this replaces all invalid characters with blank space
             .map { UTF8.CodeUnit($0) }
 
         self.init(decoding: codeUnits, as: UTF8.self)

Tests/AssimpTests/XCTestManifests.swift

@@ -7,17 +7,18 @@ extension AssimpTests {
     // to regenerate.
     static let __allTests__AssimpTests = [
         ("testFailingInitializer", testFailingInitializer),
+        ("testLoadAiScene3DS", testLoadAiScene3DS),
         ("testLoadAiSceneDAE", testLoadAiSceneDAE),
         ("testLoadAiSceneObj", testLoadAiSceneObj),
         ("testVec2fFromAiVector2D", testVec2fFromAiVector2D),
         ("testVec3fFromAiVector3D", testVec3fFromAiVector3D),
-        ("testVersion", testVersion)
+        ("testVersion", testVersion),
     ]
 }
 
 public func __allTests() -> [XCTestCaseEntry] {
     return [
-        testCase(AssimpTests.__allTests__AssimpTests)
+        testCase(AssimpTests.__allTests__AssimpTests),
     ]
 }
 #endif