3 ms·
Impressive for only 4h of work and good caveats list in the documentation. I can't see the restriction on number of path segments in the code ("/api/ followed b
by mtmail 3y ago
Impressive for only 4h of work and good caveats list in the documentation. I can't see the restriction on number of path segments in the code ("/api/ followed by 1 to 6 path segments") and they might have insisted on tests while not mentioning tests. Nobody can tell what "production readiness, and code structure" might mean for them.
- formulathree 3y ago>Impressive for only 4h of work and good caveats list in the documentation. Thanks! They requested me to spend only 4 hours on it. > I can't see the restriction on number of path segments in the code ("/api/ followed by 1 to 6 path segments") and they might have insisted on tests while not mentioning tests. the segment amount is restricted via a default value in utils.py: def generate_test_paths(test_path_amount: int, segment_amount: int = 6, #here! string_pool_amount: int = 3, string_length: int = 3) -> List[str]: The segment_amount gets inserted into this lambda: segments = (segment_generator(random.randint(1, segment_amount)) for _ in range(test_path_amount)) which generates a list of strings from a pool of pre-selected strings. Perhaps this section isn't clear? >Nobody can tell what "production readiness, and code structure" might mean for them. yeah I hate that.