Fix functional tests failed (android, push_file) (appium#375)

* Fix: test_push_file

* Move remove_fs tests

* Move teardown process

* Delete selendroid test

* tweak

* Update along to review comments

* Replace double quote with single quote under android dir

* Remove creating tmp file

* tweak

Author: ki4070ma

test/functional/android/appium_tests.py

@@ -13,11 +13,8 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-import base64
 import unittest
-from zipfile import ZipFile
 import os
-import random
 from time import sleep
 from dateutil.parser import parse
 
@@ -42,10 +39,6 @@ def setUp(self):
     def tearDown(self):
         self.driver.quit()
 
-        # remove zipped file from `test_pull_folder`
-        if hasattr(self, 'zipfilename') and os.path.isfile(self.zipfilename):
-            os.remove(self.zipfilename)
-
     def test_screen_record(self):
         self.driver.start_recording_screen(timeLimit=10, forcedRestart=True)
         sleep(10)
@@ -99,35 +92,6 @@ def test_current_package(self):
         package = self.driver.current_package
         self.assertEqual('com.example.android.apis', package)
 
-    def test_push_pull_file(self):
-        path = '/data/local/tmp/test_push_file.txt'
-        data = b'This is the contents of the file to push to the device.'
-
-        self.driver.push_file(path, base64.b64encode(data).decode('utf-8'))
-        data_ret = base64.b64decode(self.driver.pull_file(path))
-
-        self.assertEqual(data, data_ret)
-
-    def test_pull_folder(self):
-        string_data = b'random string data %d' % random.randint(0, 1000)
-        path = '/data/local/tmp'
-
-        self.driver.push_file(path + '/1.txt', base64.b64encode(string_data).decode('utf-8'))
-        self.driver.push_file(path + '/2.txt', base64.b64encode(string_data).decode('utf-8'))
-
-        folder = self.driver.pull_folder(path)
-
-        # python doesn't have any functionality for unzipping streams
-        # save temporary file, which will be deleted in `tearDown`
-        self.zipfilename = 'folder_%d.zip' % random.randint(0, 1000000)
-        with open(self.zipfilename, "wb") as fw:
-            fw.write(base64.b64decode(folder))
-
-        with ZipFile(self.zipfilename, 'r') as myzip:
-            # should find these. otherwise it will raise a `KeyError`
-            myzip.read('1.txt')
-            myzip.read('2.txt')
-
     def test_background_app(self):
         self.driver.background_app(1)
         sleep(3)
@@ -257,6 +221,6 @@ def test_device_time(self):
         parse(date_time)
 
 
-if __name__ == "__main__":
+if __name__ == '__main__':
     suite = unittest.TestLoader().loadTestsFromTestCase(AppiumTests)
     unittest.TextTestRunner(verbosity=2).run(suite)

test/functional/android/chrome_tests.py

@@ -37,6 +37,6 @@ def test_find_single_element(self):
         self.assertTrue('I am some other page content' in self.driver.page_source)
 
 
-if __name__ == "__main__":
+if __name__ == '__main__':
     suite = unittest.TestLoader().loadTestsFromTestCase(ChromeTests)
     unittest.TextTestRunner(verbosity=2).run(suite)

test/functional/android/context_switching_tests.py

@@ -56,6 +56,6 @@ def _enter_webview(self):
         self.driver.switch_to.context('WEBVIEW')
 
 
-if __name__ == "__main__":
+if __name__ == '__main__':
     suite = unittest.TestLoader().loadTestsFromTestCase(ContextSwitchingTests)
     unittest.TextTestRunner(verbosity=2).run(suite)

test/functional/android/find_by_accessibility_id_tests.py

@@ -51,6 +51,6 @@ def test_element_find_multiple_elements(self):
         self.assertIsInstance(sub_els, list)
 
 
-if __name__ == "__main__":
+if __name__ == '__main__':
     suite = unittest.TestLoader().loadTestsFromTestCase(FindByAccessibilityIDTests)
     unittest.TextTestRunner(verbosity=2).run(suite)

test/functional/android/find_by_image_tests.py

@@ -83,6 +83,6 @@ def test_find_throws_no_such_element(self):
             self.driver.find_element_by_image(image_path)
 
 
-if __name__ == "__main__":
+if __name__ == '__main__':
     suite = unittest.TestLoader().loadTestsFromTestCase(FindByImageTests)
     unittest.TextTestRunner(verbosity=2).run(suite)

test/functional/android/find_by_uiautomator_tests.py

@@ -52,6 +52,6 @@ def test_scroll_into_view(self):
         el.click()
 
 
-if __name__ == "__main__":
+if __name__ == '__main__':
     suite = unittest.TestLoader().loadTestsFromTestCase(FindByUIAutomatorTests)
     unittest.TextTestRunner(verbosity=2).run(suite)

test/functional/android/finger_print_tests.py

@@ -32,6 +32,6 @@ def test_finger_print(self):
         self.assertEqual(None, result)
 
 
-if __name__ == "__main__":
+if __name__ == '__main__':
     suite = unittest.TestLoader().loadTestsFromTestCase(FingerPrintTests)
     unittest.TextTestRunner(verbosity=2).run(suite)

test/functional/android/ime_tests.py

@@ -65,6 +65,6 @@ def test_deactivate_ime_engine(self):
         self.assertNotEqual(self.driver.active_ime_engine, engines[-1])
 
 
-if __name__ == "__main__":
+if __name__ == '__main__':
     suite = unittest.TestLoader().loadTestsFromTestCase(IMETests)
     unittest.TextTestRunner(verbosity=2).run(suite)

test/functional/android/multi_action_tests.py

@@ -131,6 +131,6 @@ def test_driver_multi_tap(self):
         sleep(10)
 
 
-if __name__ == "__main__":
+if __name__ == '__main__':
     suite = unittest.TestLoader().loadTestsFromTestCase(MultiActionTests)
     unittest.TextTestRunner(verbosity=2).run(suite)

test/functional/android/network_connection_tests.py

@@ -43,6 +43,6 @@ def test_set_network_connection(self):
         self.assertEqual(nc, ConnectionType.DATA_ONLY)
 
 
-if __name__ == "__main__":
+if __name__ == '__main__':
     suite = unittest.TestLoader().loadTestsFromTestCase(NetworkConnectionTests)
     unittest.TextTestRunner(verbosity=2).run(suite)