Adds a check to pub tool to see if a root is a Flutter project.

[gspencergoog]

Description

This adds a check in the pub tool that checks to see if a project is a Flutter project before it calls the pub tool. If it is, then it automatically runs 'flutter pub' instead of 'dart pub'.

This required that tests write to the filesystem, so I added a FileSystemSupport mixin so that the test harness could provide a MemoryFileSystem instead of using a local (real) filesystem.

FileSystemSupport can't override out-of-process tests, of course, but it's useful for in-process tests.

Related Issues

• Allow the pub tool to conditionally use dart pub or flutter pub #81

Tests

• Added a test to make sure it conditionally runs flutter.

[github-actions]

PR Health

Changelog Entry ✔️

Package	Changed Files

Changes to files need to be accounted for in their respective changelogs.

[jakemac53]

I also did add this in #113 🤣 - I guess we need to get better at actually assigning issues to ourselves.

We might ultimately want some merged version of these two approaches? They aren't all that different.

[gspencergoog]

I also did add this in #113 🤣 - I guess we need to get better at actually assigning issues to ourselves.

LOL! Yeah, I guess so...

We might ultimately want some merged version of these two approaches? They aren't all that different.

Yes, let's merge them! I like your use of the pubspec parser instead of my generic YAML, and I think we still should have my FileSystemSupport mixin: it opens the door for more comprehensive in-process tests.

I went with a builder that built the entire command line instead of just the initial command, in case there were ever different options needed for a command based on the contents of the root.

Do you want to merge, or shall I (I'm fine with either one).

[jakemac53]

I can merge stuff from yours in (primarily the file system stuff), since mine also adds a new tool it is probably easier that way.

[gspencergoog]

Works for me.

[jakemac53]

Merged it in here c07112b

[gspencergoog]

Closing this PR, since it is superseded by #113