Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix binskim warnings for mac, linux, and pyd files #1856

Merged
merged 3 commits into from
Mar 5, 2025

Fix pydevd bits too

54f0f15
Select commit
Loading
Failed to load commit list.
Merged

Fix binskim warnings for mac, linux, and pyd files #1856

Fix pydevd bits too
54f0f15
Select commit
Loading
Failed to load commit list.
Azure Pipelines / debugpy-test failed Mar 4, 2025 in 41m 43s

Build #20250304.2 had test failures

Details

Tests

  • Failed: 39 (0.20%)
  • Passed: 16,142 (82.16%)
  • Other: 3,466 (17.64%)
  • Total: 19,647

Annotations

Check failure on line 5342 in Build log

See this annotation in the file changed.

@azure-pipelines azure-pipelines / debugpy-test

Build log #L5342

PowerShell exited with code '1'.

Check failure on line 439 in Build log

See this annotation in the file changed.

@azure-pipelines azure-pipelines / debugpy-test

Build log #L439

PowerShell exited with code '1'.

Check failure on line 1 in test_targets[script_args--8888-listen-code]

See this annotation in the file changed.

@azure-pipelines azure-pipelines / debugpy-test

test_targets[script_args--8888-listen-code]

AssertionError: assert [] == ['ham', '--li...m', '--', ...]
  
  Right contains 10 more items, first extra item: 'ham'
  Use -v to get more diff
Raw output
cli = <function cli.<locals>.parse at 0x0000023B98582D40>, target_kind = 'code'
mode = 'listen', address = '8888', wait_for_client = ''
script_args = ['ham', '--listen', '--wait-for-client', '-y', 'spam', '--', ...]

    @pytest.mark.parametrize("target_kind", ["file", "module", "code"])
    @pytest.mark.parametrize("mode", ["listen", "connect"])
    @pytest.mark.parametrize("address", ["8888", "localhost:8888"])
    @pytest.mark.parametrize("wait_for_client", ["", "wait_for_client"])
    @pytest.mark.parametrize("script_args", ["", "script_args"])
    def test_targets(cli, target_kind, mode, address, wait_for_client, script_args):
        expected_options = {
            "mode": mode,
            "target_kind": target_kind,
            "wait_for_client": False
        }
    
        args = ["--" + mode, address]
    
        host, sep, port = address.partition(":")
        if sep:
            expected_options["address"] = (host, int(port))
        else:
            expected_options["address"] = ("127.0.0.1", int(address))
    
        if wait_for_client:
            args += ["--wait-for-client"]
            expected_options["wait_for_client"] = True
    
        if target_kind == "file":
            target = "spam.py"
            args += [target]
        elif target_kind == "module":
            target = "spam"
            args += ["-m", target]
        elif target_kind == "code":
            target = "123"
            args += ["-c", target]
        else:
            pytest.fail(target_kind)
        expected_options["target"] = target
    
        if script_args:
            script_args = [
                "ham",
                "--listen",
                "--wait-for-client",
                "-y",
                "spam",
                "--",
                "--connect",
                "-c",
                "--something",
                "-m",
            ]
            args += script_args
        else:
            script_args = []
    
        argv, options = cli(args)
>       assert argv == script_args
E       AssertionError: assert [] == ['ham', '--li...m', '--', ...]
E         
E         Right contains 10 more items, first extra item: 'ham'
E         Use -v to get more diff

tests\debugpy\server\test_cli.py:135: AssertionError

Check failure on line 1 in test_targets[script_args--8888-connect-file]

See this annotation in the file changed.

@azure-pipelines azure-pipelines / debugpy-test

test_targets[script_args--8888-connect-file]

AssertionError: assert [] == ['ham', '--li...m', '--', ...]
  
  Right contains 10 more items, first extra item: 'ham'
  Use -v to get more diff
Raw output
cli = <function cli.<locals>.parse at 0x0000023B9879D6C0>, target_kind = 'file'
mode = 'connect', address = '8888', wait_for_client = ''
script_args = ['ham', '--listen', '--wait-for-client', '-y', 'spam', '--', ...]

    @pytest.mark.parametrize("target_kind", ["file", "module", "code"])
    @pytest.mark.parametrize("mode", ["listen", "connect"])
    @pytest.mark.parametrize("address", ["8888", "localhost:8888"])
    @pytest.mark.parametrize("wait_for_client", ["", "wait_for_client"])
    @pytest.mark.parametrize("script_args", ["", "script_args"])
    def test_targets(cli, target_kind, mode, address, wait_for_client, script_args):
        expected_options = {
            "mode": mode,
            "target_kind": target_kind,
            "wait_for_client": False
        }
    
        args = ["--" + mode, address]
    
        host, sep, port = address.partition(":")
        if sep:
            expected_options["address"] = (host, int(port))
        else:
            expected_options["address"] = ("127.0.0.1", int(address))
    
        if wait_for_client:
            args += ["--wait-for-client"]
            expected_options["wait_for_client"] = True
    
        if target_kind == "file":
            target = "spam.py"
            args += [target]
        elif target_kind == "module":
            target = "spam"
            args += ["-m", target]
        elif target_kind == "code":
            target = "123"
            args += ["-c", target]
        else:
            pytest.fail(target_kind)
        expected_options["target"] = target
    
        if script_args:
            script_args = [
                "ham",
                "--listen",
                "--wait-for-client",
                "-y",
                "spam",
                "--",
                "--connect",
                "-c",
                "--something",
                "-m",
            ]
            args += script_args
        else:
            script_args = []
    
        argv, options = cli(args)
>       assert argv == script_args
E       AssertionError: assert [] == ['ham', '--li...m', '--', ...]
E         
E         Right contains 10 more items, first extra item: 'ham'
E         Use -v to get more diff

tests\debugpy\server\test_cli.py:135: AssertionError

Check failure on line 1 in test_targets[script_args--8888-connect-code]

See this annotation in the file changed.

@azure-pipelines azure-pipelines / debugpy-test

test_targets[script_args--8888-connect-code]

AssertionError: assert [] == ['ham', '--li...m', '--', ...]
  
  Right contains 10 more items, first extra item: 'ham'
  Use -v to get more diff
Raw output
cli = <function cli.<locals>.parse at 0x0000023B9879EB60>, target_kind = 'code'
mode = 'connect', address = '8888', wait_for_client = ''
script_args = ['ham', '--listen', '--wait-for-client', '-y', 'spam', '--', ...]

    @pytest.mark.parametrize("target_kind", ["file", "module", "code"])
    @pytest.mark.parametrize("mode", ["listen", "connect"])
    @pytest.mark.parametrize("address", ["8888", "localhost:8888"])
    @pytest.mark.parametrize("wait_for_client", ["", "wait_for_client"])
    @pytest.mark.parametrize("script_args", ["", "script_args"])
    def test_targets(cli, target_kind, mode, address, wait_for_client, script_args):
        expected_options = {
            "mode": mode,
            "target_kind": target_kind,
            "wait_for_client": False
        }
    
        args = ["--" + mode, address]
    
        host, sep, port = address.partition(":")
        if sep:
            expected_options["address"] = (host, int(port))
        else:
            expected_options["address"] = ("127.0.0.1", int(address))
    
        if wait_for_client:
            args += ["--wait-for-client"]
            expected_options["wait_for_client"] = True
    
        if target_kind == "file":
            target = "spam.py"
            args += [target]
        elif target_kind == "module":
            target = "spam"
            args += ["-m", target]
        elif target_kind == "code":
            target = "123"
            args += ["-c", target]
        else:
            pytest.fail(target_kind)
        expected_options["target"] = target
    
        if script_args:
            script_args = [
                "ham",
                "--listen",
                "--wait-for-client",
                "-y",
                "spam",
                "--",
                "--connect",
                "-c",
                "--something",
                "-m",
            ]
            args += script_args
        else:
            script_args = []
    
        argv, options = cli(args)
>       assert argv == script_args
E       AssertionError: assert [] == ['ham', '--li...m', '--', ...]
E         
E         Right contains 10 more items, first extra item: 'ham'
E         Use -v to get more diff

tests\debugpy\server\test_cli.py:135: AssertionError

Check failure on line 1 in test_targets[script_args--8888-connect-module]

See this annotation in the file changed.

@azure-pipelines azure-pipelines / debugpy-test

test_targets[script_args--8888-connect-module]

AssertionError: assert [] == ['ham', '--li...m', '--', ...]
  
  Right contains 10 more items, first extra item: 'ham'
  Use -v to get more diff
Raw output
cli = <function cli.<locals>.parse at 0x0000023B9879E0C0>
target_kind = 'module', mode = 'connect', address = '8888', wait_for_client = ''
script_args = ['ham', '--listen', '--wait-for-client', '-y', 'spam', '--', ...]

    @pytest.mark.parametrize("target_kind", ["file", "module", "code"])
    @pytest.mark.parametrize("mode", ["listen", "connect"])
    @pytest.mark.parametrize("address", ["8888", "localhost:8888"])
    @pytest.mark.parametrize("wait_for_client", ["", "wait_for_client"])
    @pytest.mark.parametrize("script_args", ["", "script_args"])
    def test_targets(cli, target_kind, mode, address, wait_for_client, script_args):
        expected_options = {
            "mode": mode,
            "target_kind": target_kind,
            "wait_for_client": False
        }
    
        args = ["--" + mode, address]
    
        host, sep, port = address.partition(":")
        if sep:
            expected_options["address"] = (host, int(port))
        else:
            expected_options["address"] = ("127.0.0.1", int(address))
    
        if wait_for_client:
            args += ["--wait-for-client"]
            expected_options["wait_for_client"] = True
    
        if target_kind == "file":
            target = "spam.py"
            args += [target]
        elif target_kind == "module":
            target = "spam"
            args += ["-m", target]
        elif target_kind == "code":
            target = "123"
            args += ["-c", target]
        else:
            pytest.fail(target_kind)
        expected_options["target"] = target
    
        if script_args:
            script_args = [
                "ham",
                "--listen",
                "--wait-for-client",
                "-y",
                "spam",
                "--",
                "--connect",
                "-c",
                "--something",
                "-m",
            ]
            args += script_args
        else:
            script_args = []
    
        argv, options = cli(args)
>       assert argv == script_args
E       AssertionError: assert [] == ['ham', '--li...m', '--', ...]
E         
E         Right contains 10 more items, first extra item: 'ham'
E         Use -v to get more diff

tests\debugpy\server\test_cli.py:135: AssertionError