From b207df4ebbd3680ec0f3b0f7f809773d365dd94a Mon Sep 17 00:00:00 2001 From: Matthew Honnibal Date: Tue, 20 May 2025 18:46:03 +0200 Subject: [PATCH] Remove test checking for compatibility warning We had a test that relied on an __init_subclass__ on the Pipe, as we did this to provide a migration warning for v2 -> v3. The __init_subclass__ no longer works on Cython classes and the test is old, so just removed it. --- spacy/tests/pipeline/test_pipe_methods.py | 11 ----------- 1 file changed, 11 deletions(-) diff --git a/spacy/tests/pipeline/test_pipe_methods.py b/spacy/tests/pipeline/test_pipe_methods.py index 4dd7bae16..9b9786f04 100644 --- a/spacy/tests/pipeline/test_pipe_methods.py +++ b/spacy/tests/pipeline/test_pipe_methods.py @@ -529,17 +529,6 @@ def test_pipe_label_data_no_labels(pipe): assert "labels" not in get_arg_names(initialize) -def test_warning_pipe_begin_training(): - with pytest.warns(UserWarning, match="begin_training"): - - class IncompatPipe(TrainablePipe): - def __init__(self): - ... - - def begin_training(*args, **kwargs): - ... - - def test_pipe_methods_initialize(): """Test that the [initialize] config reflects the components correctly.""" nlp = Language()