[ 
https://issues.apache.org/jira/browse/CASSANDRA-14298?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16463308#comment-16463308
 ] 

Patrick Bannister commented on CASSANDRA-14298:
-----------------------------------------------

Yes! I've modified test_cqlsh/test_cqlsh.py::TestCqlsh::run_cqlsh to open the 
cqlsh subprocess with universal_newlines=True and that allowed us to remove all 
bytes and calls to encode() and decode() from functions using our local 
run_cqlsh. We're still passing on trunk, 3.11, and 3.0.

I posted a new patch, CASSANDRA-14298.txt. I also re-posted the previous patch 
as CASSANDRA-14298_old.txt, in case it's needed for some reason.

> cqlshlib tests broken on b.a.o
> ------------------------------
>
>                 Key: CASSANDRA-14298
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-14298
>             Project: Cassandra
>          Issue Type: Bug
>          Components: Build, Testing
>            Reporter: Stefan Podkowinski
>            Assignee: Patrick Bannister
>            Priority: Major
>              Labels: cqlsh, dtest
>         Attachments: CASSANDRA-14298.txt, CASSANDRA-14298_old.txt, 
> cqlsh_tests_notes.md
>
>
> It appears that cqlsh-tests on builds.apache.org on all branches stopped 
> working since we removed nosetests from the system environment. See e.g. 
> [here|https://builds.apache.org/view/A-D/view/Cassandra/job/Cassandra-trunk-cqlsh-tests/458/cython=no,jdk=JDK%201.8%20(latest),label=cassandra/console].
>  Looks like we either have to make nosetests available again or migrate to 
> pytest as we did with dtests. Giving pytest a quick try resulted in many 
> errors locally, but I haven't inspected them in detail yet. 



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org
For additional commands, e-mail: commits-h...@cassandra.apache.org

Reply via email to