{"action":"closed","issue":{"url":"https://api.github.com/repos/sloria/doitlive/issues/12","labels_url":"https://api.github.com/repos/sloria/doitlive/issues/12/labels{/name}","comments_url":"https://api.github.com/repos/sloria/doitlive/issues/12/comments","events_url":"https://api.github.com/repos/sloria/doitlive/issues/12/events","html_url":"https://github.com/sloria/doitlive/issues/12","id":110982870,"number":12,"title":"Current dev branch does not pass for python2.7 (click warnings)","user":{"login":"Stefan-Code","id":9949033,"avatar_url":"https://avatars.githubusercontent.com/u/9949033?v=3","gravatar_id":"","url":"https://api.github.com/users/Stefan-Code","html_url":"https://github.com/Stefan-Code","followers_url":"https://api.github.com/users/Stefan-Code/followers","following_url":"https://api.github.com/users/Stefan-Code/following{/other_user}","gists_url":"https://api.github.com/users/Stefan-Code/gists{/gist_id}","starred_url":"https://api.github.com/users/Stefan-Code/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Stefan-Code/subscriptions","organizations_url":"https://api.github.com/users/Stefan-Code/orgs","repos_url":"https://api.github.com/users/Stefan-Code/repos","events_url":"https://api.github.com/users/Stefan-Code/events{/privacy}","received_events_url":"https://api.github.com/users/Stefan-Code/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":0,"created_at":"2015-10-12T14:15:06Z","updated_at":"2015-10-18T21:21:56Z","closed_at":"2015-10-18T21:21:56Z","body":"For me the [current dev branch] (https://github.com/sloria/doitlive/commit/784cba2ea028f85b14225af0b061f0aa90bfc9b3) without modifications does not pass the tests for python 2.7. They do pass for python3 though.\r\nTox tells me the following:\r\n```\r\ntests/test_doitlive.py::test_themes_list FAILED\r\ntests/test_doitlive.py::test_themes_preview FAILED\r\ntests/test_doitlive.py::test_version FAILED\r\n```\r\nand\r\n```\r\nrunner = <click.testing.CliRunner object at 0x7fe8148c5d50>\r\n\r\n    def test_themes_list(runner):\r\n        result1 = runner.invoke(cli, ['themes'])\r\n        assert result1.exit_code == 0\r\n        result2 = runner.invoke(cli, ['themes', '--list'])\r\n        result3 = runner.invoke(cli, ['themes', '-l'])\r\n>       assert result1.output == result2.output == result3.output\r\nE       assert '/home/sk/Pro...ssell giddie\\n' == '/home/sk/Prog...ssell giddie\\n'\r\nE         - /home/sk/Programming/doitlive-tox/tests/test_doitlive.py:165: Warning: Click detected the use of the unicode_literals __future__ import.  This is heavily discouraged because it can introduce subtle bugs in your code.  You should instead use explicit u\"\" literals for your unicode strings.  For more information see http://click.pocoo.org/python3/\r\nE         ?                                                            ^\r\nE         + /home/sk/Programming/doitlive-tox/tests/test_doitlive.py:167: Warning: Click detected the use of the unicode_literals __future__ import.  This is heavily discouraged because it can introduce subtle bugs in your code.  You should instead use explicit u\"\" literals for your unicode strings.  For more information see http://click.pocoo.org/python3/\r\nE         ?                                                            ^\r\nE         -   result1 = runner.invoke(cli, ['themes'])\r\nE         ?         ^\r\nE         +   result2 = runner.invoke(cli, ['themes', '--list'])\r\nE         ?         ^                               ++++++++++\r\nE           Available themes:\r\nE         Detailed information truncated (1 more lines), use \"-vv\" to show\r\n\r\nProgramming/doitlive-tox/tests/test_doitlive.py:169: AssertionError\r\n_________________________________________________________________________ test_themes_preview __________________________________________________________________________\r\n\r\nrunner = <click.testing.CliRunner object at 0x7fe8148c5d50>\r\n\r\n    def test_themes_preview(runner):\r\n        result1 = runner.invoke(cli, ['themes', '--preview'])\r\n        assert result1.exit_code == 0\r\n        result2 = runner.invoke(cli, ['themes', '-p'])\r\n        assert result2.exit_code == 0\r\n>       assert result1.output == result2.output\r\nE       assert '/home/sk/Pro...g2 ... argn\\n\\n' == '/home/sk/Prog...g2 ... argn\\n\\n'\r\nE         - /home/sk/Programming/doitlive-tox/tests/test_doitlive.py:173: Warning: Click detected the use of the unicode_literals __future__ import.  This is heavily discouraged because it can introduce subtle bugs in your code.  You should instead use explicit u\"\" literals for your unicode strings.  For more information see http://click.pocoo.org/python3/\r\nE         ?                                                            ^\r\nE         + /home/sk/Programming/doitlive-tox/tests/test_doitlive.py:175: Warning: Click detected the use of the unicode_literals __future__ import.  This is heavily discouraged because it can introduce subtle bugs in your code.  You should instead use explicit u\"\" literals for your unicode strings.  For more information see http://click.pocoo.org/python3/\r\nE         ?                                                            ^\r\nE         -   result1 = runner.invoke(cli, ['themes', '--preview'])\r\nE         ?         ^                                   - ------\r\nE         +   result2 = runner.invoke(cli, ['themes', '-p'])\r\nE         ?         ^\r\nE           Theme previews:\r\nE         Detailed information truncated (48 more lines), use \"-vv\" to show\r\n\r\nProgramming/doitlive-tox/tests/test_doitlive.py:177: AssertionError\r\n_____________________________________________________________________________ test_version _____________________________________________________________________________\r\n\r\nrunner = <click.testing.CliRunner object at 0x7fe8148c5d50>\r\n\r\n    def test_version(runner):\r\n        result = runner.invoke(cli, ['--version'])\r\n        print(\"huhuhu\", result.output, \"haha\")\r\n        assert doitlive.__version__ in result.output\r\n        result2 = runner.invoke(cli, ['-v'])\r\n>       assert result.output == result2.output\r\nE       assert '/home/sk/Pro...ersion 2.3.1\\n' == '/home/sk/Prog...ersion 2.3.1\\n'\r\nE         - /home/sk/Programming/doitlive-tox/tests/test_doitlive.py:181: Warning: Click detected the use of the unicode_literals __future__ import.  This is heavily discouraged because it can introduce subtle bugs in your code.  You should instead use explicit u\"\" literals for your unicode strings.  For more information see http://click.pocoo.org/python3/\r\nE         ?                                                            ^\r\nE         + /home/sk/Programming/doitlive-tox/tests/test_doitlive.py:184: Warning: Click detected the use of the unicode_literals __future__ import.  This is heavily discouraged because it can introduce subtle bugs in your code.  You should instead use explicit u\"\" literals for your unicode strings.  For more information see http://click.pocoo.org/python3/\r\nE         ?                                                            ^\r\nE         -   result = runner.invoke(cli, ['--version'])\r\nE         ?                                  - ------\r\nE         +   result2 = runner.invoke(cli, ['-v'])\r\nE         ?         +\r\nE           cli, version 2.3.1\r\nE         Detailed information truncated (0 more lines), use \"-vv\" to show\r\n\r\nProgramming/doitlive-tox/tests/test_doitlive.py:185: AssertionError\r\n```\r\nAs far as I can see there's a lot of Click warnings regarding Unicode. Maybe this warning was added recently, thus the tests fail for new versions of click?\r\nI suspect this is what's making my [pull request build fail] (https://github.com/sloria/doitlive/pull/11)\r\nEDIT:\r\nAlso a lot of ugly click warnings are appearing during normal usage:\r\n```\r\n(doitlive-2)sk@sk2 ~/Programming/doitlive-2 $ python2 doitlive.py play test.sh\r\ndoitlive.py:598: Warning: Click detected the use of the unicode_literals __future__ import.  This is heavily discouraged because it can introduce subtle bugs in your code.  You should instead use explicit u\"\" literals for your unicode strings.  For more information see http://click.pocoo.org/python3/\r\n  @click.group(context_settings={'help_option_names': ('-h', '--help')})\r\ndoitlive.py:637: Warning: Click detected the use of the unicode_literals __future__ import.  This is heavily discouraged because it can introduce subtle bugs in your code.  You should instead use explicit u\"\" literals for your unicode strings.  For more information see http://click.pocoo.org/python3/\r\n  @cli.command()\r\ndoitlive.py:686: Warning: Click detected the use of the unicode_literals __future__ import.  This is heavily discouraged because it can introduce subtle bugs in your code.  You should instead use explicit u\"\" literals for your unicode strings.  For more information see http://click.pocoo.org/python3/\r\n  @cli.command()\r\ndoitlive.py:706: Warning: Click detected the use of the unicode_literals __future__ import.  This is heavily discouraged because it can introduce subtle bugs in your code.  You should instead use explicit u\"\" literals for your unicode strings.  For more information see http://click.pocoo.org/python3/\r\n  @cli.command()\r\ndoitlive.py:784: Warning: Click detected the use of the unicode_literals __future__ import.  This is heavily discouraged because it can introduce subtle bugs in your code.  You should instead use explicit u\"\" literals for your unicode strings.  For more information see http://click.pocoo.org/python3/\r\n  @cli.command()\r\ndoitlive.py:827: Warning: Click detected the use of the unicode_literals __future__ import.  This is heavily discouraged because it can introduce subtle bugs in your code.  You should instead use explicit u\"\" literals for your unicode strings.  For more information see http://click.pocoo.org/python3/\r\n  cli()\r\nWe'll do it live!\r\nSTARTING SESSION: Press Ctrl-C at any time to exit.\r\nPress any key to continue ...\r\n```"}}