Skip to content

Commit 0db1ec9

Browse files
authored
FIX: Skipping long running test related with wstring conversion (#383)
### Work Item / Issue Reference <!-- IMPORTANT: Please follow the PR template guidelines below. For mssql-python maintainers: Insert your ADO Work Item ID below (e.g. AB#37452) For external contributors: Insert Github Issue number below (e.g. #149) Only one reference is required - either GitHub issue OR ADO Work Item. --> <!-- mssql-python maintainers: ADO Work Item --> > [AB#41135](https://sqlclientdrivers.visualstudio.com/c6d89619-62de-46a0-8b46-70b92a84d85e/_workitems/edit/41135) <!-- External contributors: GitHub Issue --> > GitHub Issue: #<ISSUE_NUMBER> ------------------------------------------------------------------- ### Summary This pull request updates the test suite in `tests/test_013_sqlwchar_conversions.py` by marking several long-running tests as skipped using the `@pytest.mark.skip` decorator. The reason provided is that these stress tests have been moved due to their long execution time, particularly affecting Manylinux64 runs. This helps streamline the test runs and avoid timeouts or delays in CI pipelines. **Test Suite Management:** * Added `@pytest.mark.skip` to multiple test methods to exclude long-running stress tests from regular test runs, with a note that these have been moved due to their duration on Manylinux64 environments. [[1]](diffhunk://#diff-6c635b84f70732583dcd72363ab4a523f912598c81ca34bd466810b3939856efR58) [[2]](diffhunk://#diff-6c635b84f70732583dcd72363ab4a523f912598c81ca34bd466810b3939856efR90) [[3]](diffhunk://#diff-6c635b84f70732583dcd72363ab4a523f912598c81ca34bd466810b3939856efR122) [[4]](diffhunk://#diff-6c635b84f70732583dcd72363ab4a523f912598c81ca34bd466810b3939856efR156) [[5]](diffhunk://#diff-6c635b84f70732583dcd72363ab4a523f912598c81ca34bd466810b3939856efR201) [[6]](diffhunk://#diff-6c635b84f70732583dcd72363ab4a523f912598c81ca34bd466810b3939856efR234) [[7]](diffhunk://#diff-6c635b84f70732583dcd72363ab4a523f912598c81ca34bd466810b3939856efR297) [[8]](diffhunk://#diff-6c635b84f70732583dcd72363ab4a523f912598c81ca34bd466810b3939856efR325) [[9]](diffhunk://#diff-6c635b84f70732583dcd72363ab4a523f912598c81ca34bd466810b3939856efR354) [[10]](diffhunk://#diff-6c635b84f70732583dcd72363ab4a523f912598c81ca34bd466810b3939856efR389) [[11]](diffhunk://#diff-6c635b84f70732583dcd72363ab4a523f912598c81ca34bd466810b3939856efR446) [[12]](diffhunk://#diff-6c635b84f70732583dcd72363ab4a523f912598c81ca34bd466810b3939856efR480) [[13]](diffhunk://#diff-6c635b84f70732583dcd72363ab4a523f912598c81ca34bd466810b3939856efR496) [[14]](diffhunk://#diff-6c635b84f70732583dcd72363ab4a523f912598c81ca34bd466810b3939856efR512)
1 parent f2d2d62 commit 0db1ec9

File tree

1 file changed

+14
-0
lines changed

1 file changed

+14
-0
lines changed

tests/test_013_sqlwchar_conversions.py

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -55,6 +55,7 @@ def test_surrogate_pair_high_without_low(self):
5555
except Exception:
5656
pass
5757

58+
@pytest.mark.skip(reason="STRESS TESTS moved due to long running time in ")
5859
def test_surrogate_pair_low_without_high(self):
5960
"""
6061
Test low surrogate without preceding high surrogate.
@@ -86,6 +87,7 @@ def test_surrogate_pair_low_without_high(self):
8687
except Exception:
8788
pass
8889

90+
@pytest.mark.skip(reason="STRESS TESTS moved due to long running time in Manylinux64 runs")
8991
def test_valid_surrogate_pairs(self):
9092
"""
9193
Test valid high+low surrogate pairs.
@@ -117,6 +119,7 @@ def test_valid_surrogate_pairs(self):
117119
except Exception:
118120
pass # Connection may fail, but string conversion should work
119121

122+
@pytest.mark.skip(reason="STRESS TESTS moved due to long running time in Manylinux64 runs")
120123
def test_bmp_characters(self):
121124
"""
122125
Test Basic Multilingual Plane (BMP) characters (U+0000 to U+FFFF).
@@ -150,6 +153,7 @@ def test_bmp_characters(self):
150153
except Exception:
151154
pass
152155

156+
@pytest.mark.skip(reason="STRESS TESTS moved due to long running time in Manylinux64 runs")
153157
def test_invalid_scalar_values(self):
154158
"""
155159
Test invalid Unicode scalar values.
@@ -194,6 +198,7 @@ def test_invalid_scalar_values(self):
194198
except Exception:
195199
pass
196200

201+
@pytest.mark.skip(reason="STRESS TESTS moved due to long running time in Manylinux64 runs")
197202
def test_wstring_to_sqlwchar_bmp(self):
198203
"""
199204
Test WStringToSQLWCHAR with BMP characters.
@@ -226,6 +231,7 @@ def test_wstring_to_sqlwchar_bmp(self):
226231
except Exception:
227232
pass
228233

234+
@pytest.mark.skip(reason="STRESS TESTS moved due to long running time in Manylinux64 runs")
229235
def test_wstring_to_sqlwchar_surrogate_pairs(self):
230236
"""
231237
Test WStringToSQLWCHAR with characters requiring surrogate pairs.
@@ -288,6 +294,7 @@ def test_wstring_to_sqlwchar_invalid_scalars(self):
288294
except Exception:
289295
pass # Expected to fail, but conversion should handle it
290296

297+
@pytest.mark.skip(reason="STRESS TESTS moved due to long running time in Manylinux64 runs")
291298
def test_empty_and_null_strings(self):
292299
"""
293300
Test edge cases with empty and null strings.
@@ -315,6 +322,7 @@ def test_empty_and_null_strings(self):
315322
except Exception:
316323
pass
317324

325+
@pytest.mark.skip(reason="STRESS TESTS moved due to long running time in Manylinux64 runs")
318326
def test_mixed_character_sets(self):
319327
"""
320328
Test strings with mixed character sets and surrogate pairs.
@@ -343,6 +351,7 @@ def test_mixed_character_sets(self):
343351
except Exception:
344352
pass
345353

354+
@pytest.mark.skip(reason="STRESS TESTS moved due to long running time in Manylinux64 runs")
346355
def test_boundary_code_points(self):
347356
"""
348357
Test boundary code points for surrogate range and Unicode limits.
@@ -377,6 +386,7 @@ def test_boundary_code_points(self):
377386
except Exception:
378387
pass # Validation happens during conversion
379388

389+
@pytest.mark.skip(reason="STRESS TESTS moved due to long running time in Manylinux64 runs")
380390
def test_surrogate_pair_calculations(self):
381391
"""
382392
Test the arithmetic for surrogate pair encoding/decoding.
@@ -433,6 +443,7 @@ def test_surrogate_pair_calculations(self):
433443
except Exception:
434444
pass
435445

446+
@pytest.mark.skip(reason="STRESS TESTS moved due to long running time in Manylinux64 runs")
436447
def test_null_terminator_handling(self):
437448
"""
438449
Test that null terminators are properly handled.
@@ -466,6 +477,7 @@ def test_null_terminator_handling(self):
466477
class TestSQLWCHARConversionsCommon:
467478
"""Tests that run on all platforms (Windows, Linux, macOS)."""
468479

480+
@pytest.mark.skip(reason="STRESS TESTS moved due to long running time in Manylinux64 runs")
469481
def test_unicode_round_trip_ascii(self):
470482
"""Test that ASCII characters round-trip correctly."""
471483
import mssql_python
@@ -481,6 +493,7 @@ def test_unicode_round_trip_ascii(self):
481493
except Exception:
482494
pass
483495

496+
@pytest.mark.skip(reason="STRESS TESTS moved due to long running time in Manylinux64 runs")
484497
def test_unicode_round_trip_emoji(self):
485498
"""Test that emoji characters round-trip correctly."""
486499
import mssql_python
@@ -496,6 +509,7 @@ def test_unicode_round_trip_emoji(self):
496509
except Exception:
497510
pass
498511

512+
@pytest.mark.skip(reason="STRESS TESTS moved due to long running time in Manylinux64 runs")
499513
def test_unicode_round_trip_multilingual(self):
500514
"""Test that multilingual text round-trips correctly."""
501515
import mssql_python

0 commit comments

Comments
 (0)