Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(255)

Unified Diff: pkg/analysis_server/test/integration/integration_test_methods.dart

Issue 576473003: Split refactoring problems into init, options and final. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 6 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: pkg/analysis_server/test/integration/integration_test_methods.dart
diff --git a/pkg/analysis_server/test/integration/integration_test_methods.dart b/pkg/analysis_server/test/integration/integration_test_methods.dart
index 8e0472accdd22d57bdc41d826b9ec07c9f6be54d..bfb174ebec60c0dd1e4aec0229f14aa16e0155dc 100644
--- a/pkg/analysis_server/test/integration/integration_test_methods.dart
+++ b/pkg/analysis_server/test/integration/integration_test_methods.dart
@@ -1137,10 +1137,23 @@ abstract class IntegrationTestMixin {
*
* Returns
*
- * problems ( List<RefactoringProblem> )
+ * initialProblems ( List<RefactoringProblem> )
*
- * The status of the refactoring. The array will be empty if there are no
- * known problems.
+ * The initial status of the refactoring, i.e. problems related to the
+ * context in which the refactoring is requested. The array will be empty
+ * if there are no known problems.
+ *
+ * optionsProblems ( List<RefactoringProblem> )
+ *
+ * The options validation status, i.e. problems in the given options, such
+ * as light-weight validation of a new name, flags compatibility, etc. The
+ * array will be empty if there are no known problems.
+ *
+ * finalProblems ( List<RefactoringProblem> )
+ *
+ * The final status of the refactoring, i.e. problems identified in the
+ * result of a full, potentially expensive validation and / or change
+ * creation. The array will be empty if there are no known problems.
*
* feedback ( optional RefactoringFeedback )
*
« no previous file with comments | « pkg/analysis_server/test/edit/refactoring_test.dart ('k') | pkg/analysis_server/test/integration/protocol_matchers.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698