MobileNativeFoundation / bluepill

Bluepill is a reliable iOS testing tool that runs UI tests using multiple simulators on a single machine
BSD 2-Clause "Simplified" License
3.18k stars 231 forks source link

Add BP index to fix trace-profile issue when BPs are reused #481

Closed YangSun33 closed 3 years ago

YangSun33 commented 3 years ago

Issue: When a BP is reused, the id increases by 1 automatically, like BP-1 & BP-5. In the trace-profile, the stats will be listed in 2 separated lines, which is not friendly to check.

Logic Changes: Added a parameter "_BP_INDEX" for bps, which would be 1 ~ SIMS_NUM, and keeps the same when a BP is reused.

Trace-profile Changes:

  1. Reused BPs are listed in the same line.
  2. Thread-name: "BP-[ID]" -> "BP #[INDEX]"
  3. Event-name: "bp ([PROCESS_ID])" -> "BP-[ID] ([PROCESS_ID])"
YangSun33 commented 3 years ago

Close it as we have another PR submitted.