[Tooling] Fix Wear app Google Play Console app prefix

[iangmaia]

Ref: https://developers.google.com/android-publisher/tracks#ff-track-name

Changing the Wear app track name to use wear: instead of the app name.

[dangermattic]

	1 Warning
⚠️	This PR is assigned to the milestone 19.2 ❄️. The due date for this milestone has already passed. Please assign it to a milestone with a later deadline or check whether the release for this milestone has already been finished.

Generated by 🚫 Danger

[wpmobilebot]

📲 You can test the changes from this Pull Request in WooCommerce-Wear Android by scanning the QR code below to install the corresponding build.

	App Name	WooCommerce-Wear Android
	Platform	⌚️ Wear OS
	Flavor	Jalapeno
	Build Type	Debug
	Commit	b8cec2d
	Direct Download	woocommerce-wear-prototype-build-pr11773-b8cec2d.apk

[wpmobilebot]

📲 You can test the changes from this Pull Request in WooCommerce Android by scanning the QR code below to install the corresponding build.

	App Name	WooCommerce Android
	Platform	📱 Mobile
	Flavor	Jalapeno
	Build Type	Debug
	Commit	b8cec2d
	Direct Download	woocommerce-prototype-build-pr11773-b8cec2d.apk

[AliSoftware]

I feel it would be better to UI.user_error! if the app is unknown there rather than have the track name ending up being unknown:beta and have the Google API fail with cryptic errors.

You could write the UI.user_error! in here for a quick fix, but I think it'd be better to use the get_app_param! (since that would DRY the error message from all calls site that checks that), after adjusting it so that it can accept the app value directly (so that it can support both lanes still using options and lanes using keyword-syntax parameters as its call sites); something like:

def get_app_param!(options)
  app = options.is_a?(String) ? options : options[:app]
  if [MOBILE_APP, WEAR_APP].include?(app)
    app
  else
    UI.user_error!("Invalid or missing `app:`. Please provide one of `app:#{MOBILE_APP}` or `app:#{WEAR_APP}`")
  end
end

[iangmaia]

Perhaps we could also extract get_app_param! validation so that it can be used separately?
To me it feels more natural than allowing options to be a String in some cases:

def validate_app_param!(app)
  unless [MOBILE_APP, WEAR_APP].include?(app)
    UI.user_error!("Invalid or missing `app:`. Please provide one of `app:#{MOBILE_APP}` or `app:#{WEAR_APP}`")
  end
end

def get_app_param!(options)
  validate_app_param!(options[:app])
  options[:app]
end

[AliSoftware]

Good idea, WFM.

Also once all our lanes will be converted to keywords params over time in the future, we'd then be able to just remove get_app_param! to only keep validate_app_param!, so that's nice to already do that separation 👍

[iangmaia]

Updated on b8cec2d