starred/orca
1
0
Fork 0
mirror of https://github.com/stablyai/orca.git synced 2026-09-24 00:02:24 +00:00
Code Issues Packages Projects Releases Wiki Activity

test(relay): type countingIterator as MapIterator and drop two suppressions

Browse Source 
The wrapper only ever receives a Map iterator, so declaring that removes the cast at
both call sites; one irreducible cast stays on the object literal, which cannot satisfy
MapIterator's full surface. Three suppressions become one.
This commit is contained in:
Neil
2026-09-17 00:18:58 -07:00
parent 87eed18ee3
commit 1aaa8937bc
1 changed files with 4 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/relay/relay-hot-path-operation-counts.test.ts
+4 -6
View File
@@ -12,7 +12,7 @@ class CountingMap<K, V> extends Map<K, V> {
visits = 0
getCalls = 0
private countingIterator<T>(inner: IterableIterator<T>): IterableIterator<T> {
private countingIterator<T>(inner: MapIterator<T>): MapIterator<T> {
const bump = (): void => {
this.visits++
}
@@ -28,17 +28,15 @@ class CountingMap<K, V> extends Map<K, V> {
[Symbol.iterator]() {
return this
}
} as IterableIterator<T>
} as MapIterator<T>
}
override [Symbol.iterator](): MapIterator<[K, V]> {
// oxlint-disable-next-line typescript/consistent-type-assertions -- SAFETY: countingIterator wraps the real Map iterator and forwards every next() to it, so the result is that iterator with a counter attached.
return this.countingIterator(super[Symbol.iterator]()) as MapIterator<[K, V]>
return this.countingIterator(super[Symbol.iterator]())
}
override values(): MapIterator<V> {
// oxlint-disable-next-line typescript/consistent-type-assertions -- SAFETY: the same wrapper as [Symbol.iterator] above, over super.values().
return this.countingIterator(super.values()) as MapIterator<V>
return this.countingIterator(super.values())
}
override get(key: K): V | undefined {