Skip to content
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.

Sign up for GitHub

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

Jump to bottom

test #3

Closed
wants to merge 66 commits into from
Closed

test #3

wants to merge 66 commits into from

Conversation

Jaehwa-Noh
Copy link
Owner

Thanks for submitting a pull request. Please include the following information.

What I have done and why
Include a summary of what your pull request contains, and why you have made these changes.

Fixes #<issue_number_goes_here>

Do tests pass?

  • Run local tests on DemoDebug variant: ./gradlew testDemoDebug
  • Check formatting: ./gradlew --init-script gradle/init.gradle.kts spotlessApply

Is this your first pull request?

dturner and others added 30 commits February 26, 2024 13:46
@dturner
Update Compose BOM to 2024.02.00. Fixes android#1218
7abcefd 
Change-Id: I838b81bf8e7fd7cb135f422653a8d8041829b28b
@dturner
Replace Accompanist TestHarness with DeviceConfigurationOverride
d6357be 
Change-Id: I6742d5e221f86a019f44d3e623609bf76d2a3a2a
@dturner @github-actions
🤖 Updates screenshots
50d5583
@dturner
Bump to ui-test:1.7.0-alpha03, add parameter name
fa811ab 
Change-Id: Ib1bbe4ad0d823c632d387587aa55c5eeb39e4b8d
@dturner
Workaround for new ForcedSize behaviour, bump ui-test version
73d0b70 
Change-Id: Ibf93c1b65250e6527e34bc90602c1b1001b2df46
@dturner @github-actions
🤖 Updates screenshots
43b2240
@dturner @alexvanyo
Update app/src/testDemo/kotlin/com/google/samples/apps/nowinandroid/u…
7737368 
…i/NiaAppScreenSizesScreenshotTests.kt

Co-authored-by: Alex Vanyo <[email protected]>
@dturner
Merge branch 'main' into replace-testharness
2d610b0 
* main: (37 commits)
  Disable animations of instrumented tests (android#1167)
  Bump gradle/wrapper-validation-action from 1 to 2
  Remove unused release variable. Fixes android#1231
  Apply Composable Parameter Ordering Guidelines
  Remove kotlinx-coroutines-guava dependency from :sync:work
  Add explicit guava-android dependency for :sync:work
  Extract ScreenshotHelper to separate testing module
  Update AGP to 8.3.0
  Remove Insert function - Insert is only used in test. - Upsert do same thing as Insert.
  🤖 Updates baselines for Dependency Guard
  Bump the kotlin-ksp-compose group with 6 updates
  🤖 Updates baselines for Dependency Guard
  Bump hilt from 2.50 to 2.51
  Bump com.google.truth:truth from 1.1.5 to 1.4.2
  Move java to kotlin folder.
  Remove disk usage testing
  Figuring out what is using 66Gb in the runner
  Dpm
  Improve converting to kotlin timezone
  Use trySend multiple times
  ...

Change-Id: If3f564108d42675ba55ef242f0d06f04aff45c4a
@sanao1006
[Refactor]: Directly call SearchContentsRepository interface in Searc…
825a5e2 
…hViewModel

- Remove GetSearchContentsCountUseCase.
- Remove GetSearchContentsCountUseCase from SearchViewModel.
- Remove GetSearchContentsCountUseCase from SearchViewModelTest.

Fixes android#1283
@dturner
Merge branch 'main' into remove-GetSearchContentsCountUseCase
51d3903
@Jaehwa-Noh
Rename fake to demo.
27ca860 
Change-Id: I5b1d7963e623832a766900aa74ef1085aac0cacc
@Jaehwa-Noh
Remove VisibleForTesting.
5d99f38 
Change-Id: Icd7005ec38495a1a4bc6f1594f70a2c1228538ab
@Jaehwa-Noh
Move main to demo.
a5be299 
Change-Id: I48bb3c26ef02d6302cb7caabcaa17d5b4659031a
@Jaehwa-Noh
Revert "Move main to demo."
2739168 
This reverts commit a5be299.
@Jaehwa-Noh
Change lint ignore name in lint.xml.
cd294a8 
Change-Id: I3b1e19dedd238ec96bd8debee5fcbcc91bc25094
@SimonMarquis
Delete unused combine extension in SyncUtilities.kt
ee8797f
@Jaehwa-Noh
Merge branch 'android:main' into rename
3849f10
@dependabot
Bump accompanist from 0.32.0 to 0.34.0
2e16bea 
Bumps `accompanist` from 0.32.0 to 0.34.0.

Updates `com.google.accompanist:accompanist-permissions` from 0.32.0 to 0.34.0
- [Release notes](https://github.com/google/accompanist/releases)
- [Commits](google/accompanist@v0.32.0...v0.34.0)

Updates `com.google.accompanist:accompanist-testharness` from 0.32.0 to 0.34.0
- [Release notes](https://github.com/google/accompanist/releases)
- [Commits](google/accompanist@v0.32.0...v0.34.0)

---
updated-dependencies:
- dependency-name: com.google.accompanist:accompanist-permissions
  dependency-type: direct:production
  update-type: version-update:semver-minor
- dependency-name: com.google.accompanist:accompanist-testharness
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
@dependabot @github-actions
🤖 Updates baselines for Dependency Guard
7d2026f
@Jaehwa-Noh
Change name Fake to Demo.
3c70183 
Change-Id: Ife321dd95238227bfdfc1f13f6832472bd699b6f
@Jaehwa-Noh
Merge remote-tracking branch 'origin/rename' into rename
97cdb6e 
Change-Id: I025fc00aab56c998c64bd88ad337cba8061d507c
@lihenggui
Ensure folder initialization and add instructions for Ubuntu users
3534a6f
@dturner
Merge pull request android#1305 from lihenggui/optimize_module_graph_…
d8566ad 
…script

Ensure folder setup and include instructions for Ubuntu users in generateModuleGraphs.sh
@yongsuk44
Remove unnecessary opt-in annotations
e2a4706
@yongsuk44
Remove empty primary constructors
af77295
@squart300kg
clean up di module in core.database package
d9b66c4
@dturner
Merge pull request android#1292 from SimonMarquis/patch-3
97dfe95 
Delete unused `combine` extension in `SyncUtilities.kt`
@dturner
Merge pull request android#1291 from Jaehwa-Noh/rename
ecdfdef 
Rename Fake to Demo
@dturner
Merge pull request android#1302 from android/dependabot/gradle/accomp…
9fd90a6 
…anist-0.34.0

Bump accompanist from 0.32.0 to 0.34.0
@dturner
Merge pull request android#1306 from yongsuk44/feat/weak-warning
bd89507 
Cleaned up unnecessary '@Opt-in' annotations and removed empty constructors
dturner and others added 27 commits March 28, 2024 10:21
@dturner
Merge branch 'main' into replace-testharness
fc2146c
@dturner
Fix spotless
739dc00 
Change-Id: I98a9b9fa1de5cc0bb3a3caa7d795053fd784b238
@dturner @github-actions
🤖 Updates screenshots
0d6566b
@JoseAlcerreca
Adds screenshot tests for snackbar (android#1343)
16f45c0 
* Adds screenshot tests for snackbar

Change-Id: I81566e3301fb17e3f31cbc408e6cc3325eab5a12

* Small tweaks, RFR

Change-Id: Ia16db6e6f5bec2714081b5ceaeeba36b2ee26105

* 🤖 Updates screenshots

* Update NiaApp.kt

Adds modifier to NiaApp
@alexvanyo
Update usage of material3 adaptive to be edge-to-edge
0be0bc1 
Change-Id: I481c79db77bbdd657182f5355200f63e2355f40d
@alexvanyo
Update to material3-adaptive alpha10
9b4c22c 
Change-Id: I810c36323a35c445c22dd3876c52df0360a5cb48
@alexvanyo
Update badging
d452f22 
Change-Id: If6b1327554bc855baa86fd47519ca56e015b9d14
@alexvanyo
Merge pull request android#1282 from android/av/material3-adaptive-e2e
59ed7f4 
Update usage of material3 adaptive to be edge-to-edge
@dturner
Merge branch 'main' into replace-testharness
61cce4b
@dturner
Update Compose UI Test to 1.7.0-alpha06
6402d27 
Change-Id: I05eb2993c41574a1ec9973e8dca3290d31ffc21f
@dturner
Replace TestHarness with DeviceConfigurationOverride after merging main
c7d5cb4 
Change-Id: Ie49b67981929fd3932ce3da68571a6305cf67aef
@dturner
Merge pull request android#1230 from android/replace-testharness
b993536 
Replace Accompanist TestHarness with DeviceConfigurationOverride
@bentrengrove
Enable strong skipping
5be13a0
@alexvanyo
Update to navigation 2.7.7
91215c5 
Change-Id: Iaceea1fc216c899f7fac58e7f5eeaea4e1c10dbb
@bentrengrove
Merge pull request android#1381 from android/ben/strongskip
0ba430e 
Enable strong skipping
@keyboardsurfer @mlykotom @Jaehwa-Noh
Configure baseline profiles in the baselineProfile lambda (android#1060)
7980c1d 
* Configure baseline profiles in the baselineProfile lambda

* This is the correct and easiest way to set up baseline profiles correctly.
  * See b/313428246 for more context.
* Handle empty topics during baseline profile generation
* Revert to API 33 GMD and fail empty screen tests
* Rename withChildren to noChildren for better legibility
* Remove unused log tags
* Update benchmark and metrics versions
* 🤖 Updates baselines for Dependency Guard

---------

Co-authored-by: Tomáš Mlynarič <[email protected]>
Co-authored-by: Jaehwa Noh <[email protected]>
@JoseAlcerreca
Revert "Configure baseline profiles in the baselineProfile lambda" (a…
1da560f 
…ndroid#1398)

This reverts commit 7980c1d.
@JoseAlcerreca
Fixes flakiness in app's screenshot tests (android#1395)
ead3f49 
* Disables WM init in app's screenshot tests

Change-Id: Ib0994e744f49b53f58c0eebd52f80c0eba67bbed

* Spotless

Change-Id: Ic0212889354481433ce45ec4292ae7be9bd24b2e

* [CI] Uploads roborazzi reports if they fail

Change-Id: I5a25a7e156d3fcb72a396d3b503e62068a20b417

* Uses unused test module and enables LocalInspection

Change-Id: Iac36347dc3d702248d7515a74823e0bc0cbf2a2b

* Fixes timezone in snackbar screenshot tests

Change-Id: Ic517e8822218abeaddd3f8d2740b03e651d72d31

* Fixes conflict

Change-Id: Ia42fb78781d2469a608183594fc25c7c69f7e76c

* 🤖 Updates screenshots
@Jaehwa-Noh
test
7e070b9
@Jaehwa-Noh
Set verification mode.
e07c899 
Change-Id: Ic61948e43b25d2133c05765a9d69362f8111be50
@Jaehwa-Noh
Create verification-metadata.xml.
61795ab 
Change-Id: I0385ed412a52420e1679f9efcfaa66032e8dcec8
@Jaehwa-Noh
Revert "Create verification-metadata.xml."
b4026e0 
This reverts commit 61795ab.
@Jaehwa-Noh
Revert "Set verification mode."
2c92f9c 
This reverts commit e07c899.
@alexvanyo
Add workaround for state loss bug
068cee0 
Change-Id: I7ac4495c8e3ada419358688c61214c33c0d4040e
@dturner
Merge pull request android#1383 from android/av/navigation-2.7.7
afad1b9 
Update to navigation 2.7.7
@Jaehwa-Noh
Merge branch 'android-remote-main' into action-test
68add74 
Change-Id: Id381428dda58d87548097f8ebb3785da7b55e81b
@Jaehwa-Noh
Change plain text to add method.
86b0156 
Change-Id: I128b974d43780f70db9fb3a6357d11c0ebc1ddaa
@github-actions GitHub Actions
Copy link

github-actions bot commented May 3, 2024

Combined test coverage report

Overall Project 40.42% -0.24% 🍏
Files changed 83.85% 🍏

Module Coverage
search 61.38% 🍏
data 60.2% 🍏
work 37.91% 🍏
database 34.72% 🍏
network 33.87% -1% ❌
app 31.93% -1.06% 🍏
designsystem 31.46% 🍏
ui 30.26% 🍏
Files
Module File Coverage
search SearchViewModel.kt 62.97% 🍏
SearchScreen.kt 61.43% 🍏
data SyncUtilities.kt 84.39% 🍏
work StubSyncSubscriber.kt 33.33% 🍏
database DatabaseModule.kt 0% 🍏
DaosModule.kt 0% 🍏
network JvmUnitTestDemoAssetManager.kt 78.95% 🍏
DemoNiaNetworkDataSource.kt 59.28% -5.67% ❌
NetworkModule.kt 0% -9.45% ❌
app InterestsListDetailScreen.kt 91.95% 🍏
NiaApp.kt 83.56% -6.19% 🍏
designsystem NiaIcons.kt 80.22% 🍏
Chip.kt 58.34% 🍏
ui NewsResourceCard.kt 67.49% 🍏

@Jaehwa-Noh
Copy link
Owner Author

Test done.

@Jaehwa-Noh Jaehwa-Noh closed this Jul 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.