Rename rojo project during upload when wally package name doesn't match

[magnalite]

At the moment it is possible to publish a package which has a name differing from a name given in a Rojo default.project.json. This means Rojo will rename the package later on causing problems with linking. This PR aims to mitigate this by preventing users publishing packages with mismatched names and prompting them to rename.

It does so by checking if a default.project.json file exists in the project during publishing, and if it does, asserts the project name matches the package name given in the wally.toml.

This PR also adds:

• A test to verify publishing will stop when names are mismatched (and users are given an appropriate error).
• Another test to verify users are given an appropriate error when the names are fine (instead of a mismatched error).
• An extra global argument to allow tests to specify a temporary package index should be used. This caused problems when multiple publish tests were being run.
• The publish command now also accepts global arguments.
• Renames the wally-test-index url in the test packages to the new one (instead of the old archived one which 404s).

[LPGhatguy]

There are some good changes in this PR! I think that we should shift the titular feature. Instead of validating that the user's default.project.json matches, we should just fix it for them in the zip that we upload. This is important because we can't or don't want to change the default.project.json for many projects because it impacts the name of the model when built by Rojo, like for standalone releases!

[magnalite]

There are some good changes in this PR! I think that we should shift the titular feature. Instead of validating that the user's default.project.json matches, we should just fix it for them in the zip that we upload. This is important because we can't or don't want to change the default.project.json for many projects because it impacts the name of the model when built by Rojo, like for standalone releases!

Sounds like a good idea! I'll change this to modify the default.project.json during upload. I think we should still output a warning indicating that this is happening and recommend changing it to match if possible. Obviously that warning will not stop the upload, it'll just clarify what is happening.

[LPGhatguy]

I think we should still output a warning indicating that this is happening and recommend changing it to match if possible.

I don't know about this. For Roact, they probably want to keep the project name as Roact because this is important for rojo build. Changing the name to roact would be desirable for Wally but would break the normal releases.

I think we can have an info-level log message, but we shouldn't necessarily push people to change their Rojo projects.

[magnalite]

I think we can have an info-level log message, but we shouldn't necessarily push people to change their Rojo projects.

Sounds good to me. I've now changed this to detect+modify a default.project.json during creation of PackageContents and output a neutral info log. I'm not sure if this still counts as an integration test and I would have preferred to keep using the publish command but that felt like unnecessary complexity.