Merge pull request #173 from nachoBonafonte/minor-details

Rename folder sdk-resource-extension-tests to SDKResourceExtensionTests for consistency with other source code
Make InMemoryExporter usable by making constructor public and fix formatting

Author: Ignacio Bonafonte

Package.swift

@@ -99,7 +99,7 @@ let package = Package(
                     path: "Tests/OpenTelemetryApiTests"),
         .testTarget(name: "ResourceExtensionTests",
                     dependencies: ["ResourceExtension", "OpenTelemetrySdk"],
-                    path: "Tests/InstrumentationTests/sdk-resource-extension-tests"),
+                    path: "Tests/InstrumentationTests/SDKResourceExtensionTests"),
         .testTarget(name: "OpenTracingShimTests",
                     dependencies: ["OpenTracingShim",
                                    "OpenTelemetrySdk"],

Sources/Exporters/InMemory/InMemoryExporter.swift

@@ -17,36 +17,38 @@ import Foundation
 import OpenTelemetrySdk
 
 public class InMemoryExporter: SpanExporter {
-  private var finishedSpanItems: [SpanData] = []
-  private var isRunning: Bool = true
+    private var finishedSpanItems: [SpanData] = []
+    private var isRunning: Bool = true
 
-  public func getFinishedSpanItems() -> [SpanData] {
-    return finishedSpanItems
-  }
+    public init() {}
 
-  public func export(spans: [SpanData]) -> SpanExporterResultCode {
-    guard isRunning else {
-      return .failure
+    public func getFinishedSpanItems() -> [SpanData] {
+        return finishedSpanItems
     }
 
-    finishedSpanItems.append(contentsOf: spans)
-    return .success
-  }
+    public func export(spans: [SpanData]) -> SpanExporterResultCode {
+        guard isRunning else {
+            return .failure
+        }
 
-  public func flush() -> SpanExporterResultCode {
-    guard isRunning else {
-      return .failure
+        finishedSpanItems.append(contentsOf: spans)
+        return .success
     }
 
-    return .success
-  }
+    public func flush() -> SpanExporterResultCode {
+        guard isRunning else {
+            return .failure
+        }
 
-  public func reset() {
-    finishedSpanItems.removeAll()
-  }
+        return .success
+    }
+
+    public func reset() {
+        finishedSpanItems.removeAll()
+    }
 
-  public func shutdown() {
-    finishedSpanItems.removeAll()
-    isRunning = false
-  }
+    public func shutdown() {
+        finishedSpanItems.removeAll()
+        isRunning = false
+    }
 }

Tests/InstrumentationTests/sdk-resource-extension-tests/ApplicationResourceProviderTests.swift renamed to Tests/InstrumentationTests/SDKResourceExtensionTests/ApplicationResourceProviderTests.swift

@@ -19,8 +19,8 @@ import OpenTelemetrySdk
 @testable import ResourceExtension
 import XCTest
 
-class ResourcePropogationTests : XCTestCase {
-    func testPropogation() {
+class ResourcePropagationTests : XCTestCase {
+    func testPropagation() {
         let defaultResource = Resource()
         let appProvider = ApplicationResourceProvider(source: ApplicationDataSource())
         let telemetryProvider = TelemetryResourceProvider(source: TelemetryDataSource())