Test scapy imports

[gpotter2]

This PR adds a test that imports every file in a new Python session, just to make sure we stop having those dumb circular issues.

The issue was triggered by the test

fix #3246

[codecov]

Codecov Report

Merging #3247 (f4ac0ba) into master (cdeea0a) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #3247   +/-   ##
=======================================
  Coverage   52.38%   52.38%           
=======================================
  Files           9        9           
  Lines        1365     1365           
=======================================
  Hits          715      715           
  Misses        650      650           

[gpotter2]

@guedou If you can help debug this it would be greatly appreciated. Should solve #3246 (once and for all), but I don't have an OSX machine to test it

[guedou]

import.uts does not find any new issue on my macOS.

Do you think that we can make this test faster?

[guedou]

Awesome !

Before the changes:

passed F66DC992 000.03s Prepare importing all scapy files
passed 816770EE 029.09s Try importing all core separately
passed 1D3DC2E3 117.76s Try importing all layers separately
passed B57BFE53 193.02s Try importing all contribs separately

After the changes:

passed 8FC9A72E 000.06s Prepare importing all scapy files
passed 816770EE 021.66s Try importing all core separately
passed 1D3DC2E3 061.64s Try importing all layers separately
passed B57BFE53 116.30s Try importing all contribs separately

This is a quite old laptop. On Github Actions, the whole .uts takes only 40 seconds.

[guedou]

@gpotter2 do you know why some tests are failing?

[gpotter2]

No I currently have no idea

[guedou]

Let’s merge this PR?

…

Sent from my iPhone

On 13 Jul 2021, at 18:49, gpotter2 ***@***.***> wrote:  No I currently have no idea — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or unsubscribe.

[gpotter2]

Sure. I disabled a few imports I thought could be problematic. Those are few contrib modules so it's fine if we don't test them