diff --git a/map_machine/moire_manager.py b/map_machine/moire_manager.py index 4de1937..659ea81 100644 --- a/map_machine/moire_manager.py +++ b/map_machine/moire_manager.py @@ -11,7 +11,7 @@ from moire.moire import Tag from map_machine import ui from map_machine.icon import ShapeExtractor -from map_machine.ui import COMMANDS +from map_machine.ui import COMMAND_LINES from map_machine.workspace import workspace __author__ = "Sergey Vartanov" @@ -143,7 +143,7 @@ class MapMachineMoire(Default, ABC): def command(self, args: Arguments) -> str: """Bash command from integration tests.""" - return "map-machine " + " ".join(COMMANDS[self.clear(args[0])]) + return "map-machine " + " ".join(COMMAND_LINES[self.clear(args[0])]) def icon(self, args: Arguments) -> str: """Image with Röntgen icon.""" diff --git a/map_machine/ui.py b/map_machine/ui.py index 3f1510c..b5a45d7 100644 --- a/map_machine/ui.py +++ b/map_machine/ui.py @@ -14,7 +14,7 @@ __email__ = "me@enzet.ru" BOXES: str = " ▏▎▍▌▋▊▉" BOXES_LENGTH: int = len(BOXES) -COMMANDS: dict[str, list[str]] = { +COMMAND_LINES: dict[str, list[str]] = { "render": ["render", "-b", "10.000,20.000,10.001,20.001"], "render_with_tooltips": [ "render", @@ -27,6 +27,15 @@ COMMANDS: dict[str, list[str]] = { "element": ["element", "--node", "amenity=bench,material=wood"], "tile": ["tile", "--coordinates", "50.000,40.000"], } +COMMANDS: list[str] = [ + "render", + "server", + "tile", + "element", + "mapcss", + "icons", + "taginfo", +] def parse_arguments(args: list[str]) -> argparse.Namespace: @@ -163,7 +172,7 @@ def add_tile_arguments(parser: argparse.ArgumentParser) -> None: parser.add_argument( "-b", "--boundary-box", - help="construct the minimum amount of tiles that cover requested " + help="construct the minimum amount of tiles that cover the requested " "boundary box", metavar=",,,", ) @@ -181,7 +190,7 @@ def add_tile_arguments(parser: argparse.ArgumentParser) -> None: "--input", dest="input_file_name", metavar="", - help="input OSM XML file name (if not specified, file will be " + help="input OSM XML file name (if not specified, the file will be " "downloaded using OpenStreetMap API)", ) @@ -233,8 +242,8 @@ def add_render_arguments(parser: argparse.ArgumentParser) -> None: "-b", "--boundary-box", metavar=",,,", - help="geo boundary box; if first value is negative, enclose the value " - "with quotes and use space before `-`", + help="geo boundary box; if the first value is negative, enclose the " + "value with quotes and use space before `-`", ) parser.add_argument( "--cache", diff --git a/tests/test_command_line.py b/tests/test_command_line.py index 6da6104..81f624f 100644 --- a/tests/test_command_line.py +++ b/tests/test_command_line.py @@ -10,7 +10,7 @@ __email__ = "me@enzet.ru" from xml.etree import ElementTree from xml.etree.ElementTree import Element -from map_machine.ui import COMMANDS +from map_machine.ui import COMMAND_LINES def error_run(arguments: list[str], message: bytes) -> None: @@ -41,7 +41,7 @@ def test_wrong_render_arguments() -> None: def test_render() -> None: """Test `render` command.""" run( - COMMANDS["render"] + ["--cache", "tests/data"], + COMMAND_LINES["render"] + ["--cache", "tests/data"], b"INFO Writing output SVG to out/map.svg...\n", ) with Path("out/map.svg").open() as output_file: @@ -58,7 +58,7 @@ def test_render() -> None: def test_render_with_tooltips() -> None: """Test `render` command.""" run( - COMMANDS["render_with_tooltips"] + ["--cache", "tests/data"], + COMMAND_LINES["render_with_tooltips"] + ["--cache", "tests/data"], b"INFO Writing output SVG to out/map.svg...\n", ) with Path("out/map.svg").open() as output_file: @@ -76,7 +76,7 @@ def test_render_with_tooltips() -> None: def test_icons() -> None: """Test `icons` command.""" run( - COMMANDS["icons"], + COMMAND_LINES["icons"], b"INFO Icon grid is written to out/icon_grid.svg.\n" b"INFO Icons are written to out/icons_by_name and out/icons_by_id.\n", ) @@ -91,7 +91,7 @@ def test_icons() -> None: def test_mapcss() -> None: """Test `mapcss` command.""" run( - COMMANDS["mapcss"], + COMMAND_LINES["mapcss"], b"INFO MapCSS 0.2 scheme is written to out/map_machine_mapcss.\n", ) @@ -105,15 +105,17 @@ def test_mapcss() -> None: def test_element() -> None: """Test `element` command.""" - run(COMMANDS["element"], b"INFO Element is written to out/element.svg.\n") - + run( + COMMAND_LINES["element"], + b"INFO Element is written to out/element.svg.\n", + ) assert (Path("out") / "element.svg").is_file() def test_tile() -> None: """Test `tile` command.""" run( - COMMANDS["tile"] + ["--cache", "tests/data"], + COMMAND_LINES["tile"] + ["--cache", "tests/data"], b"INFO Tile is drawn to out/tiles/tile_18_160199_88904.svg.\n" b"INFO SVG file is rasterized to out/tiles/tile_18_160199_88904.png.\n", )