Merge pull request #92 from spyoungtech/pre-commit

spyoungtech · web-flow · commit 029019a8d0b9 · 2021-11-08T18:08:28.000-08:00
add pre-commit
diff --git a/.coveragerc b/.coveragerc
@@ -4,4 +4,4 @@ source = behave_webdriver
 
 [report]
 exclude_lines =
-    raise NotImplementedError
+    raise NotImplementedError
diff --git a/.gitattributes b/.gitattributes
@@ -1,2 +1,2 @@
 tests/features/* linguist-vendored
-tests/experimental-features/* linguest-vendored
+tests/experimental-features/* linguest-vendored
diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
@@ -0,0 +1,7 @@
+repos:
+-   repo: https://github.com/pre-commit/pre-commit-hooks
+    rev: v4.0.1
+    hooks:
+    -   id: check-yaml
+    -   id: end-of-file-fixer
+    -   id: trailing-whitespace
diff --git a/README.rst b/README.rst
@@ -220,7 +220,7 @@ Assuming you're in the repository root (and have the demo app running) just run
 Results ✨
 ^^^^^^^^^^
 
-.. code-block:: 
+.. code-block::
 
    (behave-webdriver) $ behave examples/selenium-requests/features
 
@@ -396,4 +396,3 @@ Special thanks to the authors and contributors of `behave`_
 
 .. |coverage| image:: https://coveralls.io/repos/github/spyoungtech/behave-webdriver/badge.svg
     :target: https://coveralls.io/github/spyoungtech/behave-webdriver
-
diff --git a/behave_webdriver/__init__.py b/behave_webdriver/__init__.py
@@ -30,4 +30,3 @@
                                        before_scenario_factory)
 from behave_webdriver.fixtures import use_fixture_tag
 from behave_webdriver import transformers
-
diff --git a/behave_webdriver/driver.py b/behave_webdriver/driver.py
@@ -724,4 +724,3 @@ class Remote(BehaveDriverMixin, webdriver.Remote):
     """
     Remote driver class. Alternate constructors and browser-specific logic is implemented here.
     """
-
diff --git a/behave_webdriver/fixtures.py b/behave_webdriver/fixtures.py
@@ -167,4 +167,3 @@ def use_fixture_tag(context, tag, *args, **kwargs):
         transformer_name = tag.split('.')[-1]
         transformer_class = getattr(transformers, transformer_name)
         use_fixture(transformation_fixture, context, transformer_class, **kwargs)
-
diff --git a/behave_webdriver/steps/expectations.py b/behave_webdriver/steps/expectations.py
@@ -331,4 +331,3 @@ def check_screen_size(context, x, y):
 
 
 use_step_matcher('parse')
-
diff --git a/ci/bfcache.reg b/ci/bfcache.reg
@@ -1,4 +1,4 @@
 Windows Registry Editor Version 5.00
 
 [HKEY_LOCAL_MACHINE\SOFTWARE\Wow6432Node\Microsoft\Internet Explorer\Main\FeatureControl\FEATURE_BFCACHE]
-"iexplore.exe"=dword:00000000
+"iexplore.exe"=dword:00000000
diff --git a/ci/install.bat b/ci/install.bat
@@ -6,4 +6,4 @@ pip install -r .\requirements.txt
 
 pip install pytest mock coverage
 
-call deactivate
+call deactivate
diff --git a/ci/runtests.bat b/ci/runtests.bat
@@ -6,4 +6,4 @@ coverage run -m behave tests\features --format=progress2 --junit
 
 coverage run -a -m pytest tests\unittests --junitxml=reports\pytestresults.xml
 
-call deactivate
+call deactivate
diff --git a/docs/Makefile b/docs/Makefile
@@ -17,4 +17,4 @@ help:
 # Catch-all target: route all unknown targets to Sphinx using the new
 # "make mode" option.  $(O) is meant as a shortcut for $(SPHINXOPTS).
 %: Makefile
-	@$(SPHINXBUILD) -M $@ "$(SOURCEDIR)" "$(BUILDDIR)" $(SPHINXOPTS) $(O)
+	@$(SPHINXBUILD) -M $@ "$(SOURCEDIR)" "$(BUILDDIR)" $(SPHINXOPTS) $(O)
diff --git a/docs/api.rst b/docs/api.rst
@@ -64,4 +64,3 @@ The mixin class implements all of the general logic. If you want to alter how be
 
 .. autoclass:: behave_webdriver.driver.BehaveDriverMixin
    :members:
-
diff --git a/docs/browsers.rst b/docs/browsers.rst
@@ -122,4 +122,3 @@ Android
 -------
 
 Android is untested at this time.
-
diff --git a/docs/conf.py b/docs/conf.py
@@ -170,6 +170,3 @@
      author, 'behave-webdriver', 'One line description of project.',
      'Miscellaneous'),
 ]
-
-
-
diff --git a/docs/index.rst b/docs/index.rst
@@ -100,4 +100,3 @@ they are not all supported at this time. We plan to add support for additional b
 
 .. |coverage| image:: https://coveralls.io/repos/github/spyoungtech/behave-webdriver/badge.svg
     :target: https://coveralls.io/github/spyoungtech/behave-webdriver
-
diff --git a/docs/roadmap.rst b/docs/roadmap.rst
@@ -144,4 +144,4 @@ deprecated for selenium and `phantomJS development has been suspended`_, Phantom
 .. _raise an issue: https://github.com/spyoungtech/behave-webdriver/issues/new
 
 
-.. _phantomJS development has been suspended: https://github.com/ariya/phantomjs/issues/15344
+.. _phantomJS development has been suspended: https://github.com/ariya/phantomjs/issues/15344
diff --git a/docs/steps.rst b/docs/steps.rst
@@ -96,4 +96,3 @@ Then Steps ✔️
 - ``I expect the url "([^"]*)?" is opened in a new (tab|window)``
 - ``I expect the url to( not)* contain "([^"]*)?"``
 - ``I wait on element "([^"]*)?"(?: for (\d+)ms)*(?: to( not)* (be checked|be enabled|be selected|be visible|contain a text|contain a value|exist))*``
-
diff --git a/examples/minimal-project/features/myFeature.feature b/examples/minimal-project/features/myFeature.feature
@@ -14,4 +14,4 @@
       Given the title is not "two"
       And   I open the url "http://webdriverjs.christian-bromann.com/"
       When  I click on the link "two"
-      Then  I expect that the title is "two"
+      Then  I expect that the title is "two"
diff --git a/examples/minimal-project/features/steps/my_steps.py b/examples/minimal-project/features/steps/my_steps.py
@@ -1 +1 @@
-from behave_webdriver.steps import *
+from behave_webdriver.steps import *
diff --git a/examples/selenium-requests/features/environment.py b/examples/selenium-requests/features/environment.py
@@ -9,4 +9,4 @@ def before_all(context):
     context.behave_driver = BehaveRequestDriver()
 
 def after_all(context):
-    context.behave_driver.quit()
+    context.behave_driver.quit()
diff --git a/examples/selenium-requests/features/requests.feature b/examples/selenium-requests/features/requests.feature
@@ -6,4 +6,4 @@ Feature: Using selenium-requests
   Scenario: use selenium-requests with behave-webdriver
     Given the base url is "http://127.0.0.1:8000"
     Given I send a GET request to the page "/"
-    Then I expect the response text contains "<h1>DEMO APP</h1>"
+    Then I expect the response text contains "<h1>DEMO APP</h1>"
diff --git a/tests/demo-app/inc/script.js b/tests/demo-app/inc/script.js
@@ -242,4 +242,4 @@ $(function () {
 
     $('#moveTo')
         .on('mouseenter mouseleave', toggleMoveToElement);
-});
+});
diff --git a/tests/demo-app/index.html b/tests/demo-app/index.html
@@ -193,4 +193,4 @@ <h1>DEMO APP</h1>
 
         <script src="inc/script.js"></script>
     </body>
-</html>
+</html>
diff --git a/tests/demo-app/page.html b/tests/demo-app/page.html
@@ -16,4 +16,4 @@ <h1>This is a page</h1>
 </select>
 
 </body>
-</html>
+</html>
diff --git a/tests/demo-app/runserver.py b/tests/demo-app/runserver.py
@@ -26,4 +26,4 @@
 
 httpd = HTTPServer(("", server_port), SimpleHTTPRequestHandler)
 print("serving at port {0}".format(server_port))
-httpd.serve_forever()
+httpd.serve_forever()
diff --git a/tests/experimental-features/environment.py b/tests/experimental-features/environment.py
@@ -3,4 +3,4 @@
 def before_all(context):
     context.behave_driver = BehaveDriver.headless_chrome()
 def after_all(context):
-    context.behave_driver.quit()
+    context.behave_driver.quit()
diff --git a/tests/experimental-features/steps/webdriver_steps.py b/tests/experimental-features/steps/webdriver_steps.py
@@ -1 +1 @@
-from behave_webdriver.steps import *
+from behave_webdriver.steps import *
diff --git a/tests/features/baseUrl.feature b/tests/features/baseUrl.feature
@@ -7,4 +7,3 @@ Feature: Base URL configuration
     When I open the site "/page.html"
     Then I expect that the url is "http://127.0.0.1:8000/page.html"
     And I expect that the url is not "http://localhost:8000/page.html"
-
diff --git a/tests/features/baseUrlWithParameterTransformation.feature b/tests/features/baseUrlWithParameterTransformation.feature
@@ -13,4 +13,4 @@ Feature: Base URL configuration supports parameter transformation
     Given the base url is "{ALT_BASE_URL}/"
     When I open the site "/page.html"
     Then I expect that the url is "{ALT_BASE_URL}/page.html"
-    And I expect that the url is not "{BASE_URL}/page.html"
+    And I expect that the url is not "{BASE_URL}/page.html"
diff --git a/tests/features/drag.feature b/tests/features/drag.feature
@@ -9,4 +9,4 @@ Feature: Test draggable elements
 
     Scenario: Drag to dropzone
         When  I drag element "#draggable" to element "#droppable"
-        Then  I expect that element "#droppable" contains the text "Dropped!"
+        Then  I expect that element "#droppable" contains the text "Dropped!"
diff --git a/tests/features/environment.py b/tests/features/environment.py
@@ -61,4 +61,4 @@ def before_scenario(context, scenario):
         return
     if "skip_safari" in scenario.effective_tags and os.environ.get("BEHAVE_WEBDRIVER", '').lower() == 'safari':
         scenario.skip("Skipping because safari has issues we're not dealing with right now")
-        return
+        return
diff --git a/tests/features/screenSize.feature b/tests/features/screenSize.feature
@@ -15,4 +15,4 @@ Feature: Screen Size
 
   Scenario: Change just X dimension
     Given I have a screen that is 700 pixels broad
-    Then I expect the screen is 700 by 500 pixels
+    Then I expect the screen is 700 by 500 pixels
diff --git a/tests/features/select.feature b/tests/features/select.feature
@@ -63,4 +63,3 @@ Feature: Test select elements
         Then I expect that element "#selectElementTest option:nth-child(2)" is not selected
         When I select the option with the name "secondOption" for element "//select[@id='selectElementTest']"
         Then I expect that element "#selectElementTest option:nth-child(2)" is selected
-
diff --git a/tests/features/steps/webdriver_steps.py b/tests/features/steps/webdriver_steps.py
@@ -10,4 +10,4 @@ def test_step_raises_exception(context, step_text):
     except Exception as e:
         print(e)
     else:
-        raise AssertionError('Step did not raise exception')
+        raise AssertionError('Step did not raise exception')
diff --git a/tests/features/textComparison.feature b/tests/features/textComparison.feature
@@ -42,4 +42,4 @@ Feature: Test text contents of elements
 
     Scenario: Test Input matches text
       Then I expect that element "#valueDoesContainCucumber" matches the text "This input contains cucumber"
-      And I expect that element "#valueDoesContainCucumber" not matches the text "Something else"
+      And I expect that element "#valueDoesContainCucumber" not matches the text "Something else"
diff --git a/tests/features/urlValidation.feature b/tests/features/urlValidation.feature
@@ -24,4 +24,4 @@ Feature: Test if the url is a certain value
 
     Scenario: The url should not contain "index"
         Given I open the site "/index.html"
-        Then  I expect the url to contain "index"
+        Then  I expect the url to contain "index"
diff --git a/tests/unittests/test_parameter_trasformations.py b/tests/unittests/test_parameter_trasformations.py
@@ -5,4 +5,3 @@
 root_dir = os.path.abspath(os.path.join(present_dir, '..', '..'))
 sys.path.insert(0, root_dir)
 from behave_webdriver.transformers import FormatTransformer
-

Original file line number	Diff line number	Diff line change
`@@ -1,2 +1,2 @@`
`1`	`1`	`tests/features/* linguist-vendored`
`2`		`-tests/experimental-features/* linguest-vendored`
	`2`	`+tests/experimental-features/* linguest-vendored`
-Original file line number
+Diff line change
     """
     Remote driver class. Alternate constructors and browser-specific logic is implemented here.
     """
+-
Original file line number	Diff line number	Diff line change
`@@ -331,4 +331,3 @@ def check_screen_size(context, x, y):`
`331`	`331`
`332`	`332`
`333`	`333`	`use_step_matcher('parse')`
`334`		`-`
Original file line number	Diff line number	Diff line change
`@@ -6,4 +6,4 @@ pip install -r .\requirements.txt`
`6`	`6`
`7`	`7`	`pip install pytest mock coverage`
`8`	`8`
`9`		`-call deactivate`
	`9`	`+call deactivate`