Browse Source

wip

Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
tobiasKaminsky 4 years ago
parent
commit
50c7635869

+ 21 - 21
build.gradle

@@ -63,7 +63,7 @@ ext {
     markwonVersion =  "4.5.1"
     prismVersion = "2.0.0"
     butterknifeVersion = "10.2.3"
-    androidLibraryVersion = "master-SNAPSHOT"
+    androidLibraryVersion = "sharingPart2-SNAPSHOT"
     mockitoVersion = "3.5.10"
     espressoVersion = "3.3.0"
 
@@ -271,11 +271,11 @@ android {
 dependencies {
     // dependencies for app building
     implementation 'androidx.multidex:multidex:2.0.1'
-    implementation project('nextcloud-android-library')
-//    genericImplementation "com.github.nextcloud:android-library:$androidLibraryVersion"
-//    gplayImplementation "com.github.nextcloud:android-library:$androidLibraryVersion"
-//    versionDevImplementation "com.github.nextcloud:android-library:$androidLibraryVersion"
-//    qaImplementation "com.github.nextcloud:android-library:$androidLibraryVersion"
+//    implementation project('nextcloud-android-library')
+    genericImplementation "com.github.nextcloud:android-library:$androidLibraryVersion"
+    gplayImplementation "com.github.nextcloud:android-library:$androidLibraryVersion"
+    versionDevImplementation "com.github.nextcloud:android-library:$androidLibraryVersion"
+    qaImplementation "com.github.nextcloud:android-library:$androidLibraryVersion"
 
     kapt 'javax.xml.bind:jaxb-api:2.3.1'
     kapt 'org.glassfish.jaxb:jaxb-core:2.3.0.1'
@@ -434,21 +434,21 @@ android.applicationVariants.all { variant ->
     }
 }
 
-//tasks.register("combinedTestReport", JacocoReport) {
-//
-//    reports {
-//        xml.enabled true
-//        html.enabled true
-//        csv.enabled false
-//    }
-//
-//    additionalSourceDirs.setFrom files(subprojects.sourceSets.main.allSource.srcDirs)
-//    sourceDirectories.setFrom files(subprojects.sourceSets.main.allSource.srcDirs)
-//    classDirectories.setFrom files(subprojects.sourceSets.main.output)
-//    executionData.setFrom project.fileTree(dir: project.buildDir, includes: [
-//        'jacoco/testGplayDebugUnitTest.exec', 'outputs/code-coverage/connected/flavors/GPLAY/*coverage.ec'
-//    ])
-//}
+tasks.register("combinedTestReport", JacocoReport) {
+
+    reports {
+        xml.enabled true
+        html.enabled true
+        csv.enabled false
+    }
+
+    additionalSourceDirs.setFrom files(subprojects.sourceSets.main.allSource.srcDirs)
+    sourceDirectories.setFrom files(subprojects.sourceSets.main.allSource.srcDirs)
+    classDirectories.setFrom files(subprojects.sourceSets.main.output)
+    executionData.setFrom project.fileTree(dir: project.buildDir, includes: [
+        'jacoco/testGplayDebugUnitTest.exec', 'outputs/code-coverage/connected/flavors/GPLAY/*coverage.ec'
+    ])
+}
 
 task ktlint(type: JavaExec, group: "verification") {
     description = "Check Kotlin code style."

+ 2 - 2
settings.gradle

@@ -1,2 +1,2 @@
-//include ':'
-include ':nextcloud-android-library'
+include ':'
+//include ':nextcloud-android-library'

+ 8 - 7
src/androidTest/java/com/owncloud/android/ui/fragment/FileDetailSharingFragmentIT.kt

@@ -94,7 +94,7 @@ class FileDetailSharingFragmentIT : AbstractIT() {
             remoteId = 1
             shareType = ShareType.USER
             sharedWithDisplayName = "Admin"
-            permissions = OCShare.MAXIMUM_PERMISSIONS_FOR_FILE
+            permissions = MAXIMUM_PERMISSIONS_FOR_FILE
             userId = getUserId(user)
             activity.storageManager.saveShare(this)
         }
@@ -103,7 +103,7 @@ class FileDetailSharingFragmentIT : AbstractIT() {
             remoteId = 2
             shareType = ShareType.GROUP
             sharedWithDisplayName = "Group"
-            permissions = OCShare.MAXIMUM_PERMISSIONS_FOR_FILE
+            permissions = MAXIMUM_PERMISSIONS_FOR_FILE
             userId = getUserId(user)
             activity.storageManager.saveShare(this)
         }
@@ -143,7 +143,7 @@ class FileDetailSharingFragmentIT : AbstractIT() {
             remoteId = 7
             shareType = ShareType.CIRCLE
             sharedWithDisplayName = "Private circle"
-            permissions = OCShare.SHARE_PERMISSION_FLAG
+            permissions = SHARE_PERMISSION_FLAG
             userId = getUserId(user)
             activity.storageManager.saveShare(this)
         }
@@ -152,7 +152,7 @@ class FileDetailSharingFragmentIT : AbstractIT() {
             remoteId = 8
             shareType = ShareType.ROOM
             sharedWithDisplayName = "Meeting"
-            permissions = OCShare.SHARE_PERMISSION_FLAG
+            permissions = SHARE_PERMISSION_FLAG
             userId = getUserId(user)
             activity.storageManager.saveShare(this)
         }
@@ -205,7 +205,7 @@ class FileDetailSharingFragmentIT : AbstractIT() {
         assertFalse(popup.menu.findItem(R.id.link_share_file_drop).isChecked)
 
         // upload and editing
-        publicShare.permissions = OCShare.MAXIMUM_PERMISSIONS_FOR_FOLDER
+        publicShare.permissions = MAXIMUM_PERMISSIONS_FOR_FOLDER
         sut.prepareLinkOptionsMenu(popup.menu, publicShare)
         assertFalse(popup.menu.findItem(R.id.link_share_read_only).isChecked)
         assertTrue(popup.menu.findItem(R.id.link_share_allow_upload_and_editing).isChecked)
@@ -231,6 +231,7 @@ class FileDetailSharingFragmentIT : AbstractIT() {
 
         // hide download
         publicShare.isHideFileDownload = true
+        publicShare.permissions = MAXIMUM_PERMISSIONS_FOR_FOLDER
         sut.prepareLinkOptionsMenu(popup.menu, publicShare)
         assertTrue(popup.menu.findItem(R.id.action_hide_file_download).isChecked)
 
@@ -250,7 +251,7 @@ class FileDetailSharingFragmentIT : AbstractIT() {
 
         // file
         publicShare.isFolder = false
-        publicShare.permissions = OCShare.READ_PERMISSION_FLAG
+        publicShare.permissions = READ_PERMISSION_FLAG
         sut.prepareLinkOptionsMenu(popup.menu, publicShare)
         // check if items are visible
         assertTrue(popup.menu.findItem(R.id.action_hide_file_download).isVisible)
@@ -376,7 +377,7 @@ class FileDetailSharingFragmentIT : AbstractIT() {
             targetContext.getString(R.string.share_no_expiration_date_label))
 
         publicShare.isFolder = false
-        publicShare.permissions = OCShare.READ_PERMISSION_FLAG
+        publicShare.permissions = READ_PERMISSION_FLAG
         sut.prepareLinkOptionsMenu(popup.menu, publicShare)
 
         // allow editing