Skip to content

Commit adc41a4

Browse files
committed
Updated tests to use XCTAssertEqual where possible
1 parent 0f9da13 commit adc41a4

File tree

4 files changed

+14
-15
lines changed

4 files changed

+14
-15
lines changed

Tests/InstrumentationTests/sdk-resource-extension-tests/ApplicationResourceProviderTests.swift

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ import XCTest
2626

2727
let resource = provider.create()
2828

29-
XCTAssertTrue(appData.name == resource.attributes[ResourceAttributes.serviceName.rawValue]?.description)
29+
XCTAssertEqual(appData.name, resource.attributes[ResourceAttributes.serviceName.rawValue]?.description)
3030
let versionDescription = resource.attributes[ResourceAttributes.serviceVersion.rawValue]?.description
3131

3232
XCTAssertNotNil(versionDescription)
@@ -42,12 +42,12 @@ import XCTest
4242

4343
let resource = provider.create()
4444

45-
XCTAssertTrue(appData.name == resource.attributes[ResourceAttributes.serviceName.rawValue]?.description)
45+
XCTAssertEqual(appData.name, resource.attributes[ResourceAttributes.serviceName.rawValue]?.description)
4646
let versionDescription = resource.attributes[ResourceAttributes.serviceVersion.rawValue]?.description
4747

4848
XCTAssertNotNil(versionDescription)
4949

50-
XCTAssertTrue(versionDescription! == appData.version!)
50+
XCTAssertEqual(versionDescription, appData.version)
5151
}
5252

5353
func testNoShortVersion() {
@@ -57,12 +57,11 @@ import XCTest
5757

5858
let resource = provider.create()
5959

60-
XCTAssertTrue(appData.name == resource.attributes[ResourceAttributes.serviceName.rawValue]?.description)
60+
XCTAssertEqual(appData.name, resource.attributes[ResourceAttributes.serviceName.rawValue]?.description)
6161
let versionDescription = resource.attributes[ResourceAttributes.serviceVersion.rawValue]?.description
6262

6363
XCTAssertNotNil(versionDescription)
64-
65-
XCTAssertTrue(versionDescription! == appData.build!)
64+
XCTAssertEqual(versionDescription, appData.build)
6665
}
6766

6867
func testNoAppData() {

Tests/InstrumentationTests/sdk-resource-extension-tests/DeviceResourceProviderTests.swift

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ class DeviceResourceProviderTests: XCTestCase {
2626

2727
let resource = provider.create()
2828

29-
XCTAssertTrue(resource.attributes["device.model"]?.description == mock.model)
30-
XCTAssertTrue(resource.attributes[ResourceAttributes.hostId.rawValue]?.description == mock.identifier)
29+
XCTAssertEqual(mock.model, resource.attributes["device.model"]?.description)
30+
XCTAssertEqual(mock.identifier, resource.attributes[ResourceAttributes.hostId.rawValue]?.description)
3131
}
3232
}

Tests/InstrumentationTests/sdk-resource-extension-tests/OSResourceProviderTests.swift

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,8 +27,8 @@ import XCTest
2727

2828
let resource = provider.create()
2929

30-
XCTAssertTrue(resource.attributes["os.type"]?.description == mock.type)
31-
XCTAssertTrue(resource.attributes["os.description"]?.description == mock.description)
30+
XCTAssertEqual(mock.type, resource.attributes["os.type"]?.description)
31+
XCTAssertEqual(mock.description, resource.attributes["os.description"]?.description)
3232
}
3333
}
3434

Tests/InstrumentationTests/sdk-resource-extension-tests/TelemetryResourceProviderTests.swift

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -32,9 +32,9 @@ class TelemetryResourceProviderTests: XCTestCase {
3232

3333
let resource = provider.create()
3434

35-
XCTAssertTrue(resource.attributes["telemetry.sdk.name"]?.description == mock.name)
36-
XCTAssertTrue(resource.attributes["telemetry.sdk.language"]?.description == mock.language)
37-
XCTAssertTrue(resource.attributes["telemetry.sdk.version"]?.description == mock.version)
35+
XCTAssertEqual(mock.name, resource.attributes["telemetry.sdk.name"]?.description)
36+
XCTAssertEqual(mock.language, resource.attributes["telemetry.sdk.language"]?.description)
37+
XCTAssertEqual(mock.version, resource.attributes["telemetry.sdk.version"]?.description)
3838
}
3939

4040
func testNils() {
@@ -43,8 +43,8 @@ class TelemetryResourceProviderTests: XCTestCase {
4343

4444
let resource = provider.create()
4545

46-
XCTAssertTrue(resource.attributes["telemetry.sdk.name"]?.description == mock.name)
47-
XCTAssertTrue(resource.attributes["telemetry.sdk.language"]?.description == mock.language)
46+
XCTAssertEqual(mock.name, resource.attributes["telemetry.sdk.name"]?.description)
47+
XCTAssertEqual(mock.language, resource.attributes["telemetry.sdk.language"]?.description)
4848
XCTAssertNil(resource.attributes["telemetry.sdk.version"])
4949
}
5050
}

0 commit comments

Comments
 (0)