[Python] fix logic about which scenario to add msrest as dependency (#3267)

* Update dependencies

* Add changelog .chronus/changes/auto-microsoft-python-fix-msrest-dependency-2025-10-28-11-58-45.md

* Update dependencies (2025-11-28 04:57:04)

* Regenerate for autorest.python (2025-11-28 05:16:39)

* add test case

---------

Co-authored-by: AutoPrFromHttpClientPython <AutoPrFromHttpClientPython>

Author: msyyc

.chronus/changes/auto-microsoft-python-fix-msrest-dependency-2025-10-28-11-58-45.md

@@ -0,0 +1,8 @@
+---
+changeKind: fix
+packages:
+  - "@autorest/python"
+  - "@azure-tools/typespec-python"
+---
+
+fix logic about which scenario to add msrest as dependency

packages/autorest.python/package.json

@@ -29,7 +29,7 @@
   },
   "homepage": "https://github.com/Azure/autorest.python/blob/main/README.md",
   "dependencies": {
-    "@typespec/http-client-python": "https://artprodcus3.artifacts.visualstudio.com/A0fb41ef4-5012-48a9-bf39-4ee3de03ee35/29ec6040-b234-4e31-b139-33dc4287b756/_apis/artifact/cGlwZWxpbmVhcnRpZmFjdDovL2F6dXJlLXNkay9wcm9qZWN0SWQvMjllYzYwNDAtYjIzNC00ZTMxLWIxMzktMzNkYzQyODdiNzU2L2J1aWxkSWQvNTYxODM0NC9hcnRpZmFjdE5hbWUvYnVpbGRfYXJ0aWZhY3RzX3B5dGhvbg2/content?format=file&subPath=%2Fpackages%2Ftypespec-http-client-python-0.20.3.tgz",
+    "@typespec/http-client-python": "https://artprodcus3.artifacts.visualstudio.com/A0fb41ef4-5012-48a9-bf39-4ee3de03ee35/29ec6040-b234-4e31-b139-33dc4287b756/_apis/artifact/cGlwZWxpbmVhcnRpZmFjdDovL2F6dXJlLXNkay9wcm9qZWN0SWQvMjllYzYwNDAtYjIzNC00ZTMxLWIxMzktMzNkYzQyODdiNzU2L2J1aWxkSWQvNTYxODM5OC9hcnRpZmFjdE5hbWUvYnVpbGRfYXJ0aWZhY3RzX3B5dGhvbg2/content?format=file&subPath=%2Fpackages%2Ftypespec-http-client-python-0.20.3.tgz",
     "@autorest/system-requirements": "~1.0.2",
     "fs-extra": "~11.2.0",
     "tsx": "~4.19.1"

packages/autorest.python/samples/specification/azure-mgmt-pyproject/test/azure-mgmt-pyproject/pyproject.toml

@@ -31,7 +31,7 @@ requires-python = ">=3.9"
 keywords = ["azure", "azure sdk"]
 
 dependencies = [
-    "msrest>=0.7.1",
+    "isodate>=0.6.1",
     "azure-mgmt-core>=1.6.0",
     "typing-extensions>=4.6.0",
 ]

packages/autorest.python/samples/specification/azure-mgmt-test/test/azure-mgmt-test/setup.py

@@ -63,7 +63,7 @@
         "azure.mgmt.test": ["py.typed"],
     },
     install_requires=[
-        "msrest>=0.7.1",
+        "isodate>=0.6.1",
         "azure-mgmt-core>=1.6.0",
         "typing-extensions>=4.6.0",
     ],

packages/autorest.python/samples/specification/management/generated/setup.py

@@ -23,7 +23,7 @@
     packages=find_packages(),
     include_package_data=True,
     install_requires=[
-        "msrest>=0.7.1",
+        "isodate>=0.6.1",
         "azure-mgmt-core>=1.6.0",
         "typing-extensions>=4.6.0",
     ],

packages/autorest.python/test/azure/legacy/AcceptanceTests/test_duration.py

@@ -24,15 +24,8 @@
 #
 # --------------------------------------------------------------------------
 
-import unittest
-import subprocess
-import sys
-import isodate
-import tempfile
-import json
 from uuid import uuid4
-from datetime import date, datetime, timedelta
-import os
+from datetime import timedelta
 from os.path import dirname, pardir, join, realpath
 
 from azure.core.exceptions import DeserializationError
@@ -76,3 +69,28 @@ def test_operation_groups(self):
         from bodyduration.operations._duration_operations import DurationOperations as DurationOperationsPy2
 
         assert DurationOperations == DurationOperationsPy2
+
+    def test_setup_py_requirements(self):
+        # this file is generated, so we can basic check on it
+        # We want to make sure "isodate" is a dependency and not "msrest"
+        setup_path = realpath(
+            join(
+                dirname(realpath(__file__)),
+                "..",
+                "Expected",
+                "AcceptanceTests",
+                "AzureBodyDuration",
+                "setup.py",
+            )
+        )
+        # Super ugly, but I don't want to write a parser for a setup.py
+        with open(setup_path, "r") as setup_file:
+            content = setup_file.read()
+
+        # Let's just check the install_requires is what we expect
+        # We can't use ast, since it's not abstract syntax tree, it's just text.
+        # Let's find the install_requires
+        install_requires_part = content[content.find("install_requires") :]
+        # This is very fragile, but for a test on generated code, that's what we want
+        assert "isodate" in install_requires_part
+        assert "msrest" not in install_requires_part

packages/autorest.python/test/azure/legacy/Expected/AcceptanceTests/AzureBodyDuration/setup.py

@@ -23,7 +23,7 @@
     packages=find_packages(),
     include_package_data=True,
     install_requires=[
-        "msrest>=0.7.1",
+        "isodate>=0.6.1",
         "azure-core>=1.35.0",
         "typing-extensions>=4.6.0",
     ],

packages/autorest.python/test/azure/legacy/Expected/AcceptanceTests/AzureReport/setup.py

@@ -23,7 +23,7 @@
     packages=find_packages(),
     include_package_data=True,
     install_requires=[
-        "msrest>=0.7.1",
+        "isodate>=0.6.1",
         "azure-core>=1.35.0",
         "typing-extensions>=4.6.0",
     ],

packages/autorest.python/test/azure/legacy/Expected/AcceptanceTests/CustomPollerPager/setup.py

@@ -23,7 +23,7 @@
     packages=find_packages(),
     include_package_data=True,
     install_requires=[
-        "msrest>=0.7.1",
+        "isodate>=0.6.1",
         "azure-mgmt-core>=1.6.0",
         "typing-extensions>=4.6.0",
     ],

packages/autorest.python/test/azure/legacy/Expected/AcceptanceTests/CustomUrlPaging/setup.py

@@ -23,7 +23,7 @@
     packages=find_packages(),
     include_package_data=True,
     install_requires=[
-        "msrest>=0.7.1",
+        "isodate>=0.6.1",
         "azure-core>=1.35.0",
         "typing-extensions>=4.6.0",
     ],