Skip to content

Commit

Permalink
Fix subprocess for ktutil on linux. Temporary limit test runner to de…
Browse files Browse the repository at this point in the history
…bug sasl tests
  • Loading branch information
tvoinarovskyi committed Mar 9, 2019
1 parent bba3492 commit 74fda69
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ ci-test-unit:
py.test -s --cov aiokafka --cov-report html $(FLAGS) tests

ci-test-all:
py.test -s --cov aiokafka --cov-report html --docker-image $(DOCKER_IMAGE) $(FLAGS) tests
py.test -s --cov aiokafka --cov-report html --docker-image $(DOCKER_IMAGE) $(FLAGS) -k sasl tests

coverage.xml: .coverage
coverage xml
Expand Down
2 changes: 1 addition & 1 deletion tests/_testutil.py
Original file line number Diff line number Diff line change
Expand Up @@ -236,7 +236,7 @@ def create_keytab(
subprocess.run(
['ktutil'],
cwd=str(keytab_dir.absolute()),
input=input_data, check=True)
input=input_data.encode(), check=True)
else:
raise NotImplementedError

Expand Down

0 comments on commit 74fda69

Please sign in to comment.