Skip to content
This repository was archived by the owner on Oct 29, 2024. It is now read-only.

Fix testsuite #333

Merged
merged 8 commits into from
Jun 13, 2016
Merged

Fix testsuite #333

merged 8 commits into from
Jun 13, 2016

Conversation

wwentland
Copy link
Contributor

These commits fix some of the errors in the testsuite and mostly concern unicode handling and retention policy tests.

This makes it easier to see changes to test data in diffs.
This information was introduced in InfluxDB 0.10.0 and tests fail
without it.
There can only be a single default retention policy per database.
This will run tests on travis using 0.13.0
@aviau
Copy link
Collaborator

aviau commented Jun 12, 2016

It seems like the test have failed due to an unrelated HTTP error. I have launched the test suite again!

@aviau
Copy link
Collaborator

aviau commented Jun 12, 2016

Hmm what is going on...

OpenSSL: error:14077410:SSL routines:SSL23_GET_SERVER_HELLO:sslv3 alert handshake failure
Unable to establish SSL connection.

I'll wait a few hours and try again, this is probably due to network issues at Travis?

@aviau
Copy link
Collaborator

aviau commented Jun 12, 2016

I'll open a PR against your branch.

The following PR will fix the download issue: https://github.com/babilen/influxdb-python/pull/1

aviau and others added 3 commits June 11, 2016 21:43
This removes tests and compatibility code for Python 2.6.
@aviau
Copy link
Collaborator

aviau commented Jun 13, 2016

Almost there, only a few failing tests.

@wwentland
Copy link
Contributor Author

wwentland commented Jun 13, 2016

Well, #314 is a tricky one. It would be much appreciated if you could merge this so that other people can get PRs in also.

@aviau
Copy link
Collaborator

aviau commented Jun 13, 2016

I'll merge this as it is a great improvement on what we had before.

Thank you for your time @BABILEN

@aviau aviau merged commit 68c95e5 into influxdata:master Jun 13, 2016
@wwentland
Copy link
Contributor Author

You are welcome, let's see if we can tackle the remaining issues in due course.

@wwentland wwentland deleted the fix-testsuite branch June 13, 2016 13:58
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants