Skip to content

Change "user" to "tester" in app distro javadocs #4810

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Mar 22, 2023

Conversation

lfkellogg
Copy link
Contributor

No description provided.

@google-oss-bot
Copy link
Contributor

1 Warning
⚠️ Did you forget to add a changelog entry? (Add the 'no-changelog' label to the PR to silence this warning.)

Generated by 🚫 Danger

@google-oss-bot
Copy link
Contributor

@lfkellogg lfkellogg requested a review from tagboola March 22, 2023 13:09
@github-actions
Copy link
Contributor

Unit Test Results

  36 files  +    8    36 suites  +8   1m 49s ⏱️ +25s
191 tests  -   85  191 ✔️  -   85  0 💤 ±0  0 ±0 
382 runs   - 170  382 ✔️  - 170  0 💤 ±0  0 ±0 

Results for commit e2e14a7. ± Comparison against base commit 1057863.

This pull request removes 276 and adds 191 tests. Note that renamed tests count towards both.
com.google.firebase.remoteconfig.FirebaseRemoteConfigSettingsTest ‑ toBuilder_withFieldsSet_buildsObjectWithFieldsSet
com.google.firebase.remoteconfig.FirebaseRemoteConfigTest ‑ activate2p_hasAbtExperiments_doesNotCallAbt
com.google.firebase.remoteconfig.FirebaseRemoteConfigTest ‑ activate2p_hasNoAbtExperiments_doesNotCallAbt
com.google.firebase.remoteconfig.FirebaseRemoteConfigTest ‑ activate_callToAbtFails_activateStillSucceeds
com.google.firebase.remoteconfig.FirebaseRemoteConfigTest ‑ activate_fileWriteFails_doesNotClearFetchedAndReturnsFalse
com.google.firebase.remoteconfig.FirebaseRemoteConfigTest ‑ activate_fireperfNamespace_freshFetchedConfigs_activatesAndClearsFetched
com.google.firebase.remoteconfig.FirebaseRemoteConfigTest ‑ activate_fireperfNamespace_noFetchedConfigs_returnsFalse
com.google.firebase.remoteconfig.FirebaseRemoteConfigTest ‑ activate_freshFetchedConfigs_activatesAndClearsFetched
com.google.firebase.remoteconfig.FirebaseRemoteConfigTest ‑ activate_getActivatedFailed_activatesAndClearsFetched
com.google.firebase.remoteconfig.FirebaseRemoteConfigTest ‑ activate_getFetchedFailed_returnsFalse
…
com.google.firebase.appdistribution.impl.AabUpdaterTest ‑ updateAppTask_emptyLocationHeader_setsDownloadFailure
com.google.firebase.appdistribution.impl.AabUpdaterTest ‑ updateAppTask_isNotRedirectResponse_setsDownloadFailure
com.google.firebase.appdistribution.impl.AabUpdaterTest ‑ updateAppTask_missingLocationHeader_setsDownloadFailure
com.google.firebase.appdistribution.impl.AabUpdaterTest ‑ updateAppTask_onAppResume_setsUpdateCancelled
com.google.firebase.appdistribution.impl.AabUpdaterTest ‑ updateAppTask_whenAabReleaseAvailable_redirectsToPlay
com.google.firebase.appdistribution.impl.AabUpdaterTest ‑ updateAppTask_whenOpenConnectionFails_setsNetworkFailure
com.google.firebase.appdistribution.impl.AabUpdaterTest ‑ updateApp_whenCalledMultipleTimesWithAAB_onlyMakesOneRequest
com.google.firebase.appdistribution.impl.ApkInstallerTests ‑ installActivityDestroyed_setsInstallError
com.google.firebase.appdistribution.impl.ApkInstallerTests ‑ installApk_currentActivityNotNull_InstallIntentOnCurrentActivity
com.google.firebase.appdistribution.impl.ApkInstallerTests ‑ whenCalledMultipleTimes_onlyEmitsOneIntent
…

@google-oss-bot
Copy link
Contributor

@github-actions
Copy link
Contributor

Javadoc Changes:
--- /Users/runner/diff/original/firebase-kotlindoc/android/com/google/firebase/appdistribution/FirebaseAppDistribution.html	2023-03-22 13:16:21.000000000 +0000
+++ /Users/runner/diff/modified/firebase-kotlindoc/android/com/google/firebase/appdistribution/FirebaseAppDistribution.html	2023-03-22 13:11:07.000000000 +0000
@@ -186,7 +186,7 @@
               <tr>
                 <td><code>@<a href="https://developer.android.com/reference/kotlin/androidx/annotation/NonNull.html">NonNull</a> <a href="/docs/reference/android/com/google/firebase/appdistribution/InterruptionLevel.html">InterruptionLevel</a>&nbsp;interruptionLevel</code></td>
                 <td>
-                  <p>the level of interruption for the feedback notification. On platforms below Android 8, this corresponds to a <a href="https://developer.android.com/develop/ui/views/notifications/channels#importance">notification channel importance</a> and once set cannot be changed except by the user.</p>
+                  <p>the level of interruption for the feedback notification. On platforms below Android 8, this corresponds to a <a href="https://developer.android.com/develop/ui/views/notifications/channels#importance">notification channel importance</a> and once set cannot be changed except by the tester.</p>
                 </td>
               </tr>
             </tbody>
@@ -226,7 +226,7 @@
               <tr>
                 <td><code>@<a href="https://developer.android.com/reference/kotlin/androidx/annotation/NonNull.html">NonNull</a> <a href="/docs/reference/android/com/google/firebase/appdistribution/InterruptionLevel.html">InterruptionLevel</a>&nbsp;interruptionLevel</code></td>
                 <td>
-                  <p>the level of interruption for the feedback notification. On platforms below Android 8, this corresponds to a <a href="https://developer.android.com/develop/ui/views/notifications/channels#importance">notification channel importance</a> and once set cannot be changed except by the user.</p>
+                  <p>the level of interruption for the feedback notification. On platforms below Android 8, this corresponds to a <a href="https://developer.android.com/develop/ui/views/notifications/channels#importance">notification channel importance</a> and once set cannot be changed except by the tester.</p>
                 </td>
               </tr>
             </tbody>
--- /Users/runner/diff/original/firebase-kotlindoc/kotlin/com/google/firebase/appdistribution/FirebaseAppDistribution.html	2023-03-22 13:16:21.000000000 +0000
+++ /Users/runner/diff/modified/firebase-kotlindoc/kotlin/com/google/firebase/appdistribution/FirebaseAppDistribution.html	2023-03-22 13:11:07.000000000 +0000
@@ -186,7 +186,7 @@
               <tr>
                 <td><code>interruptionLevel:&nbsp;<a href="/docs/reference/kotlin/com/google/firebase/appdistribution/InterruptionLevel.html">InterruptionLevel</a></code></td>
                 <td>
-                  <p>the level of interruption for the feedback notification. On platforms below Android 8, this corresponds to a <a href="https://developer.android.com/develop/ui/views/notifications/channels#importance">notification channel importance</a> and once set cannot be changed except by the user.</p>
+                  <p>the level of interruption for the feedback notification. On platforms below Android 8, this corresponds to a <a href="https://developer.android.com/develop/ui/views/notifications/channels#importance">notification channel importance</a> and once set cannot be changed except by the tester.</p>
                 </td>
               </tr>
             </tbody>
@@ -226,7 +226,7 @@
               <tr>
                 <td><code>interruptionLevel:&nbsp;<a href="/docs/reference/kotlin/com/google/firebase/appdistribution/InterruptionLevel.html">InterruptionLevel</a></code></td>
                 <td>
-                  <p>the level of interruption for the feedback notification. On platforms below Android 8, this corresponds to a <a href="https://developer.android.com/develop/ui/views/notifications/channels#importance">notification channel importance</a> and once set cannot be changed except by the user.</p>
+                  <p>the level of interruption for the feedback notification. On platforms below Android 8, this corresponds to a <a href="https://developer.android.com/develop/ui/views/notifications/channels#importance">notification channel importance</a> and once set cannot be changed except by the tester.</p>
                 </td>
               </tr>
             </tbody>

@google-oss-bot
Copy link
Contributor

Startup Time Report 1

Note: Layout is sometimes suboptimal due to limited formatting support on GitHub. Please check this report on GCS.

Notes

Startup Times

  • fire-appdistribution

    DeviceStatisticsDistributions
    oriole-32
    Percentile10578631b16216DiffSignificant (?)
    p10559 ±307 μs936 ±645 μs+377 μs (+67.4%)NO
    p25607 ±315 μs975 ±671 μs+368 μs (+60.7%)NO
    p50656 ±327 μs1.04 ±0.7 ms+384 μs (+58.6%)NO
    p75803 ±333 μs1.21 ±0.8 ms+411 μs (+51.1%)NO
    p901.00 ±0.5 ms1.54 ±1 ms+532 μs (+53.0%)NO

    20 test runs in comparison
    CommitTest Runs
    1057863
    • 2023-03-22_01:02:04.783574_gONU
    • 2023-03-22_01:02:04.785762_psJl
    • 2023-03-22_01:02:04.785772_HoxS
    • 2023-03-22_01:02:04.785778_bVnO
    • 2023-03-22_01:02:04.785783_LZeD
    • 2023-03-22_01:02:04.785789_olVF
    • 2023-03-22_01:02:04.785795_idZl
    • 2023-03-22_01:02:04.785800_UXJl
    • 2023-03-22_01:02:04.785805_tuXI
    • 2023-03-22_01:02:04.785810_tlBC
    1b16216
    • 2023-03-22_13:20:05.937244_Hzls
    • 2023-03-22_13:20:05.943095_amzM
    • 2023-03-22_13:20:05.943111_JDTT
    • 2023-03-22_13:20:05.943117_tEJK
    • 2023-03-22_13:20:05.943277_smdX
    • 2023-03-22_13:20:05.943284_VOvv
    • 2023-03-22_13:20:05.943291_DZCF
    • 2023-03-22_13:20:05.943297_iukc
    • 2023-03-22_13:20:05.943302_FgMx
    • 2023-03-22_13:20:05.943308_qQxM
    redfin-30
    Percentile10578631b16216DiffSignificant (?)
    p10874 ±151 μs2.96 ±2 ms+2.08 ms (+238.3%)MAYBE
    p25907 ±172 μs3.11 ±2 ms+2.20 ms (+242.9%)MAYBE
    p50955 ±208 μs3.35 ±2 ms+2.40 ms (+250.8%)MAYBE
    p751.03 ±0.3 ms3.88 ±3 ms+2.84 ms (+274.9%)MAYBE
    p901.14 ±0.4 ms4.78 ±3 ms+3.63 ms (+317.7%)MAYBE

    20 test runs in comparison
    CommitTest Runs
    1057863
    • 2023-03-22_01:02:04.783574_gONU
    • 2023-03-22_01:02:04.785762_psJl
    • 2023-03-22_01:02:04.785772_HoxS
    • 2023-03-22_01:02:04.785778_bVnO
    • 2023-03-22_01:02:04.785783_LZeD
    • 2023-03-22_01:02:04.785789_olVF
    • 2023-03-22_01:02:04.785795_idZl
    • 2023-03-22_01:02:04.785800_UXJl
    • 2023-03-22_01:02:04.785805_tuXI
    • 2023-03-22_01:02:04.785810_tlBC
    1b16216
    • 2023-03-22_13:20:05.937244_Hzls
    • 2023-03-22_13:20:05.943095_amzM
    • 2023-03-22_13:20:05.943111_JDTT
    • 2023-03-22_13:20:05.943117_tEJK
    • 2023-03-22_13:20:05.943277_smdX
    • 2023-03-22_13:20:05.943284_VOvv
    • 2023-03-22_13:20:05.943291_DZCF
    • 2023-03-22_13:20:05.943297_iukc
    • 2023-03-22_13:20:05.943302_FgMx
    • 2023-03-22_13:20:05.943308_qQxM
  • timeToInitialDisplay

    DeviceStatisticsDistributions
    oriole-32
    Percentile10578631b16216DiffSignificant (?)
    p10200 ±23 ms197 ±8 ms-2.73 ms (-1.4%)NO
    p25213 ±38 ms204 ±9.7 ms-8.54 ms (-4.0%)NO
    p50226 ±46 ms213 ±11 ms-13.1 ms (-5.8%)NO
    p75239 ±55 ms222 ±13 ms-16.4 ms (-6.9%)NO
    p90252 ±67 ms238 ±16 ms-13.6 ms (-5.4%)NO

    20 test runs in comparison
    CommitTest Runs
    1057863
    • 2023-03-22_01:02:04.783574_gONU
    • 2023-03-22_01:02:04.785762_psJl
    • 2023-03-22_01:02:04.785772_HoxS
    • 2023-03-22_01:02:04.785778_bVnO
    • 2023-03-22_01:02:04.785783_LZeD
    • 2023-03-22_01:02:04.785789_olVF
    • 2023-03-22_01:02:04.785795_idZl
    • 2023-03-22_01:02:04.785800_UXJl
    • 2023-03-22_01:02:04.785805_tuXI
    • 2023-03-22_01:02:04.785810_tlBC
    1b16216
    • 2023-03-22_13:20:05.937244_Hzls
    • 2023-03-22_13:20:05.943095_amzM
    • 2023-03-22_13:20:05.943111_JDTT
    • 2023-03-22_13:20:05.943117_tEJK
    • 2023-03-22_13:20:05.943277_smdX
    • 2023-03-22_13:20:05.943284_VOvv
    • 2023-03-22_13:20:05.943291_DZCF
    • 2023-03-22_13:20:05.943297_iukc
    • 2023-03-22_13:20:05.943302_FgMx
    • 2023-03-22_13:20:05.943308_qQxM
    redfin-30
    Percentile10578631b16216DiffSignificant (?)
    p10231 ±4 ms254 ±7 ms+23.7 ms (+10.3%)MAYBE
    p25237 ±4 ms262 ±8 ms+25.3 ms (+10.7%)MAYBE
    p50244 ±4 ms271 ±8 ms+26.7 ms (+11.0%)MAYBE
    p75252 ±4 ms279 ±9 ms+27.4 ms (+10.9%)MAYBE
    p90260 ±5 ms291 ±11 ms+31.0 ms (+11.9%)MAYBE

    20 test runs in comparison
    CommitTest Runs
    1057863
    • 2023-03-22_01:02:04.783574_gONU
    • 2023-03-22_01:02:04.785762_psJl
    • 2023-03-22_01:02:04.785772_HoxS
    • 2023-03-22_01:02:04.785778_bVnO
    • 2023-03-22_01:02:04.785783_LZeD
    • 2023-03-22_01:02:04.785789_olVF
    • 2023-03-22_01:02:04.785795_idZl
    • 2023-03-22_01:02:04.785800_UXJl
    • 2023-03-22_01:02:04.785805_tuXI
    • 2023-03-22_01:02:04.785810_tlBC
    1b16216
    • 2023-03-22_13:20:05.937244_Hzls
    • 2023-03-22_13:20:05.943095_amzM
    • 2023-03-22_13:20:05.943111_JDTT
    • 2023-03-22_13:20:05.943117_tEJK
    • 2023-03-22_13:20:05.943277_smdX
    • 2023-03-22_13:20:05.943284_VOvv
    • 2023-03-22_13:20:05.943291_DZCF
    • 2023-03-22_13:20:05.943297_iukc
    • 2023-03-22_13:20:05.943302_FgMx
    • 2023-03-22_13:20:05.943308_qQxM

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

@lfkellogg lfkellogg merged commit 89bc90e into fad/next Mar 22, 2023
@lfkellogg lfkellogg deleted the lk/user-to-tester branch March 22, 2023 14:02
lfkellogg added a commit that referenced this pull request Apr 3, 2023
* Change "user" to "tester" in app distro javadocs (#4810)

* Fix notification on API level 19 (#4842)

* Update firebase-appdistribution{-api} CHANGELOG and gradle.properties for M130
@firebase firebase locked and limited conversation to collaborators Apr 22, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants