Skip to content

Fix tests#55

Merged
windreamer merged 2 commits intodouban:masterfrom
mckelvin:fix-tests
Nov 4, 2017
Merged

Fix tests#55
windreamer merged 2 commits intodouban:masterfrom
mckelvin:fix-tests

Conversation

@mckelvin
Copy link
Contributor

@mckelvin mckelvin commented Nov 4, 2017

  • Update the cppcheck suppression list
  • Install libmemcached10 instead of libmemcached6 in Travis-CI since the latter one is not available now.
  • Update test_version in test_client.cpp because the version number is 1.4.14 (Ubuntu) on Travis CI. The last char used to be a digit but now it is not.

The last char of memcached version used to be a digit,
but right now it could be a non-digit. So let's check
the first digit instead.
@windreamer windreamer merged commit 6821bde into douban:master Nov 4, 2017
@mckelvin mckelvin deleted the fix-tests branch November 4, 2017 05:20
@mckelvin
Copy link
Contributor Author

mckelvin commented Nov 4, 2017

Thanks @windreamer

tclh123 added a commit that referenced this pull request Feb 18, 2019
ChangeLog:

- use pickle protocol version 2 (#52)
- fix rvalue reference (#53)
- Fix tests (#55)
- golibmc_test: Fix fragile test case (#56)
- Use connection pool in golibmc(again) (#57)
- golibmc Quit ret err (#59)
- cmake: Incorporate gtest in a standard way (#58)
- Rename refactor (#60)
- sync rapidjson/itoa.h upstream fix (#65)
- split cppcheck (#67)
- split FSM_GET_BYTES_CAS (#68)
- Avoid noisy recv_err when broadcast quit (#73)
- Try to support updating some servers without affecting others (#74)
- make normalize_key cpdef & add warnings for unpickle unmarshal fail (#66)
- fix fcntl usage when set socket nonblock (#69)
- minor fix (#61)
- add condition log macro (#77)
- Define and use notWaitForRetryTimeout (#82)
- upgrade travis to gcc 7 (#91)
- add func errCodeToString (#79)
- Introduce a reconnect mechanism in waitPoll (#88)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants