Fixed comments

Author: anchouls

ExtractingCode/WhatIsExtractVariableRefactoring/ExtractMagicConstantsPractice/test/ExtractMagicNumbersTest.java

@@ -11,6 +11,8 @@
 public class ExtractMagicNumbersTest extends BaseIjTestClass {
 
     private static String sourceText;
+    private static final String SPEED_OF_LIGHT = "299792458.0";
+    private static final String PLANCK_CONSTANT = "6.62607015e-34";
 
     @BeforeAll
     static void beforeAll() throws IOException {
@@ -24,12 +26,10 @@ static void beforeAll() throws IOException {
     public void testExtractedConstant() throws Exception {
         setUp();
         myFixture.configureByText("Task.java", sourceText);
-        String speedOfLight = "299792458.0";
-        Assertions.assertTrue(hasConstantWithGivenValue(speedOfLight),
-                "Please, create constant values for " + speedOfLight);
-        String planckConstant = "6.62607015e-34";
-        Assertions.assertTrue(hasConstantWithGivenValue(planckConstant),
-                "Please, create constant values for " + planckConstant);
+        Assertions.assertTrue(hasConstantWithGivenValue(SPEED_OF_LIGHT),
+                "Please, create constant values for " + SPEED_OF_LIGHT);
+        Assertions.assertTrue(hasConstantWithGivenValue(PLANCK_CONSTANT),
+                "Please, create constant values for " + PLANCK_CONSTANT);
         String propertyName = "waveLength";
         Assertions.assertTrue(hasLocalVariable(propertyName),
                 "Please, create property for " + propertyName);

MovingCode/WhatArePullUpAndPushDownRefactorings/ApplyPullUpRefactoringPractice/task-info.yaml

@@ -1,10 +1,10 @@
 type: edu
 custom_name: Apply Pull Up refactoring
 files:
-  - name: src/main/java/jetbrains/refactoring/course/moving/Animal.java
-    visible: true
   - name: src/main/java/jetbrains/refactoring/course/moving/Cat.java
     visible: true
+  - name: src/main/java/jetbrains/refactoring/course/moving/Animal.java
+    visible: true
   - name: src/main/java/jetbrains/refactoring/course/moving/Dog.java
     visible: true
   - name: test/PullUpTest.java

MovingCode/WhatIsMoveMethodRefactoring/FindMoreAppropriateClassesForMethodsPractice/task-info.yaml

@@ -1,11 +1,11 @@
 type: edu
 custom_name: Find more appropriate classes for methods
 files:
+  - name: src/main/java/jetbrains/refactoring/course/moving/driver/Driver.java
+    visible: true
   - name: src/main/java/jetbrains/refactoring/course/moving/Main.java
     visible: true
   - name: src/main/java/jetbrains/refactoring/course/moving/car/Car.java
     visible: true
-  - name: src/main/java/jetbrains/refactoring/course/moving/driver/Driver.java
-    visible: true
   - name: test/MovingMethodsTest.java
     visible: false

MovingCode/WhatIsMoveMethodRefactoring/FindMoreAppropriateFilesForClassesPractice/test/MovingTest.java

@@ -29,14 +29,14 @@ public void testCarClassMoved() throws Exception {
         setUp();
         myFixture.configureByText("Car.java", carText);
         Assertions.assertTrue(hasClass("Car"),
-                "Please, move the Car class to a separate file");
+                "Please, move the Car class to a separate file in the car package");
     }
 
     @Test
     public void testDriverClassMoved() throws Exception {
         setUp();
         myFixture.configureByText("Driver.java", driverText);
         Assertions.assertTrue(hasClass("Driver"),
-                "Please, move the Driver class to a separate file");
+                "Please, move the Driver class to a separate file in the driver package");
     }
 }