Merge pull request #351 from keboola/odin-SOX-431

Allow integers for orchestrationTaskId

Author: odinuv

src/JobFactory/FullJobDefinition.php

@@ -193,10 +193,6 @@ protected function getRootDefinition(TreeBuilder $treeBuilder): ArrayNodeDefinit
                     ->beforeNormalization()->always($this->getStringNormalizer())->end()
                 ->end()
                 ->scalarNode('orchestrationTaskId')
-                    ->validate()
-                        ->ifTrue(fn($v) => $v !== null && !is_string($v))
-                        ->thenInvalid('value must be a string')
-                    ->end()
                     ->validate()
                         ->ifTrue(fn($v) => $v === '')
                         ->thenInvalid('value cannot be empty string')

src/JobFactory/NewJobDefinition.php

@@ -86,10 +86,6 @@ protected function getRootDefinition(TreeBuilder $treeBuilder): ArrayNodeDefinit
                 ->end()
                 ->scalarNode('orchestrationJobId')->end()
                 ->scalarNode('orchestrationTaskId')
-                    ->validate()
-                        ->ifTrue(fn($v) => $v !== null && !is_string($v))
-                        ->thenInvalid('value must be a string')
-                    ->end()
                     ->validate()
                         ->ifTrue(fn($v) => $v === '')
                         ->thenInvalid('value cannot be empty string')

tests/JobFactory/FullJobDefinitionTest.php

@@ -409,7 +409,7 @@ public function invalidJobProvider(): array
                 ],
                 '#The value "invalid" is not allowed for path "job.branchType". Permissible values: "default", "dev"#',
             ],
-            'orchestrationTaskId not string' => [
+            'orchestrationTaskId not scalar' => [
                 [
                     '#tokenString' => getenv('TEST_STORAGE_API_TOKEN'),
                     'id' => '12345',
@@ -421,9 +421,9 @@ public function invalidJobProvider(): array
                     'configId' => '123',
                     'componentId' => 'keboola.test',
                     'mode' => 'run',
-                    'orchestrationTaskId' => 134,
+                    'orchestrationTaskId' => [123],
                 ],
-                '/Invalid configuration for path "job.orchestrationTaskId": value must be a string/',
+                '#Invalid type for path "job.orchestrationTaskId". Expected "?scalar"?, but got "?array"?.#',
             ],
             'orchestrationTaskId empty string' => [
                 [

tests/JobFactory/NewJobDefinitionTest.php

@@ -154,15 +154,15 @@ public function invalidJobProvider(): array
                 ],
                 '#Invalid type for path "job.parentRunId". Expected "?scalar"?, but got "?array"?.#',
             ],
-            'orchestrationTaskId not string' => [
+            'orchestrationTaskId not scalar' => [
                 [
                     '#tokenString' => getenv('TEST_STORAGE_API_TOKEN'),
                     'configId' => '123',
                     'componentId' => 'keboola.test',
                     'result' => [],
-                    'orchestrationTaskId' => 134,
+                    'orchestrationTaskId' => [134],
                 ],
-                '/Invalid configuration for path "job.orchestrationTaskId": value must be a string/',
+                '#Invalid type for path "job.orchestrationTaskId". Expected "?scalar"?, but got "?array"?.#',
             ],
             'orchestrationTaskId empty string' => [
                 [