From 9c4a239d03a16f6465273f83f86edc81337ffd82 Mon Sep 17 00:00:00 2001
From: Jan Vonka <jan.vonka@alfresco.com>
Date: Mon, 29 Mar 2021 15:52:17 +0100
Subject: [PATCH] ATS-812: PoC FFmpeg skeleton - unit test fixes

- fix the -ve unit test files (back to same as other T-Engines)
- for now, remove the empty FFmpeg t-options

- TODO add some example config options (+ unit tests)
---
 .../src/test/resources/engine_config_complete.json          | 2 +-
 .../src/test/resources/engine_config_incomplete.json        | 2 +-
 .../test/resources/engine_config_no_transform_options.json  | 2 +-
 .../src/test/resources/engine_config_with_duplicates.json   | 6 +++---
 .../src/test/resources/ffmpeg_engine_config.json            | 5 -----
 5 files changed, 6 insertions(+), 11 deletions(-)

diff --git a/alfresco-transform-ffmpeg/alfresco-transform-ffmpeg-boot/src/test/resources/engine_config_complete.json b/alfresco-transform-ffmpeg/alfresco-transform-ffmpeg-boot/src/test/resources/engine_config_complete.json
index 9990c57c..0ab0ea15 100644
--- a/alfresco-transform-ffmpeg/alfresco-transform-ffmpeg-boot/src/test/resources/engine_config_complete.json
+++ b/alfresco-transform-ffmpeg/alfresco-transform-ffmpeg-boot/src/test/resources/engine_config_complete.json
@@ -12,7 +12,7 @@
     {
       "transformerName": "engineX",
       "supportedSourceAndTargetList": [
-        {"sourceMediaType": "video/mp4", "targetMediaType": "audio/mpeg" }
+        {"sourceMediaType": "application/pdf", "targetMediaType": "image/png" }
       ],
       "transformOptions": [
         "engineXOptions"
diff --git a/alfresco-transform-ffmpeg/alfresco-transform-ffmpeg-boot/src/test/resources/engine_config_incomplete.json b/alfresco-transform-ffmpeg/alfresco-transform-ffmpeg-boot/src/test/resources/engine_config_incomplete.json
index 1f901c24..04935a7b 100644
--- a/alfresco-transform-ffmpeg/alfresco-transform-ffmpeg-boot/src/test/resources/engine_config_incomplete.json
+++ b/alfresco-transform-ffmpeg/alfresco-transform-ffmpeg-boot/src/test/resources/engine_config_incomplete.json
@@ -3,7 +3,7 @@
   "transformers": [
     {
       "supportedSourceAndTargetList": [
-        {"sourceMediaType": "video/mp4", "targetMediaType": "audio/mpeg" }
+        {"sourceMediaType": "application/pdf", "targetMediaType": "image/png" }
       ]
     }
   ]
diff --git a/alfresco-transform-ffmpeg/alfresco-transform-ffmpeg-boot/src/test/resources/engine_config_no_transform_options.json b/alfresco-transform-ffmpeg/alfresco-transform-ffmpeg-boot/src/test/resources/engine_config_no_transform_options.json
index f313e630..1ceebe95 100644
--- a/alfresco-transform-ffmpeg/alfresco-transform-ffmpeg-boot/src/test/resources/engine_config_no_transform_options.json
+++ b/alfresco-transform-ffmpeg/alfresco-transform-ffmpeg-boot/src/test/resources/engine_config_no_transform_options.json
@@ -3,7 +3,7 @@
     {
       "transformerName": "engineX",
       "supportedSourceAndTargetList": [
-        {"sourceMediaType": "video/mp4", "targetMediaType": "audio/mpeg" }
+        {"sourceMediaType": "application/pdf", "targetMediaType": "image/png" }
       ]
     }
   ]
diff --git a/alfresco-transform-ffmpeg/alfresco-transform-ffmpeg-boot/src/test/resources/engine_config_with_duplicates.json b/alfresco-transform-ffmpeg/alfresco-transform-ffmpeg-boot/src/test/resources/engine_config_with_duplicates.json
index b5da9909..581ad9cc 100644
--- a/alfresco-transform-ffmpeg/alfresco-transform-ffmpeg-boot/src/test/resources/engine_config_with_duplicates.json
+++ b/alfresco-transform-ffmpeg/alfresco-transform-ffmpeg-boot/src/test/resources/engine_config_with_duplicates.json
@@ -13,9 +13,9 @@
     {
       "transformerName": "engineX",
       "supportedSourceAndTargetList": [
-        {"sourceMediaType": "video/mp4", "targetMediaType": "audio/mpeg" },
-        {"sourceMediaType": "video/mp4", "targetMediaType": "audio/mpeg" },
-        {"sourceMediaType": "video/mp4", "targetMediaType": "audio/mpeg" }
+        {"sourceMediaType": "application/pdf", "targetMediaType": "image/png" },
+        {"sourceMediaType": "application/pdf", "targetMediaType": "image/png" },
+        {"sourceMediaType": "application/pdf", "targetMediaType": "image/png" }
       ],
       "transformOptions": [
         "engineXOptions",
diff --git a/alfresco-transform-ffmpeg/alfresco-transform-ffmpeg-boot/src/test/resources/ffmpeg_engine_config.json b/alfresco-transform-ffmpeg/alfresco-transform-ffmpeg-boot/src/test/resources/ffmpeg_engine_config.json
index 2c0ff4b4..446f2524 100644
--- a/alfresco-transform-ffmpeg/alfresco-transform-ffmpeg-boot/src/test/resources/ffmpeg_engine_config.json
+++ b/alfresco-transform-ffmpeg/alfresco-transform-ffmpeg-boot/src/test/resources/ffmpeg_engine_config.json
@@ -1,7 +1,5 @@
 {
   "transformOptions": {
-    "ffmpegOptions": [
-    ]
   },
   "transformers": [
     {
@@ -9,9 +7,6 @@
       "supportedSourceAndTargetList": [
         {"sourceMediaType": "video/mp4",                                  "targetMediaType": "video/avi" },
         {"sourceMediaType": "video/mp4",                                  "targetMediaType": "audio/mpeg" }
-      ],
-      "transformOptions": [
-        "ffmpegOptions"
       ]
     }
   ]