LockFileValidator should store the lock file hash in the stage's metadata, rather than directly in state

This is internal refactoring, so it doesn't need explicit test coverage. Assigning to @tedbow for review.

Looks good!

Are there other validators or stage event subscribers where we are using state where we could now use stage metadata?

I'm not sure, off the top of my head.

But, in this branch, a search of the Drupal\package_manager\Validator namespace for ->state turns up nothing. There's one use in Automatic Updates, but it's not for stage-specific metadata.

ok. thanks for checking

The change I needed to make to fix the tests necessitate a re-review.

Nice!

Tests failing