Skip to content

Commit 5ecf03d

Browse files
committed
clean up formatting
1 parent 19627d6 commit 5ecf03d

File tree

1 file changed

+34
-9
lines changed

1 file changed

+34
-9
lines changed

Tests/PackageGraphPerformanceTests/PackageGraphPerfTests.swift

Lines changed: 34 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -22,8 +22,7 @@ import class TSCBasic.InMemoryFileSystem
2222

2323
import class TSCTestSupport.XCTestCasePerf
2424

25-
class PackageGraphPerfTests: XCTestCasePerf {
26-
25+
final class PackageGraphPerfTests: XCTestCasePerf {
2726
func testLoading100Packages() throws {
2827
#if !os(macOS)
2928
try XCTSkipIf(true, "test is only supported on macOS")
@@ -49,8 +48,16 @@ class PackageGraphPerfTests: XCTestCasePerf {
4948
} else {
5049
let depName = "Foo\(pkg + 1)"
5150
let depUrl = "/\(depName)"
52-
dependencies = [.localSourceControl(deprecatedName: depName, path: try .init(validating: depUrl), requirement: .upToNextMajor(from: "1.0.0"))]
53-
targets = [try TargetDescription(name: name, dependencies: [.byName(name: depName, condition: nil)], path: ".")]
51+
dependencies = [.localSourceControl(
52+
deprecatedName: depName,
53+
path: try .init(validating: depUrl),
54+
requirement: .upToNextMajor(from: "1.0.0")
55+
)]
56+
targets = [try TargetDescription(
57+
name: name,
58+
dependencies: [.byName(name: depName, condition: nil)],
59+
path: "."
60+
)]
5461
}
5562
// Create manifest.
5663
let isRoot = pkg == 1
@@ -79,7 +86,11 @@ class PackageGraphPerfTests: XCTestCasePerf {
7986
measure {
8087
let observability = ObservabilitySystem.makeForTesting()
8188
let g = try! PackageGraph.load(
82-
root: PackageGraphRoot(input: PackageGraphRootInput(packages: [rootManifest.path]), manifests: [rootManifest.path: rootManifest], observabilityScope: observability.topScope),
89+
root: PackageGraphRoot(
90+
input: PackageGraphRootInput(packages: [rootManifest.path]),
91+
manifests: [rootManifest.path: rootManifest],
92+
observabilityScope: observability.topScope
93+
),
8394
identityResolver: identityResolver,
8495
externalManifests: externalManifests,
8596
binaryArtifacts: [:],
@@ -95,7 +106,10 @@ class PackageGraphPerfTests: XCTestCasePerf {
95106
let lastPackageNumber = 20
96107
let packageNumberSequence = (1...lastPackageNumber)
97108

98-
let fs = InMemoryFileSystem(emptyFiles: packageNumberSequence.map({ "/Package\($0)/Sources/Target\($0)/s.swift" }) + ["/PackageA/Sources/TargetA/s.swift"])
109+
let fs = InMemoryFileSystem(
110+
emptyFiles: packageNumberSequence.map({ "/Package\($0)/Sources/Target\($0)/s.swift" }) +
111+
["/PackageA/Sources/TargetA/s.swift"]
112+
)
99113

100114
let packageSequence: [Manifest] = try packageNumberSequence.map { (sequenceNumber: Int) -> Manifest in
101115
let dependencySequence = sequenceNumber < lastPackageNumber ? Array((sequenceNumber + 1)...lastPackageNumber) : []
@@ -105,10 +119,17 @@ class PackageGraphPerfTests: XCTestCasePerf {
105119
toolsVersion: .v5_7,
106120
dependencies: try dependencySequence.map({ .fileSystem(path: try .init(validating: "/Package\($0)")) }),
107121
products: [
108-
try .init(name: "Package\(sequenceNumber)", type: .library(.dynamic), targets: ["Target\(sequenceNumber)"])
122+
try .init(
123+
name: "Package\(sequenceNumber)",
124+
type: .library(.dynamic),
125+
targets: ["Target\(sequenceNumber)"]
126+
)
109127
],
110128
targets: [
111-
try .init(name: "Target\(sequenceNumber)", dependencies: dependencySequence.map { .product(name: "Target\($0)", package: "Package\($0)") })
129+
try .init(name: "Target\(sequenceNumber)",
130+
dependencies: dependencySequence.map {
131+
.product(name: "Target\($0)", package: "Package\($0)")
132+
})
112133
]
113134
)
114135
}
@@ -127,7 +148,11 @@ class PackageGraphPerfTests: XCTestCasePerf {
127148
measure {
128149
do {
129150
for _ in 0..<N {
130-
_ = try loadPackageGraph(fileSystem: fs, manifests: [root] + packageSequence, observabilityScope: observability.topScope)
151+
_ = try loadPackageGraph(
152+
fileSystem: fs,
153+
manifests: [root] + packageSequence,
154+
observabilityScope: observability.topScope
155+
)
131156
}
132157
} catch {
133158
XCTFail("Loading package graph is not expected to fail in this test.")

0 commit comments

Comments
 (0)