🔑 Add authenticated property to context

[Kezzsim]

Resolves #818

Checklist

• Have the authenticated property return True or False depending on the state of the client
• Have the authenticated property show up in the tab context menu of ipython or in the output of dir(c.context)

Optional

• Write a test for authenticated property
• Type assertion

[danielballan]

If we set this attribute inside __repr__ we'll have an odd behavior wherein it is only set (and updated) when __repr__ is called. Take a look at this:

In [1]: from tiled.client import from_profile

In [2]: client = from_profile('demo')

In [3]: 'authenticated' in dir(client.context)
Out[3]: False

In [4]: client.context  # Displaying the object calls `__repr__()` internally.
Out[4]: <Context (unauthenticated)>

In [5]: 'authenticated' in dir(client.context)
Out[5]: True

In #818 I suggested "exposing it as a property", i.e.

class Context:

    ...


    @property
    def authenticated(self):
        return ...

Thus, authenticated will always be defined, and the current value will be computed when the user checks it.

[danielballan]

In this test, we check whether the repr shows the Context being authenticated or not. This would be a good place to test the new authenticated property has the expected value.

tiled/tiled/_tests/test_authentication.py

Lines 66 to 94 in 48bb130

	def test_password_auth(enter_username_password, config):
	"""
	A password that is wrong, empty, or belonging to a different user fails.
	"""
	with Context.from_app(build_app_from_config(config)) as context:
	# Log in as Alice.
	with enter_username_password("alice", "secret1"):
	from_context(context)
	# Reuse token from cache.
	client = from_context(context)
	assert "authenticated as 'alice'" in repr(client.context)
	client.logout()
	assert "unauthenticated" in repr(client.context)
	# Log in as Bob.
	with enter_username_password("bob", "secret2"):
	client = from_context(context)
	assert "authenticated as 'bob'" in repr(client.context)
	client.logout()
	# Bob's password should not work for Alice.
	with pytest.raises(PasswordRejected):
	with enter_username_password("alice", "secret2"):
	from_context(context)
	# Empty password should not work.
	with pytest.raises(PasswordRejected):
	with enter_username_password("alice", ""):
	from_context(context)

And here for the API key case:

tiled/tiled/_tests/test_authentication.py

Lines 342 to 358 in 48bb130

	def test_api_key_activity(enter_username_password, config):
	"""
	Create and use an API. Verify that latest_activity updates.
	"""
	with Context.from_app(build_app_from_config(config)) as context:
	# Log in as user.
	with enter_username_password("alice", "secret1"):
	context.authenticate()
	# Make and use an API key. Check that latest_activity is not set.
	key_info = context.create_api_key()
	context.logout()
	assert key_info["latest_activity"] is None # never used
	context.api_key = key_info["secret"]
	assert "authenticated as 'alice'" in repr(context)
	assert "with API key" in repr(context)
	assert key_info["secret"][:8] in repr(context)
	assert key_info["secret"][8:] not in repr(context)

[nmaytan]

Tested and looks good!