Skip to content

Commit 3cd6550

Browse files
authored
Merge pull request RasaHQ#5861 from Archish27/common-utils-warnings
Updated stack level for FutureWarnings
2 parents ceb8eac + 6169546 commit 3cd6550

File tree

3 files changed

+11
-4
lines changed

3 files changed

+11
-4
lines changed

changelog/5385.bugfix.rst

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
Stack level for ``FutureWarning`` updated to level 2.

rasa/utils/common.py

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -357,10 +357,8 @@ def formatwarning(
357357
# try to set useful defaults for the most common warning categories
358358
if category == DeprecationWarning:
359359
kwargs["stacklevel"] = 3
360-
elif category == UserWarning:
360+
elif category in (UserWarning, FutureWarning):
361361
kwargs["stacklevel"] = 2
362-
elif category == FutureWarning:
363-
kwargs["stacklevel"] = 3
364362

365363
warnings.formatwarning = formatwarning
366364
warnings.warn(message, category=category, **kwargs)

tests/utils/test_common.py

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,14 +12,22 @@ def test_sort_dicts_by_keys():
1212
assert actual == expected
1313

1414

15-
def test_raise_warning():
15+
def test_raise_user_warning():
1616
with pytest.warns(UserWarning) as record:
1717
raise_warning("My warning.")
1818

1919
assert len(record) == 1
2020
assert record[0].message.args[0] == "My warning."
2121

2222

23+
def test_raise_future_warning():
24+
with pytest.warns(FutureWarning) as record:
25+
raise_warning("My future warning.", FutureWarning)
26+
27+
assert len(record) == 1
28+
assert record[0].message.args[0] == "My future warning."
29+
30+
2331
def test_raise_deprecation():
2432
with pytest.warns(DeprecationWarning) as record:
2533
raise_warning("My warning.", DeprecationWarning)

0 commit comments

Comments
 (0)