fix: added cache eviction tests for #2971

Author: Julien Kronegg

cucumber-core/src/test/java/io/cucumber/core/runner/CachingGlueTest.java

@@ -16,6 +16,7 @@
 import io.cucumber.core.gherkin.Feature;
 import io.cucumber.core.gherkin.Step;
 import io.cucumber.core.runtime.TimeServiceEventBus;
+import io.cucumber.core.stepexpression.StepTypeRegistry;
 import io.cucumber.cucumberexpressions.ParameterByTypeTransformer;
 import io.cucumber.cucumberexpressions.ParameterType;
 import io.cucumber.datatable.DataTable;
@@ -528,6 +529,94 @@ void parameterTypeDefinition_with_source_reference_emits_parameterType_with_non_
         assertNotNull(sourceReference.getJavaStackTraceElement());
     }
 
+    @Test
+    void prepareGlue_cache_evicted_when_language_changes() {
+        // Given
+        glue.prepareGlue(LANGUAGE);
+        StepTypeRegistry stepTypeRegistry1 = glue.getStepTypeRegistry();
+
+        // When
+        glue.prepareGlue(Locale.FRENCH);
+        StepTypeRegistry stepTypeRegistry2 = glue.getStepTypeRegistry();
+
+        // Then
+        assertThat(stepTypeRegistry1!=stepTypeRegistry2, is(true));
+    }
+
+    @Test
+    void prepareGlue_cache_not_evicted_when_language_remains() {
+        // Given
+        glue.prepareGlue(LANGUAGE);
+        StepTypeRegistry stepTypeRegistry1 = glue.getStepTypeRegistry();
+
+        // When
+        glue.prepareGlue(LANGUAGE);
+        StepTypeRegistry stepTypeRegistry2 = glue.getStepTypeRegistry();
+
+        // Then
+        assertThat(stepTypeRegistry1==stepTypeRegistry2, is(true));
+    }
+
+    @Test
+    void prepareGlue_cache_evicted_when_stepDefinition_added() {
+        // Given
+        glue.prepareGlue(LANGUAGE);
+        StepTypeRegistry stepTypeRegistry1 = glue.getStepTypeRegistry();
+
+        // When
+        glue.addStepDefinition(new MockedStepDefinition("mock"));
+        glue.prepareGlue(LANGUAGE);
+        StepTypeRegistry stepTypeRegistry2 = glue.getStepTypeRegistry();
+
+        // Then
+        assertThat(stepTypeRegistry1!=stepTypeRegistry2, is(true));
+    }
+
+    @Test
+    void prepareGlue_cache_evicted_when_parameterType_added() {
+        // Given
+        glue.prepareGlue(LANGUAGE);
+        StepTypeRegistry stepTypeRegistry1 = glue.getStepTypeRegistry();
+
+        // When
+        glue.addParameterType(new MockedParameterTypeDefinition());
+        glue.prepareGlue(LANGUAGE);
+        StepTypeRegistry stepTypeRegistry2 = glue.getStepTypeRegistry();
+
+        // Then
+        assertThat(stepTypeRegistry1!=stepTypeRegistry2, is(true));
+    }
+
+    @Test
+    void prepareGlue_cache_evicted_when_dataTableType_added() {
+        // Given
+        glue.prepareGlue(LANGUAGE);
+        StepTypeRegistry stepTypeRegistry1 = glue.getStepTypeRegistry();
+
+        // When
+        glue.addDataTableType(new MockedDataTableTypeDefinition());
+        glue.prepareGlue(LANGUAGE);
+        StepTypeRegistry stepTypeRegistry2 = glue.getStepTypeRegistry();
+
+        // Then
+        assertThat(stepTypeRegistry1!=stepTypeRegistry2, is(true));
+    }
+
+    @Test
+    void prepareGlue_cache_evicted_when_docString_added() {
+        // Given
+        glue.prepareGlue(LANGUAGE);
+        StepTypeRegistry stepTypeRegistry1 = glue.getStepTypeRegistry();
+
+        // When
+        glue.addDocStringType(new MockedDocStringTypeDefinition());
+        glue.prepareGlue(LANGUAGE);
+        StepTypeRegistry stepTypeRegistry2 = glue.getStepTypeRegistry();
+
+        // Then
+        assertThat(stepTypeRegistry1!=stepTypeRegistry2, is(true));
+    }
+
     private static class MockedScenarioScopedStepDefinition extends StubStepDefinition implements ScenarioScoped {
 
         MockedScenarioScopedStepDefinition(String pattern, Type... types) {