Skip to content

Commit 862813d

Browse files
authored
feat: remove unnecessary code (#680)
1 parent 63af649 commit 862813d

3 files changed

Lines changed: 0 additions & 95 deletions

File tree

lib/appium_lib_core/device.rb

Lines changed: 0 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -20,26 +20,6 @@ module Device
2020
class << self
2121
def extended(_mod)
2222
extend_webdriver_with_forwardable
23-
24-
# Compatibility for appium_lib. Below command are extended by `extend Appium::Core::Deivce`in appium_lib.
25-
# TODO: Will remove
26-
[
27-
:take_element_screenshot, :save_viewport_screenshot,
28-
:lock, :device_locked?, :unlock,
29-
:hide_keyboard, :is_keyboard_shown,
30-
:ime_activate, :ime_available_engines, :ime_active_engine, :ime_activated, :ime_deactivate,
31-
:get_settings, :update_settings,
32-
:within_context, :current_context, :available_contexts, :set_context,
33-
:push_file, :pull_file, :pull_folder,
34-
:keyevent, :press_keycode, :long_press_keycode,
35-
:match_images_features, :find_image_occurrence, :get_images_similarity, :compare_images,
36-
:app_strings, :background_app,
37-
:install_app, :remove_app, :app_installed?, :activate_app, :terminate_app,
38-
:app_state,
39-
:stop_recording_screen, :stop_and_save_recording_screen,
40-
:shake, :device_time,
41-
:execute_driver, :execute_cdp
42-
].each(&method(:delegate_from_appium_driver))
4323
end
4424

4525
# def extended

test/unit/android/device/w3c/definition_test.rb

Lines changed: 0 additions & 38 deletions
Original file line numberDiff line numberDiff line change
@@ -42,44 +42,6 @@ def parameterized_method_defined_check(array)
4242
delegate_from_appium_driver(v)
4343
end
4444
end
45-
46-
def test_with_arg_definitions
47-
parameterized_method_defined_check([:shake,
48-
:device_locked?,
49-
:unlock,
50-
:device_time,
51-
:current_context,
52-
:open_notifications,
53-
:current_activity,
54-
:current_package,
55-
:get_system_bars,
56-
:get_display_density,
57-
:is_keyboard_shown,
58-
:available_contexts,
59-
:set_context,
60-
:app_strings,
61-
:lock,
62-
:install_app,
63-
:remove_app,
64-
:app_installed?,
65-
:terminate_app,
66-
:activate_app,
67-
:app_state,
68-
:background_app,
69-
:hide_keyboard,
70-
:keyevent,
71-
:press_keycode,
72-
:long_press_keycode,
73-
:push_file,
74-
:pull_file,
75-
:pull_folder,
76-
:get_settings,
77-
:update_settings,
78-
:get_clipboard,
79-
:set_clipboard,
80-
:execute_driver,
81-
:execute_cdp])
82-
end
8345
end # class DefinitionTest
8446
end # module W3C
8547
end # module Device

test/unit/ios/device/w3c/definition_test.rb

Lines changed: 0 additions & 37 deletions
Original file line numberDiff line numberDiff line change
@@ -32,10 +32,6 @@ def test_delegate_driver_method
3232
assert @driver.respond_to? :device_locked?
3333
end
3434

35-
def test_delegate_from_appium_driver
36-
assert @core.send(:delegated_target_for_test).respond_to? :device_locked?
37-
end
38-
3935
def delegate_from_appium_driver(key)
4036
assert @core.send(:delegated_target_for_test).respond_to? key
4137
end
@@ -46,39 +42,6 @@ def parameterized_method_defined_check(array)
4642
delegate_from_appium_driver(v)
4743
end
4844
end
49-
50-
def test_with_arg_definitions
51-
parameterized_method_defined_check([:shake,
52-
:device_locked?,
53-
:unlock,
54-
:device_time,
55-
:current_context,
56-
:available_contexts,
57-
:set_context,
58-
:app_strings,
59-
:lock,
60-
:install_app,
61-
:remove_app,
62-
:terminate_app,
63-
:activate_app,
64-
:app_state,
65-
:app_installed?,
66-
:background_app,
67-
:hide_keyboard,
68-
:keyevent,
69-
:press_keycode,
70-
:long_press_keycode,
71-
:push_file,
72-
:pull_file,
73-
:pull_folder,
74-
:get_clipboard,
75-
:set_clipboard,
76-
:get_settings,
77-
:update_settings,
78-
:touch_id,
79-
:toggle_touch_id_enrollment,
80-
:execute_driver])
81-
end
8245
end # class DefinitionTest
8346
end # module W3C
8447
end # module Device

0 commit comments

Comments
 (0)