Skip to content
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.

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

Adding file path APIs for "list" #873

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

cpepin
Copy link
Contributor

@cpepin cpepin commented Dec 19, 2024

Summary:

Context

idb list produces an abundance of "Bundle not found at path" errors, as it attempts to read test descriptors from a static symlink directory that is not automatically cleaned. This leads to messy error logs, as old symlinks (that have had their source file removed) cannot be loaded properly.

This diff

  • Adds file path APIs for list so that we load the descriptors from the source file, as opposed to the symlink proxy.

Differential Revision: D67464085

Summary:
## Context

`idb list` produces an abundance of "Bundle not found at path" errors, as it attempts to read test descriptors from a static symlink directory that is not automatically cleaned. This leads to messy error logs, as old symlinks (that have had their source file removed) cannot be loaded properly.

## This diff
- Adds file path APIs for `list` so that we load the descriptors from the source file, as opposed to the symlink proxy.

Differential Revision: D67464085
@facebook-github-bot
Copy link

This pull request was exported from Phabricator. Differential Revision: D67464085

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants