Skip to content

support copy testing path #25257

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 4 commits into from
Jul 9, 2025
Merged

support copy testing path #25257

merged 4 commits into from
Jul 9, 2025

Conversation

@eleanorjboyd
Copy link
Member

@eleanorjboyd eleanorjboyd commented Jul 8, 2025

fixes #20047

@eleanorjboyd eleanorjboyd self-assigned this Jul 8, 2025
@eleanorjboyd eleanorjboyd added the feature-request Request for new features or functionality label Jul 8, 2025
@eleanorjboyd eleanorjboyd added the skip package*.json package.json and package-lock.json don't both need updating label Jul 8, 2025
@eleanorjboyd eleanorjboyd marked this pull request as ready for review July 8, 2025 20:15
@vs-code-engineering
Copy link

vs-code-engineering bot commented Jul 8, 2025

⚠️ This PR originates from a fork. Due to security restrictions, pipelines from forks are no longer triggered automatically. Learn more.

If the changes appear safe, you can manually trigger the pipeline by commenting /AzurePipelines run.

@vs-code-engineering vs-code-engineering bot added this to the July 2025 milestone Jul 8, 2025
@anthonykim1
Copy link

anthonykim1 commented Jul 9, 2025

@eleanorjboyd Is it intended that path of user.. shows up when copying test id from inner-most test, but not when copying test id from a more upper level (class that contains bunch of other tests)?

Screen.Recording.2025-07-09.at.9.30.28.AM.mov

@eleanorjboyd
Copy link
Member Author

eleanorjboyd commented Jul 9, 2025 •
edited
Loading

hm good catch - is this pytest or unittest you are configured with?

edit: all good fixed it

@eleanorjboyd eleanorjboyd mentioned this pull request Jul 9, 2025
Merged
@eleanorjboyd eleanorjboyd enabled auto-merge (squash) July 9, 2025 20:50
@eleanorjboyd eleanorjboyd merged commit 7607548 into microsoft:main Jul 9, 2025
46 checks passed
@kingbuzzman
Copy link

kingbuzzman commented Jul 10, 2025

My heroine! Thank you so much!!

eleanorjboyd added a commit that referenced this pull request Jul 10, 2025
@eleanorjboyd
switch to absolute ids for class type testing objects (#25266)
0986bdc 
problem with continuity here:
#25257
@MetRonnie MetRonnie mentioned this pull request Aug 21, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@anthonykim1 anthonykim1 anthonykim1 approved these changes

@karthiknadig karthiknadig Awaiting requested review from karthiknadig

Assignees

@eleanorjboyd eleanorjboyd

Labels

feature-request Request for new features or functionality skip package*.json package.json and package-lock.json don't both need updating

Projects

None yet

Milestone

July 2025

Development

Successfully merging this pull request may close these issues.

Add path of function in pytest format to the clipboard from breadcrumb

3 participants

@eleanorjboyd @anthonykim1 @kingbuzzman