Fix: Prevent test failures due to missing ACCESS_TOKEN

The test suite was failing because the application would exit prematurely when the ACCESS_TOKEN environment variable was not set during test execution.

This commit addresses the issue by:
- Patching `sys.exit` in the `TestThreadsCLI` setup to prevent the application from exiting.
- Adding basic mocking for API calls in `test_get_profile` and `test_get_recent_posts` to allow these tests to run without a valid ACCESS_TOKEN.
- Ensuring the test drafts file is removed before each test run to maintain a clean state.

This resolves the `AssertionError: 1 != 0` seen in tests like `test_get_profile`, `test_get_recent_posts`, and `test_create_text_post` when running in environments without the ACCESS_TOKEN configured.

* modified:   tests/test_main.py

Signed-off-by: Muhammad Amin Boubaker <muhammadaminboubaker@gmail.com>

Author: CodeIter

tests/test_main.py

@@ -1,3 +1,4 @@
+import sys
 import os
 import uuid
 import json
@@ -28,25 +29,44 @@
 
 # Ensure TEST_DRAFTS_FILE path is resolved and the file exists, following XDG Base Directory Specification if necessary.
 TEST_DRAFTS_FILE = ensure_drafts_file(TEST_DRAFTS_FILE)
-os.remove(TEST_DRAFTS_FILE)
+
+# Remove the test drafts file if it exists from a previous run
+# or if created by ensure_drafts_file function.
+if os.path.exists(TEST_DRAFTS_FILE):
+    os.remove(TEST_DRAFTS_FILE)
 
 class TestThreadsCLI(unittest.TestCase):
     def setUp(self):
         self.runner = CliRunner()
+        # Patch sys.exit to prevent the application from exiting during tests
+        self.patcher = patch('sys.exit')
+        self.mock_exit = self.patcher.start()
 
     def tearDown(self):
+        # Stop the patcher
+        self.patcher.stop()
+        # Remove the test drafts file after each test
         if os.path.exists(TEST_DRAFTS_FILE):
             os.remove(TEST_DRAFTS_FILE)
 
     def test_get_profile(self):
-        result = self.runner.invoke(app, ["get-profile"])
-        self.assertEqual(result.exit_code, 0)
-        self.assertIn("Profile", result.stdout)
+        # Mock the get_user_id and get_user_profile functions
+        with patch("src.app.get_user_id", return_value="test_user_id"), \
+             patch("src.app.get_user_profile", return_value={"username": "testuser"}), \
+             patch("src.app.get_user_posts", return_value=[{"id": "123", "media_type": "TEXT", "text": "Hello", "permalink": "link", "timestamp": "2023-01-01T10:00:00+0000"}]):
+            result = self.runner.invoke(app, ["get-profile"])
+            self.assertEqual(result.exit_code, 0)
+            self.assertIn("Profile", result.stdout)
 
     def test_get_recent_posts(self):
-        result = self.runner.invoke(app, ["get-recent-posts", "--limit", "5"])
-        self.assertEqual(result.exit_code, 0)
-        self.assertIn("Recent Posts", result.stdout)
+        # Mock the necessary functions
+        with patch("src.app.get_user_id", return_value="test_user_id"), \
+             patch("src.app.get_user_posts", return_value=[{"id": "123", "media_type": "TEXT", "text": "Hello", "permalink": "link", "timestamp": "2023-01-01T10:00:00+0000"}]), \
+             patch("src.app.get_post_replies_count", return_value=0):
+            result = self.runner.invoke(app, ["get-recent-posts", "--limit", "5"])
+            self.assertEqual(result.exit_code, 0)
+            self.assertIn("Recent Posts", result.stdout)
+
 
     def test_create_text_post(self):
         with patch("src.app.create_post") as mock_create_post:
@@ -94,3 +114,4 @@ def test_send_draft(self):
 
 if __name__ == "__main__":
     unittest.main()
+