CodeUnit doesn't handle eggs

[nedbat]

Originally reported by rozza (Bitbucket: rozza, GitHub: rozza)

---

CodeUnit fails with eggs as the files aren't actually available for reading.

Patch included

---

• Bitbucket: https://bitbucket.org/ned/coveragepy/issue/25
• This issue had attachments: codeunit.patch. See the original issue for details.

[nedbat]

Issue #26 was marked as a duplicate of this issue.

[nedbat]

Thanks! This is fixed in <<changeset 92b5372042d2 (bb)>>

[nedbat]

Original comment by Vanuan (Bitbucket: Vanuan, GitHub: Vanuan)

---

Alas, it is not fixed everywhere consistently.

results.py and codeunit.py contain duplicated code.

The bug still exists in results.py:

https://bitbucket.org/ned/coveragepy/src/a621660fa6e671477500b723a8ead869c0bdfea5/coverage/results.py?at=default#cl-90

if source:

should be

if source is not None:

[nedbat]

Original comment by Vanuan (Bitbucket: Vanuan, GitHub: Vanuan)

---

Similar bugs in parser.py (lines 27, 337)

https://bitbucket.org/ned/coveragepy/src/a621660fa6e6/coverage/parser.py?at=default#cl-27
https://bitbucket.org/ned/coveragepy/src/a621660fa6e6/coverage/parser.py?at=default#cl-337

should be fixed the same way.

This is due to the fact that the file __init__.py could be empty (its length is zero).

[nedbat]

Original comment by Vanuan (Bitbucket: Vanuan, GitHub: Vanuan)

---

The fix is trivial: Vanuan/coveragepy@f453ced

[nedbat]

Vanuan: can you provide some test scenarios?

[nedbat]

Original comment by Vanuan (Bitbucket: Vanuan, GitHub: Vanuan)

---

I'm afraid I need some help here.

The test case should look like that:

• coverage run import_egg1.py
• coverage html

where import_egg1.py contains import egg1 and egg1 is an egg module (and contains an empty __init__.py).

My best effort is here: Vanuan/coveragepy@7cba392

[nedbat]

I tried to duplicate this using the egg1 egg in the tests directory, with coverage 4.0. There were no problems. Maybe I don't know how to reproduce it. If you can give me manual steps to make it fail, I'll re-open the issue.