diff --git a/Makefile b/Makefile index 1826382..1d404f9 100644 --- a/Makefile +++ b/Makefile @@ -6,6 +6,10 @@ test: virt: pytest -m smokevirt +# run the smoke1 test (after doing a factory reset and unplugging/replugging in device) +smoke1: + pytest -m smoke1 -s -vv + # local install install: pip install . diff --git a/meshtastic/tests/test_smoke1.py b/meshtastic/tests/test_smoke1.py index 7f58dc1..8579051 100644 --- a/meshtastic/tests/test_smoke1.py +++ b/meshtastic/tests/test_smoke1.py @@ -170,20 +170,6 @@ def test_smoke1_port(): assert return_value == 0 -@pytest.mark.smoke1 -def test_smoke1_set_is_router_true(): - """Test --set is_router true""" - return_value, out = subprocess.getstatusoutput('meshtastic --set is_router true') - assert re.match(r'Connected to radio', out) - assert re.search(r'^Set is_router to true', out, re.MULTILINE) - assert return_value == 0 - # pause for the radio - time.sleep(PAUSE_AFTER_COMMAND) - return_value, out = subprocess.getstatusoutput('meshtastic --get is_router') - assert re.search(r'^is_router: True', out, re.MULTILINE) - assert return_value == 0 - - @pytest.mark.smoke1 def test_smoke1_set_location_info(): """Test --setlat, --setlon and --setalt """ @@ -202,20 +188,6 @@ def test_smoke1_set_location_info(): assert return_value == 0 -@pytest.mark.smoke1 -def test_smoke1_set_is_router_false(): - """Test --set is_router false""" - return_value, out = subprocess.getstatusoutput('meshtastic --set is_router false') - assert re.match(r'Connected to radio', out) - assert re.search(r'^Set is_router to false', out, re.MULTILINE) - assert return_value == 0 - # pause for the radio - time.sleep(PAUSE_AFTER_COMMAND) - return_value, out = subprocess.getstatusoutput('meshtastic --get is_router') - assert re.search(r'^is_router: False', out, re.MULTILINE) - assert return_value == 0 - - @pytest.mark.smoke1 def test_smoke1_set_owner(): """Test --set-owner name""" @@ -264,17 +236,17 @@ def test_smoke1_set_team(): @pytest.mark.smoke1 def test_smoke1_ch_values(): - """Test --ch-longslow, --ch-longfast, --ch-mediumslow, --ch-mediumsfast, + """Test --ch-vlongslow --ch-longslow, --ch-longfast, --ch-mediumslow, --ch-mediumsfast, --ch-shortslow, and --ch-shortfast arguments """ exp = { - '--ch-longslow': 'Bw125Cr48Sf4096', - '--ch-longfast': 'Bw31_25Cr48Sf512', - '--ch-mediumslow': 'Bw250Cr46Sf2048', - '--ch-mediumfast': 'Bw250Cr47Sf1024', - # for some reason, this value does not show any modemConfig - '--ch-shortslow': '{ "psk', - '--ch-shortfast': 'Bw500Cr45Sf128' + '--ch-vlongslow': '{ "psk": "AQ==" }', + '--ch-longslow': 'LongSlow', + '--ch-longfast': 'LongFast', + '--ch-midslow': 'MidSlow', + '--ch-midfast': 'MidFast', + '--ch-shortslow': 'ShortSlow', + '--ch-shortfast': 'ShortFast' } for key, val in exp.items(): @@ -587,18 +559,18 @@ def test_smoke1_ch_set_modem_config(): # pause for the radio time.sleep(PAUSE_AFTER_COMMAND) return_value, out = subprocess.getstatusoutput('meshtastic --info') - assert not re.search(r'Bw31_25Cr48Sf512', out, re.MULTILINE) + assert not re.search(r'ShortFast', out, re.MULTILINE) assert return_value == 0 # pause for the radio time.sleep(PAUSE_AFTER_COMMAND) - return_value, out = subprocess.getstatusoutput('meshtastic --ch-set modem_config Bw31_25Cr48Sf512 --ch-index 0') + return_value, out = subprocess.getstatusoutput('meshtastic --ch-set modem_config ShortFast --ch-index 0') assert re.match(r'Connected to radio', out) - assert re.search(r'^Set modem_config to Bw31_25Cr48Sf512', out, re.MULTILINE) + assert re.search(r'^Set modem_config to ShortFast', out, re.MULTILINE) assert return_value == 0 # pause for the radio time.sleep(PAUSE_AFTER_COMMAND) return_value, out = subprocess.getstatusoutput('meshtastic --info') - assert re.search(r'Bw31_25Cr48Sf512', out, re.MULTILINE) + assert re.search(r'ShortFast', out, re.MULTILINE) assert return_value == 0