Opened 4 years ago

Closed 4 years ago

#20404 closed New feature (fixed)

Add a keys() method to ContextList

Reported by: Chris Wilson Owned by: nobody
Component: Testing framework Version: master
Severity: Normal Keywords:
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: yes
Easy pickings: yes UI/UX: no

Description

It's useful to be able to list all the (flattened) keys of a
ContextList, to help you figure out why the variable that's supposed to
be there is not. A ContextList is hard to interpret in pdb without a
method like this.

Change History (7)

comment:1 Changed 4 years ago by Claude Paroz

Component: UncategorizedTesting framework
Has patch: unset
Triage Stage: UnreviewedAccepted
Type: UncategorizedNew feature

comment:2 Changed 4 years ago by Russell Keith-Magee

Easy pickings: set

comment:3 Changed 4 years ago by Chris Wilson

comment:4 Changed 4 years ago by EmilStenstrom

Has patch: set
Patch needs improvement: set

comment:5 Changed 4 years ago by Chris Wilson

thanks @mjtamlyn for the review, i've removed the useless init method.

comment:6 Changed 4 years ago by FrankBie

Triage Stage: AcceptedReady for checkin

works for me ready to checkin

comment:7 Changed 4 years ago by Anssi Kääriäinen

Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.
Back to Top