Skip to content

Commit

Permalink
ci: update test matrix
Browse files Browse the repository at this point in the history
Update test matrix of test-run, fix crashed test
on Tarantool 2.10 at call.test.py.

Resolves #363
  • Loading branch information
slavakirichenko committed Dec 13, 2022
1 parent 08d4ff6 commit 5ac4a55
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 19 deletions.
21 changes: 4 additions & 17 deletions .github/workflows/test.yml
Expand Up @@ -13,8 +13,8 @@ jobs:
strategy:
fail-fast: false
matrix:
python-version: ['2.7', '3.5', '3.6', '3.7', '3.8', '3.9', '3.10']
tarantool-version: [2.6, 2.7]
python-version: ['3.7', '3.8', '3.9', '3.10']
tarantool-version: ['2.8', '2.10']

steps:
- uses: actions/checkout@v3
Expand All @@ -35,19 +35,6 @@ jobs:
sudo apt update -y
sudo apt-get -y install lua5.1 luarocks
sudo luarocks install luacheck
- name: set default pip to pip3
run: |
sudo rm -f /usr/local/bin/pip
sudo ln -s /usr/bin/pip3 /usr/local/bin/pip
- name: set default python and pip to v2.7
if: ${{ matrix.python-version }} == 2.7
run: |
curl https://bootstrap.pypa.io/pip/2.7/get-pip.py --output get-pip.py
sudo python2 get-pip.py
sudo rm -f /usr/bin/python
sudo ln -s /usr/bin/python2 /usr/bin/python
sudo rm -f /usr/local/bin/pip
sudo ln -s /usr/local/bin/pip2 /usr/local/bin/pip
- name: setup python dependencies
run: |
pip install -r requirements.txt
Expand All @@ -64,12 +51,12 @@ jobs:
run: |
make test_integration
- name: code coverage
if: ${{ matrix.python-version == '3.8' && matrix.tarantool-version == '2.7' }}
if: ${{ matrix.python-version == '3.8' && matrix.tarantool-version == '2.10' }}
run: |
pip install coveralls==3.*
make coverage
- name: upload coverage data to coveralls.io
if: ${{ matrix.python-version == '3.8' && matrix.tarantool-version == '2.7' }}
if: ${{ matrix.python-version == '3.8' && matrix.tarantool-version == '2.10' }}
run: coveralls --service=github
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
3 changes: 1 addition & 2 deletions test/test-tarantool/call.result
Expand Up @@ -249,8 +249,7 @@ call myinsert (3, 'old', 2)
- [3, 'old', 2]
space:update({3}, {{'=', 1, 4}, {'=', 2, 'new'}})
---
- error: Attempt to modify a tuple field which is part of index 'primary' in space
'tweedledum'
- error: Attempt to modify a tuple field in space
...
space:insert(space:get{3}:update{{'=', 1, 4}, {'=', 2, 'new'}}) space:delete{3}
---
Expand Down
2 changes: 2 additions & 0 deletions test/test-tarantool/call.test.py
Expand Up @@ -125,6 +125,8 @@ def call(name, *args):
call("space:delete", 2)
call("space:delete", 2)
call("myinsert", 3, "old", 2)
sys.stdout.push_filter("Attempt to modify a tuple field.*in space.*", "Attempt to modify a tuple field in space")
sys.stdout.push_filter("\s+'tweedledum'", "")
admin("space:update({3}, {{'=', 1, 4}, {'=', 2, 'new'}})")
admin("space:insert(space:get{3}:update{{'=', 1, 4}, {'=', 2, 'new'}}) space:delete{3}")
call("space:get", 4)
Expand Down

0 comments on commit 5ac4a55

Please sign in to comment.