#3666 Fix usage of str.decode() in our tests
Closed: Fixed 6 years ago Opened 6 years ago by jhrozek.

There are some occurences of str.decode() in our tests, which are not compatible with Python3. We should fix those.


Lukas tols me that this original assumption was not correct. This issue with python3 is not caused by how the decode method is currently used but that it is not used.

The actual error looks like:

_______________________________________________________________________________________________________ test_sss_cache_invalidate_user _______________________________________________________________________________________________________
Traceback (most recent call last):                                                                                                                                                                                                           
  File "/home/build/sssd/src/tests/intg/test_ts_cache.py", line 635, in test_sss_cache_invalidate_user                                                                                                                                        
    assert sysdb_attrs.get("dataExpireTimestamp") == '1'                                                                                                                                                                                      
AssertionError: assert b'1' == '1'                                                                                                                                                                                                           
 +  where b'1' = <built-in method get of dict object at 0x7ff2ff743708>('dataExpireTimestamp')                                                                                                                                                
 +    where <built-in method get of dict object at 0x7ff2ff743708> = {'dataExpireTimestamp': b'1', 'originalModifyTimestamp': b'20180313115027Z'}.get                                                                                         
______________________________________________________________________________________________________ test_sss_cache_invalidate_group _______________________________________________________________________________________________________
Traceback (most recent call last):                                                                                                                                                                                                            
  File "/home/build/sssd/src/tests/intg/test_ts_cache.py", line 664, in test_sss_cache_invalidate_group                                                                                                                                       
    assert sysdb_attrs.get("dataExpireTimestamp") == '1'                                                                                                                                                                                      
AssertionError: assert b'1' == '1'                                                                                                                                                                                                            
 +  where b'1' = <built-in method get of dict object at 0x7ff2fd11d750>('dataExpireTimestamp')                                                                                                                                                
 +    where <built-in method get of dict object at 0x7ff2fd11d750> = {'dataExpireTimestamp': b'1', 'originalModifyTimestamp': b'20180313115029Z'}.get  

Metadata Update from @sbose:
- Issue assigned to sbose

6 years ago

Metadata Update from @sbose:
- Custom field patch adjusted to on

6 years ago

Metadata Update from @jhrozek:
- Issue set to the milestone: SSSD 1.16.2
- Issue tagged with: PR

6 years ago

Metadata Update from @jhrozek:
- Issue close_status updated to: Fixed
- Issue status updated to: Closed (was: Open)

6 years ago

SSSD is moving from Pagure to Github. This means that new issues and pull requests
will be accepted only in SSSD's github repository.

This issue has been cloned to Github and is available here:
- https://github.com/SSSD/sssd/issues/4686

If you want to receive further updates on the issue, please navigate to the github issue
and click on subscribe button.

Thank you for understanding. We apologize for all inconvenience.

Login to comment on this ticket.

Metadata