Prechádzať zdrojové kódy

detekt

Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
tobiasKaminsky 4 rokov pred
rodič
commit
1a32e9b268

+ 2 - 2
src/androidTest/java/com/owncloud/android/ui/fragment/FileDetailFragmentStaticServerIT.kt

@@ -58,7 +58,7 @@ class FileDetailFragmentStaticServerIT : AbstractIT() {
 
 
     @Test
     @Test
     @ScreenshotTest
     @ScreenshotTest
-    fun showDetails_Activities() {
+    fun showDetailsActivities() {
         val sut = testActivityRule.launchActivity(null)
         val sut = testActivityRule.launchActivity(null)
         sut.addFragment(FileDetailFragment.newInstance(file, user, 0))
         sut.addFragment(FileDetailFragment.newInstance(file, user, 0))
 
 
@@ -68,7 +68,7 @@ class FileDetailFragmentStaticServerIT : AbstractIT() {
 
 
     @Test
     @Test
     @ScreenshotTest
     @ScreenshotTest
-    fun showDetails_Sharing() {
+    fun showDetailsSharing() {
         val sut = testActivityRule.launchActivity(null)
         val sut = testActivityRule.launchActivity(null)
         sut.addFragment(FileDetailFragment.newInstance(file, user, 1))
         sut.addFragment(FileDetailFragment.newInstance(file, user, 1))
 
 

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

@@ -75,13 +75,13 @@ class FileDetailSharingFragmentIT : AbstractIT() {
 
 
     @Test
     @Test
     @ScreenshotTest
     @ScreenshotTest
-    fun listShares_file_none() {
+    fun listSharesFileNone() {
         show(file)
         show(file)
     }
     }
 
 
     @Test
     @Test
     @ScreenshotTest
     @ScreenshotTest
-    fun listShares_file_resharing_not_allowed() {
+    fun listSharesFileResharingNotAllowed() {
         file.permissions = ""
         file.permissions = ""
 
 
         show(file)
         show(file)
@@ -89,7 +89,8 @@ class FileDetailSharingFragmentIT : AbstractIT() {
 
 
     @Test
     @Test
     @ScreenshotTest
     @ScreenshotTest
-    fun listShares_file_allShareTypes() {
+    @Suppress("MagicNumber")
+    fun listSharesFileAllShareTypes() {
         OCShare(file.decryptedRemotePath).apply {
         OCShare(file.decryptedRemotePath).apply {
             remoteId = 1
             remoteId = 1
             shareType = ShareType.USER
             shareType = ShareType.USER
@@ -171,8 +172,9 @@ class FileDetailSharingFragmentIT : AbstractIT() {
     }
     }
 
 
     @Test
     @Test
+    @Suppress("MagicNumber")
     // public link and email are handled the same way
     // public link and email are handled the same way
-    fun publicLink_optionMenu_Folder() {
+    fun publicLinkOptionMenuFolder() {
         val sut = FileDetailSharingFragment.newInstance(file, user)
         val sut = FileDetailSharingFragment.newInstance(file, user)
         activity.addFragment(sut)
         activity.addFragment(sut)
         shortSleep()
         shortSleep()
@@ -312,8 +314,9 @@ class FileDetailSharingFragmentIT : AbstractIT() {
     }
     }
 
 
     @Test
     @Test
+    @Suppress("MagicNumber")
     // public link and email are handled the same way
     // public link and email are handled the same way
-    fun publicLink_optionMenu_File() {
+    fun publicLinkOptionMenuFile() {
         val sut = FileDetailSharingFragment.newInstance(file, user)
         val sut = FileDetailSharingFragment.newInstance(file, user)
         activity.addFragment(sut)
         activity.addFragment(sut)
         shortSleep()
         shortSleep()
@@ -390,12 +393,13 @@ class FileDetailSharingFragmentIT : AbstractIT() {
     }
     }
 
 
     @Test
     @Test
+    @Suppress("MagicNumber")
     // also applies for
     // also applies for
     // group
     // group
     // conversation
     // conversation
     // circle
     // circle
     // federated share
     // federated share
-    fun user_optionMenu_File() {
+    fun userOptionMenuFile() {
         val sut = FileDetailSharingFragment.newInstance(file, user)
         val sut = FileDetailSharingFragment.newInstance(file, user)
         activity.addFragment(sut)
         activity.addFragment(sut)
         shortSleep()
         shortSleep()
@@ -447,12 +451,13 @@ class FileDetailSharingFragmentIT : AbstractIT() {
     }
     }
 
 
     @Test
     @Test
+    @Suppress("MagicNumber")
     // also applies for
     // also applies for
     // group
     // group
     // conversation
     // conversation
     // circle
     // circle
     // federated share
     // federated share
-    fun user_optionMenu_Folder() {
+    fun userOptionMenuFolder() {
         val sut = FileDetailSharingFragment.newInstance(file, user)
         val sut = FileDetailSharingFragment.newInstance(file, user)
         activity.addFragment(sut)
         activity.addFragment(sut)
         shortSleep()
         shortSleep()
@@ -529,6 +534,7 @@ class FileDetailSharingFragmentIT : AbstractIT() {
     }
     }
 
 
     @Test
     @Test
+    @Suppress("MagicNumber")
     fun testReadOnlySharePermissions() {
     fun testReadOnlySharePermissions() {
         val sut = FileDetailSharingFragment()
         val sut = FileDetailSharingFragment()
 
 
@@ -560,6 +566,7 @@ class FileDetailSharingFragmentIT : AbstractIT() {
     }
     }
 
 
     @Test
     @Test
+    @Suppress("MagicNumber")
     fun testFileDropSharePermissions() {
     fun testFileDropSharePermissions() {
         val sut = FileDetailSharingFragment()
         val sut = FileDetailSharingFragment()
 
 

+ 1 - 0
src/androidTest/java/com/owncloud/android/ui/fragment/OCFileListFragmentStaticServerIT.kt

@@ -42,6 +42,7 @@ class OCFileListFragmentStaticServerIT : AbstractIT() {
 
 
     @Test
     @Test
     @ScreenshotTest
     @ScreenshotTest
+    @Suppress("MagicNumber")
     fun showFiles() {
     fun showFiles() {
         val sut = testActivityRule.launchActivity(null)
         val sut = testActivityRule.launchActivity(null)