What you are doing is a very non-idempotent setup module, so there's
really no point to your usage of handlers.
I would find it much better to template out the /etc/apache2/.htdigestUser file.
I'd use the template module and remove the shell logic because without
a "creates=" the shell module is always going to return changed and if
you run it again, you're going to keep adding lines to that file,
which
seems quite undesirable.