opencos-eda 0.3.8__py3-none-any.whl → 0.3.10__py3-none-any.whl

opencos/tests/test_eda_elab.py

@@ -1,110 +0,0 @@
-'''pytests for: eda elab <command> [args] <target>'''
-
-import os
-import pytest
-
-from opencos import eda_tool_helper
-from opencos.tests import helpers
-from opencos.tests.helpers import eda_wrap, eda_elab_wrap, eda_lint_wrap
-
-
-thispath = os.path.dirname(__file__)
-
-def chdir_remove_work_dir(relpath):
-    '''Changes dir to relpath, removes the work directories (eda.work, eda.export*)'''
-    return helpers.chdir_remove_work_dir(thispath, relpath)
-
-# Figure out what tools the system has available, without calling eda.main(..)
-config, tools_loaded = eda_tool_helper.get_config_and_tools_loaded()
-
-# list of tools we'd like to try:
-list_of_elab_tools = [
-    'slang',
-    'verilator',
-    'vivado',
-    'modelsim_ase'
-    'questa_fse',
-    'invio',
-    'surelog',
-    'invio_yosys',
-]
-
-list_of_lint_tools = [
-    'slang',
-    'verilator',
-    'vivado',
-    'modelsim_ase'
-    'questa_fse',
-    'invio',
-    'surelog',
-]
-
-list_of_elab_tools_cant_sim = [
-    'slang',
-    'invio',
-    'surelog',
-    'invio_yosys',
-]
-
-list_of_commands = [
-    'elab',
-    'lint',
-]
-
-def skip_it(tool) -> bool:
-    '''Returns True if this test should be skipped
-
-    For example, run in a github Action w/ container that doesn't have a tool
-    in tool_loaded.
-    '''
-    return bool( tool not in tools_loaded )
-
-@pytest.mark.parametrize("command", list_of_commands)
-@pytest.mark.parametrize("tool", list_of_elab_tools)
-class Tests:
-    '''Test tools from list_of_elab_tools for 'eda elab' and 'eda multi elab'.'''
-
-    def test_args_elab(self, command, tool):
-        '''tests: eda elab --tool oclib_priarb'''
-        if command == 'lint' and tool not in list_of_lint_tools:
-            pytest.skip(f"lint skipped for {tool=} b/c it can't run lint")
-        if skip_it(tool):
-            pytest.skip(f"{tool=} skipped, {tools_loaded=}")
-            return # skip/pass
-        chdir_remove_work_dir('../../lib')
-        if command == 'elab':
-            rc = eda_elab_wrap('--tool', tool, 'oclib_priarb')
-        else:
-            rc = eda_lint_wrap('--tool', tool, 'oclib_priarb')
-        print(f'{rc=}')
-        assert rc == 0
-
-    def test_args_multi_elab(self, command, tool):
-        '''tests: eda multi elab --tool oclib_*arb'''
-        if skip_it(tool):
-            pytest.skip(f"{tool=} skipped, {tools_loaded=}")
-            return # skip/pass
-        chdir_remove_work_dir('../../lib')
-        rc = eda_wrap('multi', command, '--tool', tool, 'oclib_*arb')
-        print(f'{rc=}')
-        assert rc == 0
-
-
-@pytest.mark.parametrize("tool", list_of_elab_tools_cant_sim)
-def test_elab_tool_cant_run_sim(tool):
-    '''Checks eda.check_command_handler_cls(...) so we don't fallback to a different tool'''
-    if skip_it(tool):
-        pytest.skip(f"{tool=}skipped, {tools_loaded=}")
-        return # skip/pass
-    chdir_remove_work_dir('../../lib')
-
-    # Calling this will have rc non-zero, but will also throw CommandSim NotImplementedError.
-    rc = 0
-    try:
-        rc = eda_wrap('sim', '--tool', tool, 'oclib_fifo')
-        print(f'{rc=}')
-        assert rc > 1
-    except NotImplementedError:
-        rc = 3
-        print(f'{rc=} (forced to 3 for NotImplementedError)')
-    assert rc > 1

opencos/tests/test_eda_synth.py

@@ -1,150 +0,0 @@
-'''pytests for: eda [multi|tools-multi] synth [args] <target(s)>'''
-
-import os
-import shutil
-
-import pytest
-
-from opencos import eda, eda_tool_helper
-from opencos.tests import helpers
-from opencos.tests.helpers import Helpers
-from opencos.utils.markup_helpers import yaml_safe_load
-
-
-THISPATH = os.path.dirname(__file__)
-
-def chdir_remove_work_dir(relpath):
-    '''Changes dir to relpath, removes the work directories (eda.work, eda.export*)'''
-    return helpers.chdir_remove_work_dir(THISPATH, relpath)
-
-# Figure out what tools the system has available, without calling eda.main(..)
-config, tools_loaded = eda_tool_helper.get_config_and_tools_loaded()
-
-# list of tools we'd like to try:
-list_of_synth_tools = [
-    'invio_yosys',
-    'tabbycad_yosys',
-    'slang_yosys'
-]
-
-list_of_elab_tools = [
-    'invio_yosys',
-    'slang_yosys'
-]
-
-def skip_it(tool, command) -> bool:
-    '''skip_it: returns True if we should skip due to lack of tool existence'''
-    return bool( tool not in tools_loaded or
-                 (command == 'elab' and tool not in list_of_elab_tools) or
-                 (command == 'synth' and tool not in list_of_synth_tools) )
-
-
-@pytest.mark.parametrize("tool", list_of_synth_tools)
-@pytest.mark.parametrize("command", ['elab', 'synth'])
-class Tests:
-    '''skippable (via pytest parameters) class holder for pytest methods'''
-
-    def test_args_multi_synth_bad_target_should_fail(self, tool, command):
-        '''Tests: eda mulit <elab|synth>, and this test should fail.'''
-        if skip_it(tool, command):
-            pytest.skip(f"{tool=} {command=} skipped, {tools_loaded=}")
-            return # skip/pass
-        chdir_remove_work_dir('../../lib')
-        cmdlist = (f'multi {command} --debug --fail-if-no-targets --tool {tool}'
-                   ' target_doesnt_exist*').split()
-        rc = eda.main(*cmdlist)
-        print(f'{rc=}')
-        assert rc > 1
-
-    def test_args_multi_synth_oclib_fifos(self, tool, command):
-        '''This should be 4 jobs and takes ~15 seconds for synth.'''
-        if skip_it(tool, command):
-            pytest.skip(f"{tool=} {command=} skipped, {tools_loaded=}")
-            return # skip/pass
-        chdir_remove_work_dir('../../lib')
-        cmdlist = (f'multi {command} --debug --fail-if-no-targets --tool {tool}'
-                   ' --yosys-synth=synth_xilinx oclib_fifo*').split()
-        rc = eda.main(*cmdlist)
-        print(f'{rc=}')
-        assert rc == 0
-
-    def test_args_multi_synth_oclib_rams(self, tool, command):
-        '''This should be 4 jobs and takes ~15 seconds for synth.'''
-        if skip_it(tool, command):
-            pytest.skip(f"{tool=} {command=} skipped, {tools_loaded=}")
-            return # skip/pass
-        chdir_remove_work_dir('../../lib')
-        cmdlist = (f'multi {command} --debug --fail-if-no-targets --tool {tool}'
-                   ' --yosys-synth=synth_xilinx rams/oclib_ram*').split()
-        rc = eda.main(*cmdlist)
-        print(f'{rc=}')
-        assert rc == 0
-
-
-def vivado_has_xpms() -> bool:
-    '''Returns True if Vivado is installed and has visibility to XPMs'''
-    if 'vivado' not in tools_loaded:
-        return False
-    vivado_exe = shutil.which('vivado')
-    vivado_bin_path, _ = os.path.split(vivado_exe)
-    vivado_base_path, _ = os.path.split(vivado_bin_path) # strip bin/vivado
-
-    return os.path.exists(os.path.join(vivado_base_path, 'data', 'ip', 'xpm'))
-
-
-@pytest.mark.skipif(
-    'slang_yosys' not in tools_loaded, reason="requires slang_yosys for synth"
-)
-class TestsSlangYosys(Helpers):
-    '''Tests that require tool=slang_yosys to be available'''
-
-    def test_sdc_file(self):
-        '''Test for 'eda synth' on oclib_fifo_with_sdc
-
-        This does not use the actual .sdc file, but it also shouldn't fail with
-        that file in the 'deps' list (CommandDesign should track it correctly)
-        '''
-        chdir_remove_work_dir('deps_files/test_sdc_files')
-        cmd_str = 'synth --tool=slang_yosys oclib_fifo_with_sdc'
-        rc = self.log_it(cmd_str, use_eda_wrap=False)
-        assert rc == 0
-
-        # Since vanilla yosys won't use the SDC file, let's at least confirm
-        # that EDA used it and tracked it:
-        eda_config_yml_path = os.path.join(
-            os.getcwd(), 'eda.work', 'oclib_fifo_with_sdc.synth', 'eda_output_config.yml'
-        )
-        data = yaml_safe_load(eda_config_yml_path)
-        assert 'files_sdc' in data
-        assert data['files_sdc']
-        assert data['files_sdc'][0].endswith('oclib_fifo_yosys.sdc')
-
-
-@pytest.mark.skipif('vivado' not in tools_loaded, reason="requires vivado")
-@pytest.mark.skipif(not vivado_has_xpms(), reason="requires install to have XPMs")
-class TestsVivado(Helpers):
-    '''Tests that require tool=vivado with XPMs available for synthesis'''
-
-    def test_sdc_file(self):
-        '''Test for 'eda synth' on oclib_fifo_with_sdc
-
-        And check that the .sdc file was used and not the default generated .xdc
-        file
-        '''
-        chdir_remove_work_dir('deps_files/test_sdc_files')
-        cmd_str = 'synth --tool=vivado oclib_fifo_with_sdc'
-        rc = self.log_it(cmd_str, use_eda_wrap=False)
-        assert rc == 0
-
-        # apparently this doesn't get saved in the eda.log. That's not great,
-        # but we can check the <target>.synth.log file.
-        synth_log = os.path.join(
-            THISPATH, 'deps_files', 'test_sdc_files',
-            'eda.work', 'oclib_fifo_with_sdc.synth', 'oclib_fifo.synth.log'
-        )
-        sdc_lines = self.get_log_lines_with('.sdc', logfile=synth_log)
-        assert sdc_lines
-        for sdc_line in sdc_lines:
-            assert 'oclib_fifo_vivado.sdc' in sdc_line
-            assert 'test_sdc_files' in sdc_line
-            assert 'eda.work' not in sdc_line

opencos/tests/test_oc_cli.py

@@ -1,25 +0,0 @@
-'''pytests for: ocli
-
-If you want to run this, consider running from the root of opencos repo:
-    > pytest --verbose opencos/*/*.py
-    > python3 -m pytest --verbose opencos/*/*.py
-which avoids using any pip installed opencos.eda
-'''
-
-from opencos.hw import oc_cli
-
-def test_args_help():
-    '''simple test for: oc_cli --help'''
-    rc = oc_cli.main('--help')
-    print(f'{rc=}')
-    assert not rc
-
-def test_parse_names():
-    '''simple test for: oc_cli.parse_names()'''
-    oc_cli.parse_names()
-    t = getattr(oc_cli, 'name_tables', None)
-    assert t is not None
-    assert len(t.keys()) == 7, f'Expect exactly 7 keys in t, {t=}'
-    entry = t.get('OC_VENDOR', None)
-    assert entry is not None
-    assert len(entry.keys()) >= 1

opencos/tests/test_tools.py

@@ -1,404 +0,0 @@
-'''pytests for testing a few different tools within opencos.eda'''
-
-# pylint: disable=R0801 # (similar lines in 2+ files)
-
-import os
-import shutil
-import sys
-import pytest
-
-from opencos import eda_base
-from opencos.tools.verilator import ToolVerilator
-from opencos.tools.vivado import ToolVivado
-from opencos.tools.cocotb import ToolCocotb
-from opencos.tests import helpers
-from opencos.tests.helpers import Helpers, eda_wrap, eda_wrap_is_sim_fail, config, tools_loaded
-from opencos.utils.markup_helpers import yaml_safe_load
-from opencos.utils import status_constants
-
-
-THISPATH = os.path.dirname(__file__)
-
-def chdir_remove_work_dir(relpath):
-    '''Changes dir to relpath, removes the work directories (eda.work, eda.export*)'''
-    return helpers.chdir_remove_work_dir(THISPATH, relpath)
-
-def filter_tools(tools: list) -> list:
-    '''Given a list of tool l, filters to return a list of tools that are loaded'''
-    return [x for x in tools if x in tools_loaded]
-
-
-def test_tools_loaded():
-    '''Does not directly call 'eda.main' instead create a few Tool
-
-    class objects and confirm the versioning methods work.
-    '''
-    assert config
-    assert len(config.keys()) > 0
-
-
-    # It's possible we're running in some container or install that has no tools, for example,
-    # Windows.
-    if sys.platform.startswith('win') and \
-       not helpers.can_run_eda_command('elab', 'sim', cfg=config):
-        # Windows, no handlers for elab or sim:
-        pass
-    else:
-        basic_tools_available_via_path = any(shutil.which(x) for x in (
-            'verilator', 'iverilog', 'xsim', 'vsim', 'slang', 'yosys'
-        ))
-        if basic_tools_available_via_path:
-            assert len(tools_loaded) > 0
-
-    def version_checker(
-            obj: eda_base.Tool, chk_str: str
-    ) -> None:
-        assert obj.get_versions()
-        full_ver = obj.get_full_tool_and_versions()
-        assert chk_str in full_ver, f'{chk_str=} not in {full_ver=}'
-        ver_num = full_ver.rsplit(':', maxsplit=1)[-1]
-        if 'b' in ver_num:
-            ver_num = ver_num.split('b')[0]
-        if '.' in ver_num:
-            major_ver = ver_num.split('.')[0]
-            assert major_ver.isdigit(), (
-                f'Major version {major_ver=} is not a digit, from {full_ver=}'
-            )
-            assert float(major_ver) >= 0, (
-                f'{major_ver=} is not a valid version number, from {full_ver=}'
-            )
-        else:
-            assert float(ver_num), f'{ver_num=} is not a float, from {full_ver=}'
-
-
-    # Do some very crude checks on the eda.Tool methods, and make
-    # sure versions work for Verilator and Vivado:
-    if 'verilator' in tools_loaded:
-        my_tool = ToolVerilator(config={})
-        version_checker(obj=my_tool, chk_str='verilator:')
-
-    if 'vivado' in tools_loaded:
-        my_tool = ToolVivado(config={})
-        version_checker(obj=my_tool, chk_str='vivado:')
-
-    if 'cocotb' in tools_loaded:
-        my_tool = ToolCocotb(config={})
-        version_checker(obj=my_tool, chk_str='cocotb:')
-
-# Run these on simulation tools.
-list_of_commands = [
-    'sim',
-    'elab'
-]
-
-list_of_tools = [
-    'iverilog',
-    'verilator',
-    'vivado',
-    'modelsim_ase',
-    'questa_fse',
-]
-
-list_of_deps_targets = [
-    ('tb_no_errs', True),       # target:str, sim_expect_pass:bool (sim only, all elab should pass)
-    ('tb_dollar_fatal', False),
-    ('tb_dollar_err', False),
-]
-
-list_of_added_sim_args = [
-    '',
-    '--gui --test-mode',
-]
-
-list_of_loaded_tools = filter_tools(list_of_tools)
-
-cannot_use_cocotb = 'cocotb' not in tools_loaded or \
-    ('iverilog' not in tools_loaded and \
-     'verilator' not in tools_loaded)
-CANNOT_USE_COCOTB_REASON = 'requires cocotb in tools_loaded, and one of (iverilog, verilator) too'
-
-
-class TestSimElabTools(Helpers):
-    '''Tests for eda sim|elab for various tools with various args'''
-
-    DEFAULT_DIR = os.path.join(THISPATH, 'deps_files', 'test_err_fatal')
-
-    @pytest.mark.parametrize("command", list_of_commands)
-    @pytest.mark.parametrize("tool", list_of_loaded_tools)
-    @pytest.mark.parametrize("target,sim_expect_pass", list_of_deps_targets)
-    @pytest.mark.parametrize("added_sim_args_str", list_of_added_sim_args)
-    def test_pass_or_fail(
-            self, command, tool, target, sim_expect_pass, added_sim_args_str
-    ):
-        '''tests that: eda <sim|elab> --tool <parameter-tool> <parameter-args> <parameter-target>
-
-        will correctly pass or fail depending on if it is supported or not.
-
-        Also tests for: non-gui, or --gui --test-mode (runs non-gui, but most python args will
-        be for --gui mode, signal logging, etc).
-        '''
-        added_args_str = ''
-        if command == 'sim':
-            added_args_str = added_sim_args_str
-
-        rc = self.log_it(f'{command} --tool {tool} {added_args_str} {target}')
-        print(f'{rc=}')
-        tool_error_count_lines = self.get_log_lines_with('tool errors')
-        if command != 'sim' or sim_expect_pass:
-            # command='elab' should pass.
-            assert rc == 0
-            assert tool_error_count_lines
-            assert all('tool warnings' in line for line in tool_error_count_lines)
-            assert all(' 0 tool errors' in line for line in tool_error_count_lines)
-
-        else:
-            assert eda_wrap_is_sim_fail(rc)
-            assert tool_error_count_lines
-            assert all('tool warnings' in line for line in tool_error_count_lines)
-            # May or may not have reported tool errors.
-            assert all('tool errors' in line for line in tool_error_count_lines)
-            # The final line of tool warnings/errors should have > 0 tool errors,
-            # since we were checking SV $error and $fatal
-            assert ' 0 tool errors' not in tool_error_count_lines[-1]
-            # The line should end with ' X tool errors'
-            parts = tool_error_count_lines[-1].split()
-            assert parts[-3].isdigit()
-            assert int(parts[-3]) in (1, 2, 3) # we should have at least 1, but some tools dup.
-
-
-@pytest.mark.skipif('vivado' not in tools_loaded, reason="requires vivado")
-def test_vivado_tool_defines():
-    '''This test attempts to confirm that the following class inheritance works:
-
-    Command <- CommandDesign <- CommandSim <- CommandSimVivado <- CommandElabVivado
-
-    in particular that CommandElabVivado(CommandSimVivado, ToolVivado) has the
-    correct ToolVivado.set_tool_defines() method, and that no other parent Command
-    class has overriden it to defeat the defines that should be set.
-
-    We also run with an added dependency (lib_ultrascale_plus_defines) to check that
-    defines are set as expected.
-    '''
-
-    chdir_remove_work_dir('../../lib')
-    rc = eda_wrap(
-        'elab', '--tool', 'vivado', 'third_party/vendors/xilinx/lib_ultrascale_plus_defines',
-        'oclib_fifo'
-    )
-    assert rc == 0
-
-    # Confirm that args and defines we expected to be set are set.
-    eda_config_yml_path = os.path.join(
-        os.getcwd(), 'eda.work', 'oclib_fifo.elab', 'eda_output_config.yml'
-    )
-
-    data = yaml_safe_load(eda_config_yml_path)
-    assert 'args' in data
-    assert data['args'].get('top', '') == 'oclib_fifo'
-    assert 'config' in data
-    assert 'eda_original_args' in data['config']
-    assert 'oclib_fifo' in data['config']['eda_original_args']
-    assert data.get('target', '') == 'oclib_fifo'
-
-
-    # This checks opencos.tools.vivado.ToolVivado.set_tool_defines():
-    # We ran with --xilinx, so we expect certain defines to be set, others not to be set.
-    assert 'defines' in data
-
-    assert 'OC_TOOL_VIVADO' in data['defines']
-    assert 'OC_LIBRARY' in data['defines']
-    assert 'OC_LIBRARY_ULTRASCALE_PLUS' in data['defines']
-
-    assert 'OC_LIBRARY_BEHAVIORAL' not in data['defines']
-    assert 'VERILATOR' not in data['defines']
-    assert 'SYNTHESIS' not in data['defines']
-
-    assert data['defines']['OC_LIBRARY'] == '1'
-    assert data['defines']['OC_LIBRARY_ULTRASCALE_PLUS'] is None # key present, no value
-
-
-class TestCocotb:
-    '''Namespace class for cocotb tests'''
-
-    @pytest.mark.skipif(cannot_use_cocotb, reason=CANNOT_USE_COCOTB_REASON)
-    def test_cocotb_tool_defines(self):
-        '''Test cocotb tool defines, configs, and integration.'''
-
-        chdir_remove_work_dir('../../examples/cocotb')
-
-        # Test 0: Using eda multi:
-        rc = eda_wrap('multi', 'sim', '--tool=cocotb', '*test')
-        assert rc == 0
-
-        # Test 1: basic cocotb sim command with Python runner (default)
-        rc = eda_wrap('sim', '--tool', 'cocotb', 'cocotb_counter_test')
-        assert rc == 0
-
-        # Test 2: cocotb works with different simulators/configurations
-        rc = eda_wrap('sim', '--tool', 'cocotb', 'cocotb_counter_waves_test')
-        assert rc == 0
-
-        # Test 3: Makefile approach
-        rc = eda_wrap('sim', '--tool', 'cocotb', 'cocotb_counter_makefile_test')
-        assert rc == 0
-
-        # Test 4: cocotb-specific defines are set correctly
-        eda_config_yml_path = os.path.join(
-            os.getcwd(), 'eda.work', 'cocotb_counter_test.sim', 'eda_output_config.yml'
-        )
-
-        data = yaml_safe_load(eda_config_yml_path)
-        assert 'args' in data
-        assert data['args'].get('top', '') == 'counter'
-        assert 'config' in data
-        assert 'eda_original_args' in data['config']
-        assert 'cocotb_counter_test' in data['config']['eda_original_args'] or \
-            './cocotb_counter_test' in data['config']['eda_original_args']
-        assert data.get('target', '') == 'cocotb_counter_test'
-
-        assert 'defines' in data
-        assert 'OC_TOOL_COCOTB' in data['defines']
-        assert 'SIMULATION' in data['defines']
-        assert 'COCOTB' in data['defines']
-
-        assert data['defines']['SIMULATION'] == 1
-        assert data['defines']['COCOTB'] == 1
-        assert data['defines']['OC_TOOL_COCOTB'] is None  # key present, no value
-
-        assert 'VERILATOR' not in data['defines']
-        assert 'SYNTHESIS' not in data['defines']
-
-
-
-    @pytest.mark.parametrize("cocotb_simulator", ['verilator', 'iverilog'])
-    @pytest.mark.parametrize("waves_arg", ["", "--waves"])
-    @pytest.mark.skipif(cannot_use_cocotb, reason=CANNOT_USE_COCOTB_REASON)
-    def test_cocotb_different_simulators(self, cocotb_simulator, waves_arg):
-        '''Test cocotb with different simulator configurations.'''
-
-        if cocotb_simulator not in tools_loaded:
-            pytest.skip(f"{cocotb_simulator=} skipped, {tools_loaded=}")
-            return #skip/bypass
-
-        chdir_remove_work_dir('../../examples/cocotb')
-
-        rc = eda_wrap(
-            'sim', '--tool', 'cocotb',
-            f'--cocotb-simulator={cocotb_simulator}',
-            waves_arg,
-            'cocotb_counter_test',
-        )
-        assert rc == 0
-
-
-
-    @pytest.mark.skipif(cannot_use_cocotb, reason=CANNOT_USE_COCOTB_REASON)
-    def test_cocotb_tool_instantiation(self):
-        '''Test that ToolCocotb can be instantiated and has correct properties.'''
-
-        tool = ToolCocotb(config={})
-
-        # version detection works
-        version = tool.get_versions()
-        assert version, "Should return a non-empty version string"
-        assert isinstance(version, str)
-
-        # tool defines
-        tool.set_tool_defines()
-        defines = tool.defines
-        assert 'SIMULATION' in defines
-        assert 'COCOTB' in defines
-        assert 'OC_TOOL_COCOTB' in defines
-        assert defines['SIMULATION'] == 1
-        assert defines['COCOTB'] == 1
-
-
-
-    @pytest.mark.skipif(cannot_use_cocotb, reason=CANNOT_USE_COCOTB_REASON)
-    def test_cocotb_failure_cases(self):
-        '''Test cocotb failure scenarios to ensure proper error handling.'''
-
-        chdir_remove_work_dir('../../examples/cocotb')
-
-        # Test 1: missing test files should fail gracefully
-        rc = eda_wrap('sim', '--tool', 'cocotb', 'counter')  # Just HDL, no test files
-        assert rc == status_constants.EDA_DEPS_TARGET_NOT_FOUND, \
-            "Should fail when no cocotb test files are found"
-
-        # Test 2: non-existent target should fail
-        rc = eda_wrap('sim', '--tool', 'cocotb', 'nonexistent_target')
-        assert rc in (
-            status_constants.EDA_DEPS_TARGET_NOT_FOUND,
-            # b/c we run eda_wrap, eda.main will continue to run after first error.
-            status_constants.EDA_COMMAND_MISSING_TOP
-        ), "Should fail for non-existent target"
-
-        # Test 3: invalid cocotb test module should fail
-        rc = eda_wrap(
-            'sim', '--tool', 'cocotb',
-            '--cocotb-test-module=nonexistent_test',
-            'cocotb_counter_test',
-        )
-        assert eda_wrap_is_sim_fail(rc), \
-            f"Should fail with invalid test module {rc=}"
-
-
-    @pytest.mark.skipif(cannot_use_cocotb, reason=CANNOT_USE_COCOTB_REASON)
-    def test_cocotb_missing_dependencies(self):
-        '''Test cocotb behavior when dependencies are missing.'''
-
-        # Test missing cocotb installation (simulate by checking error handling)
-        tool = ToolCocotb(config={})
-        version = tool.get_versions()
-        assert version, "Should return version when cocotb is properly installed"
-
-        # Test tool defines are properly set even with minimal config
-        tool.set_tool_defines()
-        defines = tool.defines
-        assert 'SIMULATION' in defines
-        assert 'COCOTB' in defines
-        assert 'OC_TOOL_COCOTB' in defines
-
-
-    @pytest.mark.skipif(cannot_use_cocotb, reason=CANNOT_USE_COCOTB_REASON)
-    def test_cocotb_invalid_simulator(self):
-        '''Test cocotb with invalid simulator configuration.'''
-
-        chdir_remove_work_dir('../../examples/cocotb')
-
-        rc = eda_wrap(
-            'sim', '--tool', 'cocotb',
-            '--cocotb-simulator=invalid_sim',
-            'cocotb_counter_test',
-        )
-        assert eda_wrap_is_sim_fail(rc), \
-            f"Should fail with invalid simulator {rc=}"
-
-    @pytest.mark.skipif(cannot_use_cocotb, reason=CANNOT_USE_COCOTB_REASON)
-    def test_cocotb_malformed_hdl(self):
-        '''Test cocotb with malformed HDL files.'''
-
-        chdir_remove_work_dir('../../lib/tests')
-
-        # Test with a target that has syntax errors - should fail during compilation
-        rc = eda_wrap(
-            'sim', '--tool', 'cocotb',
-            '--cocotb-test-module=test_counter',
-            'tb_dollar_fatal',
-        )
-
-        # eda_wrap may continue to errors beyond normal sim fails:
-        assert eda_wrap_is_sim_fail(rc) or \
-            f"Should fail with malformed HDL or failing test assertions {rc=}"
-
-
-    @pytest.mark.skipif(cannot_use_cocotb, reason=CANNOT_USE_COCOTB_REASON)
-    def test_cocotb_test_failures(self):
-        '''Test that cocotb properly reports test failures.'''
-
-        chdir_remove_work_dir('../../examples/cocotb')
-
-        # Intentionally failing cocotb tests
-        rc = eda_wrap('sim', '--tool', 'cocotb', 'cocotb_failure_test')
-        assert eda_wrap_is_sim_fail(rc), \
-            f"Should fail when cocotb tests contain assertion failures {rc=}"