valitydev/file-storage
14
0
Fork 0
mirror of https://github.com/valitydev/file-storage.git synced 2024-11-06 00:35:22 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

BJ-323: fix after review

Browse Source
This commit is contained in:
a.karlov 2018-12-12 14:58:15 +03:00
parent 441979f439
commit 113769201c
2 changed files with 17 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/main/java/com/rbkmoney/file/storage/util/CheckerUtil.java
View File

@ -7,7 +7,7 @@ import java.util.regex.Pattern;
public class CheckerUtil {
private static final Pattern FILE_NAME_PATTERN = Pattern.compile("^[a-zA-Z0-9 ,-_.]*$");
private static final Pattern FILE_NAME_PATTERN = Pattern.compile("^[а-яА-Яa-zA-Z0-9 ,-_.]*$");
public static void checkString(String string, String exMessage) throws IllegalArgumentException {
if (Strings.isNullOrEmpty(string)) {

31
src/test/java/com/rbkmoney/file/storage/FileStorageTest.java
View File

@ -19,12 +19,13 @@ import java.util.HashMap;
import java.util.Map;
import static org.junit.Assert.assertEquals;
import static org.junit.Assert.assertNotEquals;
import static org.junit.jupiter.api.Assertions.assertThrows;
// все тесты в 1 классе , чтобы сэкономить время на поднятии тест контейнера
public class FileStorageTest extends AbstractIntegrationTest {
private static final String TEST_DATA = "test";
@Test
public void uploadAndDownloadFileFromStorageTest() throws IOException, TException {
Path testFile = Files.createTempFile("", "test_file");
@ -46,10 +47,11 @@ public class FileStorageTest extends AbstractIntegrationTest {
// чтение записанного файла из хранилища
Files.copy(inputStream, testActualFile, StandardCopyOption.REPLACE_EXISTING);
// testFile пустой, testActualFile содержит данные из хранилища
assertNotEquals(Files.readAllLines(testFile), Files.readAllLines(testActualFile));
// testFile пустой
assertEquals(Files.readAllLines(testFile), Collections.emptyList());
Files.write(testFile, "test".getBytes());
// запись тестовых данных в пустой testFile
Files.write(testFile, TEST_DATA.getBytes());
assertEquals(Files.readAllLines(testFile), Files.readAllLines(testActualFile));
} finally {
Files.deleteIfExists(testFile);
@ -58,7 +60,7 @@ public class FileStorageTest extends AbstractIntegrationTest {
}
@Test
public void uploadUrlConnectionAccessTest() throws IOException, TException {
public void accessErrorToMethodsTest() throws IOException, TException {
// создание файла с доступом к файлу на день
String expirationTime = generateCurrentTimePlusDay().toString();
NewFileResult fileResult = client.createNewFile("test_file", Collections.emptyMap(), expirationTime);
@ -68,6 +70,13 @@ public class FileStorageTest extends AbstractIntegrationTest {
// ошибка доступа - файла не существует, тк не было upload
assertThrows(FileNotFound.class, () -> client.generateDownloadUrl(fileDataId, expirationTime));
assertThrows(FileNotFound.class, () -> client.getFileData(fileDataId));
}
@Test
public void uploadUrlConnectionAccessTest() throws IOException, TException {
// создание файла с доступом к файлу на день
String expirationTime = generateCurrentTimePlusDay().toString();
NewFileResult fileResult = client.createNewFile("test_file", Collections.emptyMap(), expirationTime);
URL uploadUrl = new URL(fileResult.getUploadUrl());
@ -88,10 +97,6 @@ public class FileStorageTest extends AbstractIntegrationTest {
String fileDataId = fileResult.getFileData().getFiledataId();
// ошибка доступа - файла не существует, тк не было upload
assertThrows(FileNotFound.class, () -> client.generateDownloadUrl(fileDataId, expirationTime));
assertThrows(FileNotFound.class, () -> client.getFileData(fileDataId));
// upload тестовых данных в хранилище
uploadTestData(fileResult);
@ -113,10 +118,6 @@ public class FileStorageTest extends AbstractIntegrationTest {
String validFileDataId = validFileResult.getFileData().getFiledataId();
// ошибка доступа - файла не существует, тк не было upload
assertThrows(FileNotFound.class, () -> client.generateDownloadUrl(validFileDataId, expirationTime));
assertThrows(FileNotFound.class, () -> client.getFileData(validFileDataId));
// задержка перед upload для теста expiration
Thread.sleep(1000);
@ -157,7 +158,7 @@ public class FileStorageTest extends AbstractIntegrationTest {
put("key2", Value.i(1));
put("key3", Value.flt(1));
put("key4", Value.arr(new ArrayList<>()));
put("key5", Value.str("test"));
put("key5", Value.str(TEST_DATA));
put("key6", Value.bin(new byte[]{}));
}};
NewFileResult fileResult = client.createNewFile(fileName, metadata, expirationTime);
@ -177,7 +178,7 @@ public class FileStorageTest extends AbstractIntegrationTest {
HttpURLConnection uploadUrlConnection = getHttpURLConnection(uploadUrl, true, "PUT");
OutputStreamWriter out = new OutputStreamWriter(uploadUrlConnection.getOutputStream());
out.write("test");
out.write(TEST_DATA);
out.close();
// чтобы завершить загрузку вызываем getResponseCode