-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Test that move or copy files starts in the file's parent folder #16262
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
|
APK file: https://www.kaminsky.me/nc-dev/android-artifacts/16262.apk |
|
blue-Light-Screenshot test failed, but no output was generated. Maybe a preliminary stage failed. |
| activity.supportFragmentManager.executePendingTransactions() | ||
|
|
||
| fragment.listDirectory(testFolder, false) | ||
| activity.supportFragmentManager.executePendingTransactions() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The first activity.supportFragmentManager.executePendingTransactions() is needed after adding fragment other two is not needed because we are not showing another fragment via listDirectory or onFileActionChosen(R.id.action_move_or_copy, setOf(testFile))
When I tested with only first one it worked as well.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@alperozturk96 thanks, I removed them.
This tests the changes of #15925, where the old behavior of moving up to the root folder was changed to stay in the parent folder of the file(s) to move/copy. The new behavior is in sync with iOS and the Web UI, hence this test ensures it doesn't break Signed-off-by: Philipp Hasper <[email protected]>
9445d8c to
54e9c43
Compare
Signed-off-by: Philipp Hasper <[email protected]>

Original PR: #16188
This tests the changes of #15925, where the old behavior of moving up to the root folder was changed to stay in the parent folder of the file(s) to move/copy.
The new behavior is in sync with iOS and the Web UI, hence this test ensures it doesn't break