Skip to content
This repository was archived by the owner on Apr 5, 2024. It is now read-only.

Commit 516474c

Browse files
authored
Fixing FSController (#4)
* FileSystemRestController.java offers getContentsOfFolder with Path instead of id. * I think -P is setting maven profiles not spring boot profiles. * fixed naming of feature_branch_tests.yml
1 parent 7173480 commit 516474c

File tree

7 files changed

+12
-12
lines changed

7 files changed

+12
-12
lines changed

.github/workflows/feature_branch_tests.yml renamed to .github/workflows/featureRelease.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ jobs:
3434
run: bash <(curl -s https://codecov.io/bash) -Z
3535
-
3636
name: Build Docker Image
37-
run: mvn spring-boot:build-image -f pom.xml -P prod
37+
run: mvn spring-boot:build-image -f pom.xml
3838
-
3939
name: Login to DockerHub
4040
uses: docker/login-action@v1

.github/workflows/latestRelease.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ jobs:
3232
run: mvn clean test -f pom.xml
3333
-
3434
name: Build Docker Image
35-
run: mvn spring-boot:build-image -f pom.xml -P prod
35+
run: mvn spring-boot:build-image -f pom.xml
3636
-
3737
name: Upload CodeCov.
3838
run: bash <(curl -s https://codecov.io/bash) -Z

.github/workflows/stableRelease.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ jobs:
2626
run: mvn clean test -f pom.xml
2727
-
2828
name: Build Docker Image
29-
run: mvn spring-boot:build-image -f pom.xml -P prod
29+
run: mvn spring-boot:build-image -f pom.xml
3030
-
3131
name: Login to DockerHub
3232
uses: docker/login-action@v1

src/main/java/de/filefighter/rest/domain/filesystem/rest/FileSystemRestController.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -25,14 +25,14 @@ public FileSystemRestController(FileSystemRestServiceInterface fileSystemRestSer
2525
this.fileSystemRestService = fileSystemRestService;
2626
}
2727

28-
@GetMapping(FS_BASE_URI + "{fsItemId}/contents")
28+
@GetMapping(FS_BASE_URI + "contents")
2929
public EntityModel<FolderContents> getContentsOfFolder(
30-
@PathVariable long fsItemId,
30+
@RequestHeader(value = "X-FF-PATH", defaultValue = "/") String path,
3131
@RequestHeader(value = "Authorization", defaultValue = AUTHORIZATION_BEARER_PREFIX + "token") String accessToken
3232
) {
3333

34-
LOG.info("Requested Folder contents of id {}.", fsItemId);
35-
return fileSystemRestService.getContentsOfFolderByIdAndAccessToken(fsItemId, accessToken);
34+
LOG.info("Requested Folder contents of folder with path {}.", path);
35+
return fileSystemRestService.getContentsOfFolderByIdAndAccessToken(path, accessToken);
3636
}
3737

3838
@GetMapping(FS_BASE_URI + "{fsItemId}/info")

src/main/java/de/filefighter/rest/domain/filesystem/rest/FileSystemRestService.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111
public class FileSystemRestService implements FileSystemRestServiceInterface {
1212

1313
@Override
14-
public EntityModel<FolderContents> getContentsOfFolderByIdAndAccessToken(long fsItemId, String accessToken) {
14+
public EntityModel<FolderContents> getContentsOfFolderByIdAndAccessToken(String path, String accessToken) {
1515
return null;
1616
}
1717

src/main/java/de/filefighter/rest/domain/filesystem/rest/FileSystemRestServiceInterface.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
import org.springframework.hateoas.EntityModel;
88

99
public interface FileSystemRestServiceInterface {
10-
EntityModel<FolderContents> getContentsOfFolderByIdAndAccessToken(long fsItemId, String accessToken);
10+
EntityModel<FolderContents> getContentsOfFolderByIdAndAccessToken(String path, String accessToken);
1111
EntityModel<FileSystemItem> getInfoAboutFileOrFolderByIdAndAccessToken(long fsItemId, String accessToken);
1212
EntityModel<FileSystemItem> findFileOrFolderByNameAndAccessToken(String name, String accessToken);
1313
EntityModel<FileSystemItem> uploadFileSystemItemWithAccessToken(FileSystemItemUpdate fileSystemItemUpdate, String accessToken);

src/test/java/de/filefighter/rest/domain/filesystem/rest/FileSystemRestControllerUnitTest.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -28,12 +28,12 @@ void getContentsOfFolder() {
2828
.files(new File[]{dummyFile})
2929
.folders(new Folder[]{dummyFolder}).create());
3030

31-
long id = 420;
31+
String path= "/root/data.txt";
3232
String token = "token";
3333

34-
when(fileSystemRestServiceMock.getContentsOfFolderByIdAndAccessToken(id, token)).thenReturn(expectedModel);
34+
when(fileSystemRestServiceMock.getContentsOfFolderByIdAndAccessToken(path, token)).thenReturn(expectedModel);
3535

36-
EntityModel<FolderContents> actualModel = fileSystemRestController.getContentsOfFolder(id, token);
36+
EntityModel<FolderContents> actualModel = fileSystemRestController.getContentsOfFolder(path, token);
3737
assertEquals(expectedModel, actualModel);
3838
}
3939

0 commit comments

Comments
 (0)