-
-
Notifications
You must be signed in to change notification settings - Fork 439
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
dealing with empty context names #833
Comments
The empty context might be more than just "imports". It's any code run before an explicit context has been set. When using pytest test contexts, it's all the code run by pytest before a test begins. I understand that "(empty)" isn't very helpful, but I'm not sure what else I can use as a label in the absence of any information. |
Good idea about the foreign keys. I've added those on master. |
In case of
So, any |
It depends how you are running coverage. First, you might not be using pytest at all. Second, you might be running pytest under coverage ("coverage run -m pytest"). In this case, all of pytest is being measured, even the test discovery. If you run coverage from within the pytest plugin ("--cov"), I'm not sure when coverage measurement starts. |
Just run into the same issue as #796.
I'm always frustrated when I don't understand what's happening. Although my guess was the same - that
(empty)
context is an import, I thought I open another issue to clarify how to deal with that.Describe the solution you'd like
The most basic thing that could help is to name the context as
(import)
or{import}
, so that people won't have to guess.Then it could be the top level file where import is coming from as
{import}: filename
Then it could be the import stack
{import}: filename1 -> filename2
Describe alternatives you've considered
The alternative is to display "import stack" in Python flat "import space" rather than "filesystem space". But it is more practical to know the filename rather than
{level1}.somethings.{lamdba15}.something
internal address.Perhaps it is possible to store both.
Additional context
Avoid using word "context" as it is too abstract as the title above says. Once you get the details, it is easy, but the path through abstractions is not. Give more examples and database queries.
Also, add foreign keys to allow diagram tools automatically link tables?
Forgot to add - that stuff is super awesome. =)
The text was updated successfully, but these errors were encountered: