This is not really the kind of style we use in page objects. Mostly page objects represent actions on the page (as these are UI tests we are primarily interested in actions on the page like tap, click, etc).
We also name these actions with a pattern, you see `tap_` for example throughout all of gaia_test.
So we kind of need to rename this into something like `<verb>_<object>_<html_element>` like `a11y_click_unlock_button()`  (I prefixed click with a11y to distinguish between a mouse click). Is 'a11y_click' common vernacular? I just used it as an example but it would be best to use whatever is the common term for a11y clicking in the same way we use tap for taps.

You can also make it return the Homescreen object, just like `unlock()` does, which means at the test level the next page you want to use is returned and you are ready to perform the next action or assertion at that point. That's most useful if you are going to be moving between several apps or pages during a test.