Fixed some CWL translator issues

[LanderOtto]

This commit fixes several CWL translator issues:

• Fix the retrieval of the step name from the cwl-utils objects. Before this commit, the out attribute of a WorkflowStep object was always expected to be a list of strings. Instead, it should be a list of Any, following the typing hint of the cwl_utils.parser.WorkflowStep constructor. In particular, it can contain WorkflowStepOutput objects.
• Fix the CommandOutputProcessor creation when a schema is defined. Before this commit, the CommandInputRecordSchema was not included in the RecordSchema case. Similar for EnumSchema and ArraySchema cases, respectively.
• Fix optional inputs inside a scatter step
• Fix the _get_path method in the CWL translator module. When the id field is explicitly defined in a CWL file, the element_id of the corresponding Python object is constructed as /path/to/file#id
• Fix the configuration file's path stored in the DataManager. Before this commit, it was saved the path defined in the StreamFlow file, which could be a relative path
• Fix optional inputs of type list or record. Before this commit, some workflows were failing because their input data were erroneously treated as mandatory

[codecov]

❌ 1 Tests Failed:

Tests completed	Failed	Passed	Skipped
1989	1	1988	9

View the top 1 failed tests by shortest run time

tests/test_remotepath.py::test_download[docker]

Stack Traces | 4.05s run time

context = <streamflow.core.context.StreamFlowContext object at 0x1155a7020>
connector = <streamflow.deployment.connector.container.DockerConnector object at 0x114d95f90>
location = <streamflow.core.deployment.ExecutionLocation object at 0x114b0c200>

    @pytest.mark.asyncio
    async def test_download(context, connector, location):
        """Test remote file download."""
        urls = [
            "https://raw.githubusercontent..../streamflow/master/LICENSE",
            "https://github..../refs/tags/0.1.6.zip",
        ]
        parent_dir = StreamFlowPath(
            tempfile.gettempdir() if location.local else "/tmp",
            context=context,
            location=location,
        )
        paths = [
            parent_dir / "LICENSE",
            parent_dir / "streamflow-0.1.6.zip",
        ]
        path = None
        for i, url in enumerate(urls):
            try:
                path = await remotepath.download(context, location, url, str(parent_dir))
                assert path == paths[i]
>               assert await path.exists()
E               assert False

tests/test_remotepath.py:78: AssertionError

To view more test analytics, go to the Test Analytics Dashboard
📢 Thoughts on this report? Let us know!

[GlassOfWhiskey]

LGTM