Remove simplejson

[asmundg]

Requests will actually raise a ValueError, meaning that using simplejson here is both an undeclared dependency and wrong.

The tests require #2 to run.

[dsoprea]

After removing the environments that I don't support, as well as the "-r{toxinidir}/requirements.txt" dependency (which shouldn't ever be valid), I still get a certain type of failure with all remaining environments. This is the failure for python27:

$ tox
GLOB sdist-make: /Users/dustin/build/etcd.asmundg/setup.py
py27 inst-nodeps: /Users/dustin/build/etcd.asmundg/.tox/dist/etcd-2.0.8.zip
py27 runtests: PYTHONHASHSEED='2124068991'
py27 runtests: commands[0] | py.test --junitxml=xunit_test_report.xml --cov /Users/dustin/build/etcd.asmundg/.tox/py27/lib/python2.7/site-packages/etcd --cov-report term-missing
WARNING:test command found but not installed in testenv
  cmd: /usr/local/bin/py.test
  env: /Users/dustin/build/etcd.asmundg/.tox/py27
Maybe forgot to specify a dependency?
usage: py.test [options] [file_or_dir] [file_or_dir] [...]
py.test: error: unrecognized arguments: --cov --cov-report term-missing
ERROR: InvocationError: '/usr/local/bin/py.test --junitxml=xunit_test_report.xml --cov /Users/dustin/build/etcd.asmundg/.tox/py27/lib/python2.7/site-packages/etcd --cov-report term-missing'
___________________________________________________________________________________________________________________________ summary ____________________________________________________________________________________________________________________________
ERROR:   py27: commands failed

What am I missing?

[asmundg]

You're missing the requirements.txt file you pointed out. :)

[dsoprea]

The requirements file is in the resources directory, but you were referring
to it in two separate locations.

I don't understand.
On Oct 26, 2014 3:37 AM, "Åsmund Grammeltvedt" [email protected]
wrote:

You're missing the requirements.txt file you pointed out. :)

—
Reply to this email directly or view it on GitHub
#3 (comment)
.

[asmundg]

I didn't want to mess too much with your layout, so I added a separate
requirements.txt including the test requirements (pytest,
pytest-coverage), which I then forgot to add.

The latest commit includes it and tox now runs correctly from a clean
branch.

Åsmund Grammeltvedt