Skip to content

Commit 00685b8

Browse files
committed
[Workspace] Add lookup() to DependencyManifests
1 parent b11aded commit 00685b8

File tree

2 files changed

+10
-9
lines changed

2 files changed

+10
-9
lines changed

Sources/Commands/Workspace.swift

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -176,6 +176,11 @@ public class Workspace {
176176
return requiredURLs.subtracting(availableURLs)
177177
}
178178

179+
/// Find a manifest given its name.
180+
func lookup(_ name: String) -> Manifest? {
181+
return dependencies.first(where: { $0.name == name })
182+
}
183+
179184
init(root: Manifest, dependencies: [Manifest]) {
180185
self.root = root
181186
self.dependencies = dependencies

Tests/CommandsTests/WorkspaceTests.swift

Lines changed: 5 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -159,17 +159,13 @@ final class WorkspaceTests: XCTestCase {
159159
XCTAssertEqual(manifests.root.package, graph.rootManifest.package)
160160
// B should be missing.
161161
XCTAssertEqual(manifests.missingURLs(), ["//B"])
162-
var dependencyManifests: [String: Manifest] = [:]
163-
for manifest in manifests.dependencies {
164-
dependencyManifests[manifest.package.name] = manifest
165-
}
166-
XCTAssertEqual(dependencyManifests.keys.sorted(), ["A", "AA"])
162+
XCTAssertEqual(manifests.dependencies.map{$0.name}.sorted(), ["A", "AA"])
167163
let aManifest = graph.manifest("A", version: v1)
168-
XCTAssertEqual(dependencyManifests["A"]?.package, aManifest.package)
169-
XCTAssertEqual(dependencyManifests["A"]?.version, aManifest.version)
164+
XCTAssertEqual(manifests.lookup("A")?.package, aManifest.package)
165+
XCTAssertEqual(manifests.lookup("A")?.version, aManifest.version)
170166
let aaManifest = graph.manifest("AA", version: v1)
171-
XCTAssertEqual(dependencyManifests["AA"]?.package, aaManifest.package)
172-
XCTAssertEqual(dependencyManifests["AA"]?.version, aaManifest.version)
167+
XCTAssertEqual(manifests.lookup("AA")?.package, aaManifest.package)
168+
XCTAssertEqual(manifests.lookup("AA")?.version, aaManifest.version)
173169
}
174170
}
175171

0 commit comments

Comments
 (0)