Revert "Remove In-App Feedback functionality from firebase-appdistribution (#3926)

[lfkellogg]

This reverts commit 78d2888, merging it into fad-in-app-feedback.

This will establish the new feature branch for developing the In-App Feedback feature.

[jladieu]

Nice!

[github-actions]

Unit Test Results

150 tests  +15   150 ✔️ +15   44s ⏱️ -2s
  17 suites +  3       0 💤 ±  0 
  17 files   +  3       0 ❌ ±  0 

Results for commit 60919ee. ± Comparison against base commit 78d2888.

[google-oss-bot]

Coverage Report ¹

Affected Products

• firebase-appdistribution

  Overall coverage changed from ? (78d2888) to 76.79% (feaea16) by ?.

  39 individual files with coverage change

  Filename	Base (78d2888)	Merge (feaea16)	Diff
  AabUpdater.java	?	98.68%	?
  ApkInstaller.java	?	96.88%	?
  ApkUpdater.java	?	93.75%	?
  AppDistributionReleaseImpl.java	?	100.00%	?
  AppDistributionReleaseInternal.java	?	100.00%	?
  AppIconSource.java	?	85.71%	?
  AutoValue_AppDistributionReleaseImpl.java	?	65.45%	?
  AutoValue_AppDistributionReleaseInternal.java	?	66.67%	?
  AutoValue_ImageUtils_ImageSize.java	?	30.00%	?
  AutoValue_TesterApiDisabledErrorDetails.java	?	29.41%	?
  AutoValue_TesterApiDisabledErrorDetails_HelpLink.java	?	54.17%	?
  AutoValue_UpdateProgressImpl.java	?	65.96%	?
  ErrorMessages.java	?	0.00%	?
  FeedbackActivity.java	?	0.00%	?
  FeedbackSender.java	?	90.91%	?
  FirebaseAppDistributionExceptions.java	?	80.00%	?
  FirebaseAppDistributionFileProvider.java	?	0.00%	?
  FirebaseAppDistributionImpl.java	?	92.79%	?
  FirebaseAppDistributionLifecycleNotifier.java	?	75.00%	?
  FirebaseAppDistributionNotificationsManager.java	?	80.00%	?
  FirebaseAppDistributionRegistrar.java	?	81.08%	?
  FirebaseAppDistributionTesterApiClient.java	?	87.63%	?
  HttpsUrlConnectionFactory.java	?	50.00%	?
  ImageUtils.java	?	95.45%	?
  InstallActivity.java	?	2.53%	?
  LogWrapper.java	?	53.33%	?
  NewReleaseFetcher.java	?	77.55%	?
  PackageInfoUtils.java	?	42.86%	?
  ReleaseIdentifier.java	?	88.00%	?
  ReleaseUtils.java	?	83.33%	?
  ScreenshotTaker.java	?	65.63%	?
  SignInResultActivity.java	?	0.00%	?
  SignInStorage.java	?	57.14%	?
  TaskUtils.java	?	96.30%	?
  TesterApiDisabledErrorDetails.java	?	93.75%	?
  TesterApiHttpClient.java	?	89.19%	?
  TesterSignInManager.java	?	93.62%	?
  UpdateProgressImpl.java	?	100.00%	?
  UpdateTaskImpl.java	?	75.71%	?

• firebase-appdistribution-api

  Overall coverage changed from ? (78d2888) to 62.62% (feaea16) by ?.

  11 individual files with coverage change

  Filename	Base (78d2888)	Merge (feaea16)	Diff
  AppDistributionRelease.java	?	0.00%	?
  BinaryType.java	?	0.00%	?
  FirebaseAppDistribution.java	?	0.00%	?
  FirebaseAppDistributionApiRegistrar.java	?	0.00%	?
  FirebaseAppDistributionException.java	?	76.92%	?
  FirebaseAppDistributionProxy.java	?	84.62%	?
  FirebaseAppDistributionStub.java	?	81.40%	?
  OnProgressListener.java	?	0.00%	?
  UpdateProgress.java	?	0.00%	?
  UpdateStatus.java	?	0.00%	?
  UpdateTask.java	?	100.00%	?

Test Logs

• Merge (feaea16): https://android-ci.firebaseopensource.com/view/gcs/android-ci/pr-logs/pull/firebase_firebase-android-sdk/3927/check-coverage-changed/1549819898543214593

Notes

• Commit (feaea16) is created by Prow via merging PR base commit (78d2888) and head commit (60919ee).
• Run gradle <product>:checkCoverage to produce HTML coverage reports locally. After gradle commands finished, report files can be found under <product-build-dir>/reports/jacoco/.

1. https://storage.googleapis.com/firebase-sdk-metric-reports/0A45bE6kjc.html

[google-oss-bot]

Size Report ¹

Affected Products

• base

  Type	Base (78d2888)	Merge (feaea16)	Diff
  apk (aggressive)	?	8.39 kB	? (?)
  apk (release)	?	8.65 kB	? (?)

• firebase-annotations

  Type	Base (78d2888)	Merge (feaea16)	Diff
  apk (aggressive)	?	8.39 kB	? (?)
  apk (release)	?	8.88 kB	? (?)

• firebase-appdistribution

  Type	Base (78d2888)	Merge (feaea16)	Diff
  aar	?	157 kB	? (?)
  apk (aggressive)	?	880 kB	? (?)
  apk (release)	?	2.01 MB	? (?)

• firebase-appdistribution-api

  Type	Base (78d2888)	Merge (feaea16)	Diff
  aar	?	14.5 kB	? (?)
  apk (aggressive)	?	85.5 kB	? (?)
  apk (release)	?	697 kB	? (?)

• firebase-common

  Type	Base (78d2888)	Merge (feaea16)	Diff
  aar	?	48.6 kB	? (?)
  apk (aggressive)	?	84.8 kB	? (?)
  apk (release)	?	690 kB	? (?)

• firebase-components

  Type	Base (78d2888)	Merge (feaea16)	Diff
  aar	?	41.4 kB	? (?)
  apk (aggressive)	?	8.68 kB	? (?)
  apk (release)	?	29.5 kB	? (?)

• firebase-installations

  Type	Base (78d2888)	Merge (feaea16)	Diff
  aar	?	55.0 kB	? (?)
  apk (aggressive)	?	86.4 kB	? (?)
  apk (release)	?	712 kB	? (?)

• firebase-installations-interop

  Type	Base (78d2888)	Merge (feaea16)	Diff
  aar	?	8.34 kB	? (?)
  apk (aggressive)	?	65.0 kB	? (?)
  apk (release)	?	651 kB	? (?)

Test Logs

• Merge (feaea16): https://android-ci.firebaseopensource.com/view/gcs/android-ci/pr-logs/pull/firebase_firebase-android-sdk/3927/binary-size/1549819898543214595

Notes

• Commit (feaea16) is created by Prow via merging PR base commit (78d2888) and head commit (60919ee).

1. https://storage.googleapis.com/firebase-sdk-metric-reports/haiHrDTwtF.html

[lfkellogg]

/test smoke-tests