Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test: bump test-run to version w/ updated luatest #8555

Merged
merged 4 commits into from
Apr 12, 2023

Conversation

ylobankov
Copy link
Member

@ylobankov ylobankov commented Apr 11, 2023

Revert "test: dirty fix for some flaky replication-luatest tests"


test: bump test-run to version w/ updated luatest

Bump test-run to new version with the following improvements:

  • Properly ignore Unix sockets on artifact saving [1]
  • Bump luatest to 0.5.7-34-g930b63b [2]

[1] tarantool/test-run@3afaccc
[2] tarantool/test-run@02c4828


test: update tests according to luatest changes

All tests were updated according to tarantool/luatest@930b63b. So this
should resolve the problem with Unix socket collisions that we tried to
fix, for example, in 7ac2685.


ci: reduce testing for macOS 12

We have the limited hardware resources with macOS 12, and full testing
is time-consuming. So let's check only the release build on the x86_64
platform.

This reverts commit 7ac2685.

NO_DOC=testing stuff
NO_TEST=testing stuff
NO_CHANGELOG=testing stuff
Bump test-run to new version with the following improvements:

- Properly ignore Unix sockets on artifact saving [1]
- Bump luatest to 0.5.7-34-g930b63b [2]

[1] tarantool/test-run@3afaccc
[2] tarantool/test-run@02c4828

NO_DOC=testing stuff
NO_TEST=testing stuff
NO_CHANGELOG=testing stuff
@coveralls
Copy link

coveralls commented Apr 11, 2023

Coverage Status

Coverage: 85.543% (+0.009%) from 85.534% when pulling 7a62e77 on ylobankov/bump-test-run into 210600d
on master
.

All tests were updated according to tarantool/luatest@930b63b. So this
should resolve the problem with Unix socket collisions that we tried to
fix, for example, in 7ac2685.

NO_DOC=testing stuff
NO_TEST=testing stuff
NO_CHANGELOG=testing stuff
@ylobankov ylobankov added full-ci Enables all tests for a pull request and removed full-ci Enables all tests for a pull request labels Apr 11, 2023
Copy link
Contributor

@ochaplashkin ochaplashkin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ylobankov ylobankov added full-ci Enables all tests for a pull request and removed full-ci Enables all tests for a pull request labels Apr 12, 2023
@ylobankov ylobankov added full-ci Enables all tests for a pull request and removed full-ci Enables all tests for a pull request labels Apr 12, 2023
We have the limited hardware resources with macOS 12, and full testing
is time-consuming. So let's check only the release build on the x86_64
platform.

NO_DOC=ci
NO_TEST=ci
NO_CHANGELOG=ci
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
full-ci Enables all tests for a pull request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants