Update FixtureManipulator to work with InstalledPackagesList, real composer show command

Rough but I think it proves it is possible

@Wim Leers thanks for the review

This issue has only updated 1 validator to use `InstalledPackageList`

I think we may be able to bring in the changes to validators we made in this merge request we made in #3334994: Add new InstalledPackagesList which does not rely on Composer API to get package info in before reduced the scope of that issue to not update the validators. In that merge request we still had \Drupal\package_manager\ComposerInspector::getInstalledPackagesList and I think it was pretty much the same API we shouldn't have to do all that work over again.

I may look into do that if I address the other feedback I might unassign myself. But I think using the other MR should save alot of time because those changes already got some review

I tried to get the changes from the merge request in #7 but it was not so easy. That was on 2.x and other changes have happened since this. I think we should get this current issue in with 1 validator swapped to prove it works and then do other validators in other isssues

removing "attempt" since it seems we can do this

Unassigning myself.

relating the 2 related issues I created. 1 is fixed and #3344127: Run `composer validate` after FixtureManipulator commits its changes can be worked on separately

Uploaded patch without the changes to the validator to test performance changes of just the manipulator changes.

@Wim Leers if you want to take on working the new MR when you start your day please do. I am not sure I left it in good enough place where it would be obvious what the next steps are. If it is all nonsense I can take it back when I start tomorrow.

Sorry the code is a bit sloppy(altered drupalci.yml). I wanted to work quickly to see if this approach was feasible at rather than spend days before we figured out it wasn't

But we are down to a few tests failures so I think this approach will work and is better.

Thoughts on some of the remain failures.

1. FixtureManipulatorTest I expected this to fail because we are updating the FixtureManipulator. I fixed a couple but have not look at the others. I would not worry about \Drupal\Tests\package_manager\Kernel\FixtureManipulatorTest::testModifyPackage because hopefully we can fix the remain fails without using `modifyPackage()` and then we can remove it
2. OverwriteExistingPackagesValidatorTest which has edge cases where we set specific package install paths. Hard to do now but I have left a couple ideas on how to test this https://git.drupalcode.org/project/automatic_updates/-/merge_requests/74.... but also open for better ideas.
3. ComposerUtilityTest I think would a similar problem to FixtureManipulatorTest because we were relying setting install_path here also

So probably could be fixed the same way

4. ComposerPatchesValidatorTest I fixed a bunch of test cases but haven't looked at the last one

Create follow-up #3345222: Optimize Composer calls in FixtureManipulator . We will need a todo in the code

Assigning to @Wim Leers if you want to take a look.

Of the 3 failing testing in https://www.drupal.org/pift-ci-job/2606788. (I did not see later results yet)

1. DeleteExistingUpdateTest now passing for me locally. I expect it to pass. It was a problem that I had put an `composer update` call in the fixture we didn't need. This test hit it because it staged an update cancelled and retried.
2. FixtureManipulatorTest : I haven't had chance to look at these.
3. ComposerUtilityTest : I commented here https://git.drupalcode.org/project/automatic_updates/-/merge_requests/74... and started that idea. But I am done for the day.
   If you want to try, you are welcome to otherwise I will give it a try tomorrow.

UPDATE: see results from https://www.drupal.org/pift-ci-job/2607522 that InstalledPackagesListTest is now failing 🤦🏼
Here is commit with just a comment about this https://git.drupalcode.org/project/automatic_updates/-/merge_requests/74... different tests failed on this 1 line.

👋

re #24 @Wim Leers

Leaving those to @tedbow.

PRETTY PLEASE HAVE THIS COMMITTED by the time I wake up 🥺🥺🥺🥺

Assuming this passes tests I am going to commit this. But since I am the one also who do the last work
@Wim Leers
PRETTY PLEASE HAVE REVIEW THE CHANGES SINCE YOUR LAST COMMIT 😜😜😜😜😜😜😜

Beside getting the last tests, passing it has been a lot of comment with todos. the major new part is in OverwriteExistingPackagesValidatorTest look for // Confirm that meta-package will have an install path that is the same

@Wim Leers you also have commit access so you could revert it if need be.

Committed. Re #2 back to @Wim Leers

@Wim Leers thanks for the clean up patch! Committed