Fix bug where name was not added to the manifest

[fgsch]

Use the default name if the user press enter at the name prompt.

[fgsch]

Not sure how I can write a test but haven't checked in depth yet.

[phamann]

Thanks for finding and fixing this issue @fgsch.

Also agree it would be good to get some test coverage on this. So if you don't mind can you add a test to the table in TestInit in pkg/compute/compute_integration_test.go. As a new temporary directory is dynamically created for each test it's probably easier just to match on the fastly-build prefix using the manifestIncludes property, so something like this should work:

		{
			name:       "with default name inferred from directory",
			args:       []string{"compute", "init"},
			configFile: config.File{Token: "123"},
			api: mock.API{
				GetTokenSelfFn:  tokenOK,
				GetUserFn:       getUserOk,
				CreateServiceFn: createServiceOK,
				CreateDomainFn:  createDomainOK,
				CreateBackendFn: createBackendOK,
			},
			wantOutput: []string{
				"Initializing...",
				"Fetching package template...",
				"Updating package manifest...",
			},
			manifestIncludes: `name = "fastly-build`,
		},

Hope that helps.

[phamann]

Would be good to keep the verbose output here.

[fgsch]

None of the other options that have defaults show this and the message is incorrect since this is not necessarily the name you will end up using so I think removing this is better.

[phamann]

Looks like you've got some failing tests after: 50360af

[fgsch]

Looks like you've got some failing tests after: 50360af

Yup, I will let you know when this is ready for another review. Until then please ignore this pr.

[fgsch]

Updated and ready for another review.

[phamann]

LGTM - thank you for your contribution