Commit 5856dcdf authored by Antonio Caceres's avatar Antonio Caceres
Browse files

Rename translate_complete_morse_symbol to append_morse_symbol and fix LHS/RHS output accordingly.

parent d6c5ad09
No related merge requests found
Showing with 10 additions and 9 deletions
+10 -9
from .symbols import MORSE_SYMBOL_TO_LETTER, InvalidSymbolError
def translate_complete_morse_symbol(message, symbol):
def append_morse_symbol(message, symbol):
"""
Decodes a Morse code symbol and appends the corresponding character to the
message.
......
import pytest
from src.morse_code import translate_complete_morse_symbol
from src.morse_code import append_morse_symbol
from src.symbols import InvalidSymbolError
from tests.helpers.translate_symbol_data import SINGLE_SYMBOL_TESTS
from tests.helpers.naming import apply_names
......@@ -9,24 +9,24 @@ from tests.helpers.naming import apply_names
@pytest.mark.parametrize(
"seq, msg, expected",
apply_names(
"translate_complete_morse_symbol", [True, True, False], SINGLE_SYMBOL_TESTS
"append_morse_symbol", [True, True, False], SINGLE_SYMBOL_TESTS
),
)
def test_add_random(seq, msg, expected):
if expected is None:
with pytest.raises(InvalidSymbolError):
result = translate_complete_morse_symbol(msg, seq)
append_morse_symbol(msg, seq)
else:
result = translate_complete_morse_symbol(msg, seq)
result = append_morse_symbol(msg, seq)
assert (
result == expected
expected == result
), "a valid morse code symbol wasn't translated correctly"
@pytest.mark.parametrize(
"msg, expected",
apply_names(
"translate_complete_morse_symbol",
"append_morse_symbol",
[True, False],
[
("JUMPOUTTHEHOUSE", "JUMPOUTTHEHOUS"),
......@@ -40,5 +40,6 @@ def test_add_random(seq, msg, expected):
),
)
def test_delete(msg, expected):
result = translate_complete_morse_symbol(msg, "......")
assert result == expected
result = append_morse_symbol(msg, "......")
assert expected == result, ("the delete symbol was not translated correctly; "
"check src/symbols.py for the morse code mapping")
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment