Opened 3 years ago

Closed 3 years ago

#21832 closed defect (fixed)

test_set_process_name failed.

Reported by: meto Owned by: atagar
Priority: Very Low Milestone:
Component: Core Tor/Stem Version:
Severity: Trivial Keywords: testing
Cc: Actual Points:
Parent ID: Points:
Reviewer: Sponsor:

Description

Hello,

I downloaded stem and ran the ./run_tests.py --integ on OS X 10.11.6
and found the following failing test:

test_set_process_name [FAILURE]

======================================================================
ERROR: test_set_process_name


Traceback (most recent call last):

File "/Users/denis/Desktop/stem/test/integ/util/system.py", line 591, in test_set_process_name

stem.util.system.set_process_name('stem_integ')

File "/Users/denis/Desktop/stem/stem/util/system.py", line 1217, in set_process_name

_set_proc_title(process_name)

File "/Users/denis/Desktop/stem/stem/util/system.py", line 1274, in _set_proc_title

name_buffer.value = process_name

TypeError: bytes expected instead of str instance

During handling of the above exception, another exception occurred:

Traceback (most recent call last):

File "/Users/denis/Desktop/stem/test/integ/util/system.py", line 594, in test_set_process_name

stem.util.system.set_process_name(initial_name)

File "/Users/denis/Desktop/stem/stem/util/system.py", line 1217, in set_process_name

_set_proc_title(process_name)

File "/Users/denis/Desktop/stem/stem/util/system.py", line 1274, in _set_proc_title

name_buffer.value = process_name

TypeError: bytes expected instead of str instance


Ran 29 tests in 1.040s

FAILED (errors=1, skipped=13)

After fixing the test, I tested to see if everything is okay with:

Python 3.4.4
Tor version 0.2.9.10 (git-1f6c8eda0073f464).
mock (2.0.0)
pbr (2.0.0)
pip (7.1.2)
pycodestyle (2.3.1)
pyflakes (1.5.0)
setuptools (18.2)
six (1.10.0)

Python 2.7.10
funcsigs (1.0.2)
mock (2.0.0)
pbr (2.0.0)
pip (6.1.1)
pycodestyle (2.3.1)
pyflakes (1.5.0)
setuptools (15.2)
six (1.10.0)

Fix: https://github.com/Metonimie/stem/commit/e4f6da4c5dec93b95a8173e0850471f5076cad9a

Child Tickets

Change History (2)

comment:1 Changed 3 years ago by meto

Keywords: testing added

comment:2 Changed 3 years ago by atagar

Resolution: fixed
Status: newclosed

Great, thanks Denis! Fix merged.

Note: See TracTickets for help on using tickets.