moveUp|Down should return false on failure instead of throw

- Refactoring, remove usage of extract()
- Test cases updated
update of test cases

use assertTrue|False instead of `assertEquals`
revert the throw thing

we keep refactoring changes only
Merge pull request #3111 from quickapps/3.0-tree-behavior

moveUp|Down should return false on failure instead of throw