feat: Update default caption style to `on_screen_text_top`, add caption style aliases in video renderer, and remove `test_oneup_service.py`.

src/video_renderer.py

@@ -1064,9 +1064,13 @@ class VideoRenderer:
 
             styles = {
                 "caption_1": (caption_style_1, 3),
+                "caption_style_1": (caption_style_1, 3),
                 "caption_2": (caption_style_2, 3),
+                "caption_style_2": (caption_style_2, 3),
                 "caption_3": (caption_style_3, 3),
+                "caption_style_3": (caption_style_3, 3),
                 "caption_4": (caption_style_4, 3),
+                "caption_style_4": (caption_style_4, 3),
                 "caption_style_on_screen_text": (caption_style_on_screen_text, 3),
                 "caption_style_on_screen_text_top": (caption_style_on_screen_text_top, 3),
             }

tests/test_oneup_service.py

@@ -1,112 +0,0 @@
-
-import sys
-import os
-import unittest
-from unittest.mock import MagicMock, patch
-
-# Add parent directory to path
-sys.path.append(os.path.abspath(os.path.join(os.path.dirname(__file__), '..')))
-
-from social_media_publishers.oneup_service import OneUpService
-
-class TestOneUpService(unittest.TestCase):
-
-    def setUp(self):
-        # Mock the OneUpClient inside OneUpService
-        with patch('social_media_publishers.oneup_service.OneUpClient') as MockClient:
-            self.mock_client = MockClient.return_value
-            self.service = OneUpService()
-            # Restore the mock to the instance for assertion
-            self.service.client = self.mock_client
-
-    @patch('os.path.exists')
-    @patch('social_media_publishers.oneup_service.upload_file_to_gcs')
-    def test_publish_video_with_identifier(self, mock_upload, mock_exists):
-        mock_exists.return_value = True
-        # Setup Mocks
-        self.mock_client.find_account.return_value = {
-            'social_network_id': '12345',
-            'social_network_name': 'Test Account',
-            'category_id': '999'
-        }
-        self.mock_client.schedule_video_post.return_value = {
-            'message': 'Scheduled successfully',
-            'error': False
-        }
-        
-        # Scenario: Local file path provided (mocking upload)
-        mock_upload.return_value = {'signed_url': 'http://public.url/video.mp4'}
-        
-        metadata = {'title': 'Test Video', 'description': 'Description'}
-        
-        result = self.service.publish_video(
-            platform='instagram',
-            content_path='/tmp/video.mp4',
-            metadata=metadata,
-            account_identifier='test_user'
-        )
-        
-        # Assertions
-        self.assertTrue(result['success'])
-        self.mock_client.find_account.assert_called_with('instagram', 'test_user')
-        mock_upload.assert_called()
-        self.mock_client.schedule_video_post.assert_called_with(
-            category_id='999',
-            social_network_ids=['12345'],
-            video_url='http://public.url/video.mp4',
-            title='Test Video',
-            content='Description',
-            publish_now=True
-        )
-
-    def test_publish_video_account_not_found(self):
-        self.mock_client.find_account.return_value = None
-        
-        result = self.service.publish_video(
-            platform='instagram',
-            content_path='/tmp/video.mp4',
-            metadata={},
-            account_identifier='unknown_user'
-        )
-        
-        self.assertIn('error', result)
-        self.assertIn('not found', result['error'])
-
-    @patch('os.path.exists')
-    @patch('social_media_publishers.oneup_service.upload_file_to_gcs')
-    def test_publish_video_fallback_category(self, mock_upload, mock_exists):
-        mock_exists.return_value = True
-        # Scenario: Account found but has no category_id in object, need fallback
-        self.mock_client.find_account.return_value = {
-            'social_network_id': '12345',
-            'social_network_name': 'Test Account',
-            # No category_id here
-        }
-        
-        # Mock get_categories
-        self.mock_client.get_categories.return_value = [{'id': '888', 'category_name': 'Default'}]
-        
-        mock_upload.return_value = {'signed_url': 'http://public.url/video.mp4'}
-        self.mock_client.schedule_video_post.return_value = {'message': 'OK'}
-
-        # Test Direct URL (metadata has it)
-        result = self.service.publish_video(
-            platform='youtube',
-            content_path=None,
-            metadata={'video_url': 'http://direct.url/vid.mp4'},
-            account_identifier='test_user'
-        )
-        
-        self.mock_client.get_categories.assert_called()
-        self.mock_client.schedule_video_post.assert_called_with(
-            category_id='888',
-            social_network_ids=['12345'],
-            video_url='http://direct.url/vid.mp4',
-            title='',
-            content='',
-            publish_now=True
-        )
-
-
-if __name__ == '__main__':
-    unittest.main()

workflow_env/full_ai_video_generate_workflow.env

@@ -1,6 +1,6 @@
 JOB_INDEX=0
 TOTAL_JOBS=1
-CAPTION_STYLE=caption_style_1
+CAPTION_STYLE=caption_style_on_screen_text_top
 GCS_BUCKET_NAME=infloxa
 GCS_BUCKET_FOLDER_NAME=test_xai_product_video
 GENERATION_COUNT=1