lesson4

устранение замечаний AbstractArrayStorageTest - int size = storage.size(); int sizeBefore = storage.size(); именуй переменные одинаково - в методе getAll() сравнивай массива полностью за раз, а не по элементам - зачем в методе sizeOverLoaded рефлексия? Просто сделай размер массива protected - в методе updateExist, как и в других классах используй uuid, а не метод getAll()[0]; - r_before/r_after переменные в Java так не именуются. - idea предлагает заменить assertFalse на assertEquals (edited)

Author: AlbertoJava

src/com/urise/webapp/storage/AbstractArrayStorage.java

@@ -8,7 +8,7 @@
 importjava.util.Arrays;
 
 publicabstractclassAbstractArrayStorageimplementsStorage{
-privatestaticfinalintMAX_SIZE = 10_0000;
+protectedstaticfinalintMAX_SIZE = 10_0000;
 protectedResume[] storage = newResume[MAX_SIZE];
 protectedintresumeCounter = 0;
 

test/com/urise/webapp/storage/AbstractArrayStorageTest.java

@@ -8,7 +8,7 @@
 importorg.junit.Before;
 importorg.junit.Test;
 
-importjava.lang.reflect.Field;
+importstaticcom.urise.webapp.storage.AbstractArrayStorage.MAX_SIZE;
 
 publicabstractclassAbstractArrayStorageTest{
 privatestaticfinalStringUUID_1 = "UUID_1";
@@ -47,9 +47,9 @@ public void saveExist(){
 
 @Test
 publicvoidsaveNotExist(){
-intsize = storage.size();
+intsizeBefore = storage.size();
 storage.save(R_4);
-Assert.assertEquals(storage.size(), size + 1);
+Assert.assertEquals(storage.size(), sizeBefore + 1);
 Assert.assertEquals(R_4, storage.get(UUID_4));
  }
 
@@ -81,11 +81,7 @@ public void deleteNotExist(){
 
 @Test
 publicvoidgetAll(){
-Resume[] arrResume = storage.getAll();
-Assert.assertEquals(3, arrResume.length);
-Assert.assertEquals(R_1, arrResume[0]);
-Assert.assertEquals(R_2, arrResume[1]);
-Assert.assertEquals(R_3, arrResume[2]);
+Assert.assertArrayEquals(storage.getAll(), newResume[]{R_1, R_2, R_3});
  }
 
 @Test
@@ -94,10 +90,7 @@ public void size(){
  }
 
 @Test(expected = StorageException.class)
-publicvoidsizeOverLoaded() throwsNoSuchFieldException, IllegalAccessException{
-Fieldarr_size = storage.getClass().getSuperclass().getDeclaredField("MAX_SIZE");
-arr_size.setAccessible(true);
-IntegerMAX_SIZE = (Integer) arr_size.get(storage);
+publicvoidsizeOverLoaded(){
 try{
 for (inti = 4; i <= MAX_SIZE; i++){
 storage.save(newResume());
@@ -110,16 +103,15 @@ public void sizeOverLoaded() throws NoSuchFieldException, IllegalAccessException
 
 @Test
 publicvoidupdateExist(){
-Resumer_before = storage.getAll()[0];
-Resumer_after = newResume(storage.getAll()[0].getUuid());
-storage.update(r_after);
-Assert.assertFalse(storage.getAll()[0] == r_before);
+ResumerBefore = storage.get(UUID_1);
+ResumerAfter = newResume(UUID_1);
+storage.update(rAfter);
+Assert.assertNotSame(storage.get(UUID_1), rBefore);
  }
 
 @Test(expected = NotExistStorageException.class)
 publicvoidupdateNotExist(){
-Resumer_after = newResume("test");
-storage.update(r_after);
+ResumerAfter = newResume("test");
+storage.update(rAfter);
  }
-
 }