From 0c9c4c0c31ca9d2abfb4d2af4047432df9ff4462 Mon Sep 17 00:00:00 2001
From: Philipp Boenninghausen <philipp.schlipper@fkie.fraunhofer.de>
Date: Wed, 25 Sep 2024 09:02:06 +0200
Subject: [PATCH 1/2] Add missing format string

---
 src/attacks/attack_take_screenshot.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/attacks/attack_take_screenshot.py b/src/attacks/attack_take_screenshot.py
index a605562..d2044b8 100644
--- a/src/attacks/attack_take_screenshot.py
+++ b/src/attacks/attack_take_screenshot.py
@@ -68,5 +68,5 @@ def _respond(self, line):
 
     def _collect_files(self):
         file = self.screenshot_file
-        self.ssh_client.write_lines(self.handler.stdin, ["screenshot -p {file}",
+        self.ssh_client.write_lines(self.handler.stdin, [f"screenshot -p {file}",
                                                          "background"])

From 62a049368c628000d5718506f38c5206bb34d4d9 Mon Sep 17 00:00:00 2001
From: Philipp Boenninghausen <philipp.schlipper@fkie.fraunhofer.de>
Date: Wed, 25 Sep 2024 09:02:21 +0200
Subject: [PATCH 2/2] Add missing unit tests

---
 .../tests/test_attack_take_screenshot.py      | 68 +++++++++++++++++++
 1 file changed, 68 insertions(+)
 create mode 100644 src/attacks/tests/test_attack_take_screenshot.py

diff --git a/src/attacks/tests/test_attack_take_screenshot.py b/src/attacks/tests/test_attack_take_screenshot.py
new file mode 100644
index 0000000..35b24d6
--- /dev/null
+++ b/src/attacks/tests/test_attack_take_screenshot.py
@@ -0,0 +1,68 @@
+# Copyright 2016-2022 Fraunhofer FKIE
+#
+# This file is part of SOCBED.
+#
+# SOCBED is free software: you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation, either version 3 of the License, or
+# (at your option) any later version.
+#
+# SOCBED is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with SOCBED. If not, see <http://www.gnu.org/licenses/>.
+
+
+from unittest.mock import Mock, patch
+
+import pytest
+from attacks import AttackException
+from attacks.attack_take_screenshot import TakeScreenshotAttack
+from attacks.ssh import SSHTarget
+
+
+@pytest.fixture()
+def attack():
+    TakeScreenshotAttack.ssh_client_class = Mock
+    attack = TakeScreenshotAttack()
+    attack.ssh_client.target = SSHTarget()
+    return attack
+
+class TestTakeScreenshotAttack:
+    def test_info(self):
+        assert TakeScreenshotAttack.info.name == "c2_take_screenshot"
+
+    def test_prepare_attack(self, attack: TakeScreenshotAttack):
+        attack.exec_command_on_target = Mock()
+        attack._prepare_attack()
+        attack.exec_command_on_target.assert_called_with("cd /root; mkdir -p loot; cd loot; rm -f screenshot.jpeg")
+    
+    @patch('time.sleep', return_value=None)
+    def test_respond(self, mock_sleep, attack: TakeScreenshotAttack):
+        attack._collect_files = Mock()
+        attack.handler = Mock()
+        
+        attack._respond("test")
+        assert not attack._collect_files.called
+        assert not attack.handler.shutdown.called
+        
+        attack._respond("Meterpreter session 1 opened bla bla")
+        assert attack._collect_files.called
+
+        attack._respond("Backgrounding session bla bla")
+        assert attack.handler.shutdown.called
+        attack.handler.shutdown.reset_mock()
+
+        attack._respond("Exploit completed, but no session was created bla bla")
+        assert attack.handler.shutdown.called
+
+    def test_collect_files(self, attack: TakeScreenshotAttack):
+        attack.handler = Mock()
+        attack.screenshot_file = "fancy_screenshot.jpeg"
+        attack.handler.stdin = "some stdin"
+
+        attack._collect_files()
+        attack.ssh_client.write_lines.assert_called_with("some stdin", ["screenshot -p fancy_screenshot.jpeg", "background"])
\ No newline at end of file