Rubocop - Style/BlockComments

Author: JaniJegoroff

android_tests/lib/android/specs/common/helper.rb

@@ -1,12 +1,10 @@
 describe 'common/helper' do
   wait_opts = { timeout: 0.2, interval: 0.2 } # max_wait, interval
 
-=begin
-There's no `must_not_raise` as the opposite of must_raise
+  # There's no `must_not_raise` as the opposite of must_raise
 
-By default code is expected to not raise exceptions.
-must_not_raise is a no-op.
-=end
+  # By default code is expected to not raise exceptions.
+  # must_not_raise is a no-op.
 
   # wait is a success unless an error is raised
   # max_wait=0 is infinity to use 0.1

android_tests/lib/android/specs/common/patch.rb

@@ -1,8 +1,7 @@
-=begin
-  Skip:
-    status # status patches are already tested in driver.rb
-    raw_execute # debug output for Pry
-=end
+# Skip:
+#   status # status patches are already tested in driver.rb
+#   raw_execute # debug output for Pry
+
 describe 'common/patch' do
   # Attributes are busted in Android.
   # Blocked on https://github.com/appium/appium/issues/628

android_tests/lib/android/specs/driver.rb

@@ -130,12 +130,11 @@ def validate_path(path)
       end
     end
 
-=begin
-  Skip:
-    ios_capabilities # save for iOS tests
-    absolute_app_path # tested already by starting the driver for this test
-    server_url # sauce labs only
-=end
+    # Skip:
+    #   ios_capabilities # save for iOS tests
+    #   absolute_app_path # tested already by starting the driver for this test
+    #   server_url # sauce labs only
+
     t 'restart' do
       set_wait 1 # ensure wait is 1 before we restart.
       restart
@@ -146,15 +145,14 @@ def validate_path(path)
       driver.browser.must_equal :Android
     end
 
-=begin
-  Skip:
-    screenshot   # this is slow and already tested by Appium
-    driver_quit  # tested by restart
-    start_driver # tested by restart
-    no_wait  # posts value to server, it's not stored locally
-    set_wait # posts value to server, it's not stored locally
-    execute_script # 'mobile: ' is deprecated and plain executeScript unsupported
-=end
+    # Skip:
+    #   screenshot   # this is slow and already tested by Appium
+    #   driver_quit  # tested by restart
+    #   start_driver # tested by restart
+    #   no_wait  # posts value to server, it's not stored locally
+    #   set_wait # posts value to server, it's not stored locally
+    #   execute_script # 'mobile: ' is deprecated and plain executeScript unsupported
+
     t 'default_wait' do
       set_wait 1
       default_wait.must_equal 1

android_tests/lib/format.rb

@@ -39,11 +39,9 @@
   puts line2
 end
 
-=begin
 # for Pry
-class Object
-  def must_equal b
-    raise 'not equal' unless self == b
-  end
-end
-=end
+# class Object
+#   def must_equal b
+#     raise 'not equal' unless self == b
+#   end
+# end

android_tests/lib/run.rb

@@ -4,13 +4,11 @@
 require 'hashdiff'
 require_relative '../../lib/appium_lib'
 
-=begin
-Run all Android tests:
-  ruby run.rb android
+# Run all Android tests:
+#   ruby run.rb android
 
-Run only the view album test:
-  ruby run.rb ios view_album
-=end
+# Run only the view album test:
+#   ruby run.rb ios view_album
 
 # Sanity check
 a = OpenStruct.new x: 'ok'