test: improve secure download mode test for ESP32-S2

Author: Dzarda7

test/test_esptool_sdm.py

@@ -5,8 +5,9 @@
 #
 # How to use:
 #
-# Run with a physical connection to a chip:
-#  - `pytest test_esptool_sdm.py --chip esp32 --port /dev/ttyUSB0 --baud 115200`
+# Run with a physical connection to a chip (ESP8266 and ESP32 do not support
+# secure download mode):
+#  - `pytest test_esptool_sdm.py --chip esp32s2 --port /dev/ttyUSB0 --baud 115200`
 #
 # where  - --port       - a serial port for esptool operation
 #        - --chip       - ESP chip name
@@ -17,27 +18,20 @@
 
 
 @pytest.mark.skipif(
-    arg_chip == "esp8266", reason="ESP8266 does not support Secure Download Mode"
+    arg_chip in ("esp8266", "esp32"),
+    reason="ESP8266 and ESP32 do not support secure download mode",
 )
 class TestSecureDownloadMode(EsptoolTestCase):
     expected_chip_name = esptool.util.expand_chip_name(arg_chip)
 
-    @pytest.mark.skipif(
-        arg_chip in ("esp8266", "esp32"),
-        reason="No get-security-info on ESP8266 and ESP32",
-    )
     def test_auto_detect(self):
         output = self.run_esptool("get-security-info", chip="auto")
 
-        if arg_chip == "esp32s2":  # no autodetection from security info, only magic no.
-            assert "Secure Download Mode is enabled" in output
-            assert "autodetection will not work" in output
-        else:
-            assert f"Detecting chip type... {self.expected_chip_name}" in output
-            assert (
-                f"{'Chip type:':<20}{self.expected_chip_name} "
-                "in Secure Download Mode" in output
-            )
+        assert f"Detecting chip type... {self.expected_chip_name}" in output
+        assert (
+            f"{'Chip type:':<20}{self.expected_chip_name} "
+            "in Secure Download Mode" in output
+        )
 
     # Commands not supported in SDM
     def test_sdm_incompatible_commands(self):
@@ -65,6 +59,5 @@ def test_sdm_compatible_commands(self):
         assert "Stub flasher is not supported in Secure Download Mode" in output
         assert "Flash memory region erased successfully" in output
 
-        if arg_chip != "esp32":  # esp32 does not support get-security-info
-            output = self.run_esptool("get-security-info")
-            assert "Security Information:" in output
+        output = self.run_esptool("get-security-info")
+        assert "Security Information:" in output