Skip to content

Commit

Permalink
make python2 compat
Browse files Browse the repository at this point in the history
  • Loading branch information
deargle committed Jun 28, 2019
1 parent 7e818cf commit be169c6
Show file tree
Hide file tree
Showing 4 changed files with 14 additions and 14 deletions.
9 changes: 4 additions & 5 deletions psiturk/psiturk_exceptions.py
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
class PsiturkException(Exception):
def __init__(self, **kwargs):
super().__init__()
super(PsiturkException, self).__init__()
self.message=kwargs['message'] if 'message' in kwargs else ''

def __str__(self):
Expand All @@ -22,20 +22,19 @@ def __init__(self, **kwargs):
'- `psiturk debug -p`'])

if 'message' in kwargs:
# import pytest; pytest.set_trace()
kwargs['message'] = ''.join([kwargs['message'], still_can_do])

super().__init__(**kwargs)
super(AmtServicesException, self).__init__(**kwargs)

class AWSAccessKeysNotSetError(AmtServicesException):
def __init__(self, **kwargs):
message = 'AWS access keys not set in ~/.psiturkconfig; please enter valid credentials.'
super().__init__(message=message, **kwargs)
super(AWSAccessKeysNotSetError, self).__init__(message=message, **kwargs)

class NoMturkConnectionError(AmtServicesException):
def __init__(self, **kwargs):
message = 'Sorry, unable to connect to Amazon Mechanical Turk. AWS credentials invalid.'
super().__init__(message=message, **kwargs)
super(NoMturkConnectionError, self).__init__(message=message, **kwargs)

####################################################
# AMT Service Wrapper Exceptions
Expand Down
4 changes: 2 additions & 2 deletions psiturk/psiturk_shell.py
Original file line number Diff line number Diff line change
Expand Up @@ -99,7 +99,7 @@ def postcmd(self, *args):
return Cmd.postcmd(self, *args)

def __init__(self, config, server, quiet=False):
persistent_history_file=config['Shell Parameters']['persistent_history_file']
persistent_history_file=config.get('Shell Parameters','persistent_history_file')
Cmd.__init__(self, persistent_history_file=persistent_history_file)
self.config = config
self.server = server
Expand Down Expand Up @@ -770,7 +770,7 @@ def __init__(self, config, server, sandbox, quiet=False):
self.sandbox = sandbox
self.sandbox_hits = 0
self.live_hits = 0
super().__init__(config, server, quiet)
super(PsiturkNetworkShell, self).__init__(config, server, quiet)

if not self.amt_services_wrapper:
sys.exit()
Expand Down
2 changes: 1 addition & 1 deletion psiturk/version.py
Original file line number Diff line number Diff line change
@@ -1 +1 @@
version_number = '2.3.1'
version_number = '2.3.2'
13 changes: 7 additions & 6 deletions tests/test_noaws.py
Original file line number Diff line number Diff line change
Expand Up @@ -10,9 +10,9 @@ def test_awskeys_invalid(capfd):

config = PsiturkConfig()
config.load_config()
config['Shell Parameters']['persistent_history_file'] = ''
config['AWS Access']['aws_access_key_id'] = ''
config['AWS Access']['aws_secret_access_key'] = ''
config.set('Shell Parameters', 'persistent_history_file', '')
config.set('AWS Access','aws_access_key_id', '')
config.set('AWS Access','aws_secret_access_key', '')
server = control.ExperimentServerController(config)

launch_in_sandbox_mode = True
Expand All @@ -38,9 +38,10 @@ def test_awscreds_notset(capfd):

config = PsiturkConfig()
config.load_config()
config['Shell Parameters']['persistent_history_file'] = ''
config['AWS Access']['aws_access_key_id'] = 'YourAccessKeyId'
config['AWS Access']['aws_secret_access_key'] = 'YourSecretAccessKey'
config.set('Shell Parameters', 'persistent_history_file', '')
config.set('AWS Access','aws_access_key_id', 'YourAccessKeyId')
config.set('AWS Access','aws_secret_access_key', 'YourSecretAccessKey')

server = control.ExperimentServerController(config)

launch_in_sandbox_mode = True
Expand Down

0 comments on commit be169c6

Please sign in to comment.