hhursev / recipe-scraper / 12536707618

96%

master: 96%

push

github

Update generate.py (#1451)

* Currently, generate.py tries to insert new `from ... import ...` statements after the first line (See https://github.com/hhursev/recipe-scrapers/issues/1259 and https://github.com/hhursev/recipe-scrapers/pull/1427#issuecomment-2526395409).
* It looks like the `node.level > 0` condition was added to skip the first item (`from __future__ import annotations`, which the generator considers to be a valid scraper class), but unfortunately `self.last_node = node` is still reached. The second time `__import()` is called, the first line is `self.last_node` and the new import is added beneath it.
* This PR moves the `node.level > 0` condition up into the parent `if` statement to avoid `self.last_node = node` from being called. Luckily, this doesn't impact the `elif` beneath it.
* I manually validated that we can insert a new scraper class named "AAA" and "ZZZ"

619 of 655 branches covered (94.5%)

7496 of 7839 relevant lines covered (95.62%)

0.96 hits per line