Merge pull request #91 from spyoungtech/ci-fix

Ci fix

Author: spyoungtech

.github/workflows/publish.yaml

@@ -0,0 +1,38 @@
+name: release
+
+on:
+  push:
+    tags:
+      - 'v*.*.*'
+
+jobs:
+  build:
+    runs-on: ubuntu-latest
+    steps:
+    - name: Checkout
+      uses: actions/checkout@v2
+    - name: setup python
+      uses: actions/setup-python@v2
+      with:
+        python-version: 3.9
+
+    - name: build
+      shell: bash
+      run: |
+        python -m pip install -r requirements.txt
+        python -m pip install wheel
+        python setup.py sdist bdist_wheel
+    - name: Release PyPI
+      shell: bash
+      env:
+        TWINE_USERNAME: ${{ secrets.TWINE_USERNAME }}
+        TWINE_PASSWORD: ${{ secrets.TWINE_PASSWORD }}
+      run: |
+        pip install --upgrade twine
+        twine upload dist/*
+    - name: Release GitHub
+      uses: softprops/action-gh-release@v1
+      with:
+        files: "dist/*"
+      env:
+        GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

.github/workflows/test.yaml

@@ -0,0 +1,71 @@
+on: [push, pull_request]
+
+jobs:
+  test:
+    name: test
+    runs-on: ${{ matrix.os }}
+    strategy:
+      fail-fast: false
+      matrix:
+        os: [ubuntu-latest, macos-latest, windows-latest]
+        browser: [Chrome, Firefox, Safari, Edge]
+        exclude:
+          - os: macos-latest
+            browser: Edge
+          - os: windows-latest
+            browser: Safari
+          - os: ubuntu-latest
+            browser: Safari
+          - os: ubuntu-latest
+            browser: Edge
+
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v2
+      - uses: actions/setup-python@v2
+        with:
+          python-version: '3.9'
+
+      - name: enable-safaridriver
+        if: ${{ matrix.os == 'macos-latest' }}
+        run: sudo safaridriver --enable
+
+      - name: install-deps
+        run: |
+          python -m pip install -r ./requirements.txt
+          python -m pip install coveralls pytest mock
+
+      - name: run unittests
+        env:
+          BEHAVE_WEBDRIVER: ${{ matrix.browser }}
+          BEHAVE_WEBDRIVER_HEADLESS: "1"
+          ENV_BASE_URL: "http://127.0.0.1:8000/"
+        run: |
+          coverage run -m pytest tests/unittests
+
+      - name: start-test-server
+        shell: bash
+        run: |
+          cd tests/demo-app
+          python ./runserver.py > /dev/null 2>&1 &
+          cd ../..
+
+      - name: regedit
+        if: ${{ matrix.os == 'windows-latest' }}
+        shell: cmd
+        run: |
+          regedit /s .\ci\bfcache.reg
+
+      - name: run-behave-tests
+        env:
+          BEHAVE_WEBDRIVER: ${{ matrix.browser }}
+          BEHAVE_WEBDRIVER_HEADLESS: "1"
+          ENV_BASE_URL: "http://127.0.0.1:8000/"
+        run: |
+         coverage run -a -m behave tests/features
+
+      - name: coveralls
+        if: ${{ success() }}
+        env:
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+        run: coveralls --service=github

.travis.yml

@@ -692,6 +692,7 @@ class Edge(BehaveDriverMixin, webdriver.Edge):
     """
     Edge driver class. Alternate constructors and browser-specific logic is implemented here.
     """
+    _driver_name = 'msedgedriver'
 
 
 class Opera(BehaveDriverMixin, webdriver.Opera):

behave_webdriver/driver.py

@@ -5,6 +5,7 @@ Feature: Test how clicks are handled on a certain element
     Background:
         Given I open the site "/"
 
+    @skip_safari
     Scenario: Single click on a link should navigate to another page
         When  I click on the link "Navigate to example.com"
         Then  I expect the url to contain "https://example.com"
@@ -14,12 +15,15 @@ Feature: Test how clicks are handled on a certain element
         When  I click on the button "#toggleMessage"
         Then  I expect that element "#message1" is visible
         And   I expect that element "#message2" is not visible
+
     @firefox_bug
+    @skip_safari
     Scenario: Double click on the button #toggleMessage should display another message
         When  I doubleclick on the element "#toggleMessage"
         Then  I expect that element "#message1" is not visible
         And   I expect that element "#message2" is visible
 
+    @skip_safari
     Scenario: Single click on the element #toggleBackground should make the elemnt red
         When  I click on the element "#toggleBackground"
     @firefox_bug

ci/install.sh

@@ -20,17 +20,11 @@ Feature: Test the existens and content of cookies
     Scenario: The cookie "test" should not contain the value "out of date"
         Then  I expect that cookie "test" not contains "out of date"
 
-    #
-    # Currently this test is failing in PhantomJS due to an Ghostdriver issue
-    # https://github.com/detro/ghostdriver/issues/365
-    #
-    @Pending
     Scenario: The cookie "test3" should be created
         When  I set a cookie "test3" with the content "more cookies"
         Then  I expect that cookie "test3" exists
         And   I expect that cookie "test3" contains "more cookies"
 
-    @Pending
     Scenario: The cookie "test3" should be deletable
         Then  I expect that cookie "test3" exists
         When  I delete the cookie "test3"

tests/features/click.feature

@@ -4,13 +4,13 @@ Feature: Test the width and height of a given element
 
     Background:
         Given I open the site "/"
-
+    @skip_safari
     Scenario: The element #square100x100 whould have a width of 100px
         Then  I expect that element "#square100x100" is 100px broad
 
     Scenario: The element #square100x100 whould have a height of 100px
         Then  I expect that element "#square100x100" is 100px tall
-
+    @skip_safari
     Scenario: The element #square100x100 whould not have a width of 101px
         Then  I expect that element "#square100x100" is not 101px broad
 

tests/features/cookie.feature

@@ -46,6 +46,9 @@ def before_tag(context, tag):
 
 
 def before_feature(context, feature):
+    if "skip_safari" in feature.tags and os.environ.get("BEHAVE_WEBDRIVER", '').lower() == 'safari':
+        feature.skip()
+        return
     if "fresh_driver" in feature.tags:
         context.behave_driver.quit()
         context.behave_driver = context.BehaveDriver()
@@ -56,3 +59,6 @@ def before_scenario(context, scenario):
     if "skip_firefox" in scenario.effective_tags and os.environ.get("BEHAVE_WEBDRIVER", '').lower() == 'firefox':
         scenario.skip("Skipping because @skip_firefox tag (usually this is because of a known-issue with firefox)")
         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

tests/features/elementSize.feature

@@ -19,5 +19,5 @@ Feature: Test moveTo elements
         Then  I expect that element "#moveTo" has the class "moveToClass"
     @skip_firefox
     Scenario: Move to the element with a too large offset
-        When  I move to element "#moveTo" with an offset of 5000,5000
+        When  I move to element "#moveTo" with an offset of 941,21
         Then  I expect that element "#moveTo" does not have the class "moveToClass"