search

sauce-archives / python-sauceconnect

Python library for Sauce Connect 4
Apache License 2.0
7 stars 2 forks source link

Is this stable yet? #1

Open shawnbot opened 9 years ago

shawnbot commented 9 years ago

I'm trying to institute some cross-browser testing practices at @18F, and this seems like the right thing to use since a lot of our projects are in Python. But I'm tripping up on the installation instructions for libsauceconnect:

Set the environment variable SAUCE_CONNECT_DIR to the root of the build (this will be the folder created by unzipping the downloaded file). Once this Python distribution is stable, the SauceConnect library will be included within it.

I've got the binary distribution downloaded and have exported SAUCE_CONNECT_DIR=/absolute/path/to/sc-4.3.8-osx, but importing sauceconnect in Python still throws:

>>> import sauceconnect
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "path/to/sauceconnect/__init__.py", line 1, in <module>
    from libsauceconnect import SauceConnect
  File "path/to/sauceconnect/libsauceconnect.py", line 67, in <module>
    raise Exception("can't find libsauceconnect")
Exception: can't find libsauceconnect

If this is stable, then libsauceconnect should be packaged with the distribution. But the fact that this repo hasn't been touched in over a year suggests that it may have been abandoned. Any advice?

jlipps commented 9 years ago

this is not currently being maintained, though @imurchie might have some pointers on whether this error in particular can be worked around

imurchie commented 9 years ago

It hasn't been abandoned, just not maintained of late, unfortunately. There is much functionality in Sauce Connect 4 that wasn't stable at the time and hasn't been brought into this client.

For this particular error, I'm not sure what is going on. I'm not sure what this sauceconnect is that you are importing, or where you are doing it.

imurchie commented 9 years ago

With the caveat that this is not currently maintained, it still works for the basic case.

A run with the test in test/test.py: https://saucelabs.com/tests/6e573777024e44c7a996502167676a1c

shawnbot commented 9 years ago

Have you tried installing it via pip and running the tests? Because I'm wondering if the pypi package is just out of date. Looking at setup.py, I would expect pip install sauceconnect to fail if SAUCE_CONNECT_DIR isn't set, but it doesn't. This does, however:

pip install -e git+https://github.com/saucelabs/python-sauceconnect.git#egg=Package

So, could you publish a new version and tag it as a release?

imurchie commented 9 years ago

I'm looking into this. This repo and that PyPi package are not related, however. This repo has never been released.

shawnbot commented 9 years ago

Well, that explains that. :)