Paths

Table of Contentst

Differential D10818

[lib] Add `threadInfoFromRawThreadInfo` test to `thread-utils.test`
ClosedPublic
Actions

Authored by atul on Jan 25 2024, 7:18 PM.

Tags

None

Referenced Files

None

Subscribers

Details

Reviewers

ashoat
ginsu
tomek

Commits

rCOMM3147b6e753f0: [lib] Add `threadInfoFromRawThreadInfo` test to `thread-utils.test`

Summary

Based on feedback here: https://phab.comm.dev/D10718?id=35839#inline-65319

Depends on D10812

Test Plan

Unit test passes.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

atul created this revision.Jan 25 2024, 7:18 PM

atul mentioned this in D10718: [lib] Remove branching from `threadInfoFromRawThreadInfo(...)`.Jan 25 2024, 7:28 PM

Harbormaster completed remote builds in B26161: Diff 36138.Jan 25 2024, 7:37 PM

atul requested review of this revision.Jan 25 2024, 7:37 PM

atul added a child revision: D10826: [web] Migrate existing `ClientDBThreadInfo`s on `web` to be minimally encoded.Jan 25 2024, 10:27 PM

ashoat accepted this revision.Jan 26 2024, 1:45 AM

ashoat added inline comments.

lib/shared/thread-utils.test.js
77 ↗	(On Diff #36138)	Should we use `threadTypes`?

This revision is now accepted and ready to land.Jan 26 2024, 1:45 AM

rebase

Harbormaster completed remote builds in B26291: Diff 36331.Jan 29 2024, 11:41 PM

atul added inline comments.Jan 30 2024, 1:38 AM

lib/shared/thread-utils.test.js
77 ↗	(On Diff #36138)	Yeah, I think @ginsu recently had a diff where he also cut/paste from Redux and we decided to use `threadTypes` instead of the "raw" number. Will update this diff.

fix

Harbormaster completed remote builds in B26295: Diff 36338.Jan 30 2024, 2:11 AM

I think it was @rohan actually

Closed by commit rCOMM3147b6e753f0: [lib] Add `threadInfoFromRawThreadInfo` test to `thread-utils.test`. · Explain WhyJan 30 2024, 3:16 AM

This revision was automatically updated to reflect the committed changes.

atul added a commit: rCOMM3147b6e753f0: [lib] Add `threadInfoFromRawThreadInfo` test to `thread-utils.test`.

Revision Contents
Changeset List

Path

Size

lib/

shared/

thread-utils.test.js

136 lines

Diff 36342

lib/shared/thread-utils.test.js

Loading...