Commit bd96a86
committed
Fix asyncio.CancelledError detection and add test coverage
- Fix module check to use startswith("asyncio") instead of == "asyncio"
to properly match asyncio.exceptions.CancelledError
- Add test case for asyncio.CancelledError to verify it bypasses retry loop
Addresses review feedback from chatgpt-codex-connector bot1 parent 40f4c4a commit bd96a86
File tree
3 files changed
+33
-1
lines changed- repos
- src/openai
- tests
3 files changed
+33
-1
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
112 | 112 | | |
113 | 113 | | |
114 | 114 | | |
115 | | - | |
| 115 | + | |
116 | 116 | | |
117 | 117 | | |
118 | 118 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
896 | 896 | | |
897 | 897 | | |
898 | 898 | | |
| 899 | + | |
| 900 | + | |
| 901 | + | |
| 902 | + | |
| 903 | + | |
| 904 | + | |
| 905 | + | |
| 906 | + | |
| 907 | + | |
| 908 | + | |
| 909 | + | |
| 910 | + | |
| 911 | + | |
| 912 | + | |
| 913 | + | |
| 914 | + | |
| 915 | + | |
| 916 | + | |
| 917 | + | |
| 918 | + | |
| 919 | + | |
| 920 | + | |
| 921 | + | |
| 922 | + | |
| 923 | + | |
| 924 | + | |
| 925 | + | |
| 926 | + | |
| 927 | + | |
| 928 | + | |
| 929 | + | |
899 | 930 | | |
900 | 931 | | |
901 | 932 | | |
| |||
0 commit comments