SONARPY-2097: Fix incomplete quickfix message (#2014)

Author: joke1196

python-checks/src/main/java/org/sonar/python/checks/SklearnCachedPipelineDontAccessTransformersCheck.java

@@ -146,7 +146,7 @@ private static Stream<Name> extractFromPipeline(Expression stepArgumentExpressio
 
   private static Optional<PythonQuickFix> getQuickFix(Name pipelineBindingVariable, Tree name, QualifiedExpression qualifiedExpression, Map<Name, String> nameToStepName) {
     return Optional.ofNullable(nameToStepName.get(name))
-      .map(stepName -> PythonQuickFix.newQuickFix("Access the property through the ")
+      .map(stepName -> PythonQuickFix.newQuickFix("Replace the direct access to the transformer with an access to the `named_steps` attribute of the pipeline.")
         .addTextEdit(TextEditUtils.replace(qualifiedExpression.qualifier(), String.format("%s.named_steps[\"%s\"]", pipelineBindingVariable.name(), stepName)))
         .build());
   }

python-checks/src/test/java/org/sonar/python/checks/SklearnCachedPipelineDontAccessTransformersCheckTest.java

@@ -58,5 +58,20 @@ void test_quickfix1(){
         print(pipeline.named_steps["scaler"].center_)
         """
     );
+    PythonQuickFixVerifier.verifyQuickFixMessages(
+      new SklearnCachedPipelineDontAccessTransformersCheck(),
+      """
+        from sklearn.pipeline import Pipeline
+        scaler = RobustScaler()
+        knn = KNeighborsRegressor(n_neighbors=5)
+            
+        pipeline = Pipeline([
+            ('scaler', scaler),
+            ('knn', knn),
+        ], memory="cache")
+        print(scaler.center_)
+        """,
+      "Replace the direct access to the transformer with an access to the `named_steps` attribute of the pipeline."
+    );
   }
-}
+}