fix(scrapper): manage audio and video requests

[PaulPHPE]

Modify get_requests_from_har_file method to take into account mp3 and audio requests.

Basically manage cases when _transferSize in the HTTP request is equal to -1.

close #30

[github-actions]

Branch coverage •

File	Stmts	Miss	Cover	Missing
components/ecoindex/scraper
scrap.py	95	51	46%	45, 48–49, 51, 60, 63, 66–68, 73–75, 77–81, 84–87, 89, 91, 98–101, 108–110, 112–121, 130–131, 134–135, 137, 152–153, 158–161
TOTAL	699	230	67%

[vvatelot]

I am reading the #30 issue again... I think I was not completely clear 🙂 : In this case as it is a mp3 file, the analysis should return an error: We only accept to analyse html content. This is the behaviour we had in the old scraper and I did not implement it yet...

So, here I think we have 2 issues:

• The first one is the one regarding partial responses that your PR covers
• The second one is about the type of content that we analyse

What do you think ?

[PaulPHPE]

Hello @vvatelot !

Ah alright ! I will modify the code to raise an error when the url doesn't point to a text/html content type.