Allow switch kwarg in refresh to switch to local charms

[jack-w-shaw]

Port forward #968

Check if the switch param is pointing to a local charm. If it is, run local_refresh as if it were provided

This is how the Juju CLI works, which was missing from pylibjuju. Replicate this.

Also push url parsing of switch kwarg behind a check if it's local charm, meaning we avoid the possibility where we attempt to parse a path as if it were a url

Resolves #963

QA Steps

tox -e integration -- tests/integration/test_application.py::test_refresh_charmhub_to_local

All CI tests need to pass.

Verify the following example script runs successfully:

from juju import jasyncio
from juju.model import Model


async def main():
    model = Model()
    print('Connecting to model')
    await model.connect()
    try:
        print('path="/home/jack/charms/ubuntu"')
        await depl(model, path="/home/jack/charms/ubuntu")
        print('switch="/home/jack/charms/ubuntu"')
        await depl(model, switch="/home/jack/charms/ubuntu")
        print('switch="local:/home/jack/charms/ubuntu"')
        await depl(model, switch="local:/home/jack/charms/ubuntu")
    finally:
        print('Disconnecting from model')
        await model.disconnect()

async def depl(model, **kwargs):
    try:
        app = await model.deploy("ubuntu")
        await model.block_until(lambda: all(u[0].workload_status == 'active' for u in app.units))
        await app.refresh(**kwargs)
    finally:
        await app.remove()
        await model.block_until(lambda: not len(model.applications))

if __name__ == '__main__':
    jasyncio.run(main())

[cderici]

LGTM 👍

[jack-w-shaw]

NOTE: The one integration test failure is a timeout, which passes when I try locally. Merging anyway

[jack-w-shaw]

/merge

[jack-w-shaw]

/merge

[cderici]

The error is actually a new one that I haven't seen before.

raise AssertionError('Unable to add_machine in %s attempts with spec : %s -- exception was %s' % (attempt, spec, err))
AssertionError: Unable to add_machine in 3 attempts with spec : ssh:[email protected]:/home/runner/.local/share/juju/ssh/juju_id_rsa -- exception was Authentication failed: transport shut down or saw EOF

But I'm sure that it's not related to this change. This can safely land no problem. I'll see if that issue is a release blocker or not, thanks for pointing it out 👍

[jack-w-shaw]

/merge