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.

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

[CI]Fix false positive diff in Roborazzi CI by using the same platform for store and compare #307

Merged
merged 2 commits into from
Apr 18, 2024

Conversation

takahirom
Copy link
Owner

No description provided.

@takahirom takahirom merged commit cc4a692 into main Apr 18, 2024
1 of 5 checks passed
@takahirom takahirom deleted the takahirom/fix-roborazzi-compare/2024-04-18 branch April 18, 2024 00:09
@takahirom takahirom changed the title Fix false positive diff in Roborazzi CI by using the same platform for store and compare [CI]Fix false positive diff in Roborazzi CI by using the same platform for store and compare Apr 18, 2024
Copy link

Snapshot diff report

File name Image
MainKmpTest.test_com
pare.png
MainKmpTest.test_2_c
ompare.png
MainJvmTest.test_com
pare.png
MainJvmTest.test_2_c
ompare.png
com.github.takahirom
.roborazzi.sample.Ma
nualTest.captureScre
enLevelNightWithEspr
esso_compare.png
manual_view_first_sc
reen_with_query_comp
ose_compare.png
manual_view_first_sc
reen_with_query_comp
ose_custom_compare.p
ng
com.github.takahirom
.roborazzi.sample.Ma
nualTest.captureScre
enLevelTabletWithEsp
resso_compare.png
custom_outputFilePro
vider-com.github.tak
ahirom.roborazzi.sam
ple.RuleTestWithPath
.captureRoboImageWit
hPath_compare.png
custom_outputFilePro
vider-com.github.tak
ahirom.roborazzi.sam
ple.RuleTestWithPath
.roboOutputNameTest_
compare.png
custom_file_compare.
png
custom_outputFilePro
vider-com.github.tak
ahirom.roborazzi.sam
ple.RuleTestWithPath
.captureRoboImage_co
mpare.png
com.github.takahirom
.roborazzi.sample.Co
mposeTest.roundTrans
parentResizeCompose_
2_compare.png
com.github.takahirom
.roborazzi.sample.Ma
nualTest.captureScre
enLevelImageWithEspr
esso_compare.png
com.github.takahirom
.roborazzi.sample.Co
mparisonStyleTest.di
ffNoLabel_compare.pn
g
manual_all_1_compare
.png
manual_view_without_
window_compare.png
custom-com.github.ta
kahirom.roborazzi.sa
mple.RuleTestWithAll
Image.captureRoboGif
Sample.4_compare.png
manual_view_first_sc
reen_with_query_view
_compare.png
manual_bitmap_compar
e.png
com.github.takahirom
.roborazzi.sample.Wi
ndowCaptureTest.comp
oseDialog_compare.pn
g
manual_all_3_compare
.png
com.github.takahirom
.roborazzi.sample.Sd
k25DontWorkScreensho
tTest.sdk25DontWorkS
creenshot_compare.pn
g
com.github.takahirom
.roborazzi.sample.Co
mposeTest.wearCompos
able_compare.png
com.github.takahirom
.roborazzi.sample.Wi
ndowCaptureTest.comp
oseBottomSheet_compa
re.png
com.github.takahirom
.roborazzi.sample.Co
mparisonStyleTest.di
ffDefault_compare.pn
g
custom-com.github.ta
kahirom.roborazzi.sa
mple.RuleTestWithAll
Image.captureRoboGif
Sample.2_compare.png
manual_last_compare.
png
custom-com.github.ta
kahirom.roborazzi.sa
mple.RuleTestWithAll
Image.captureRoboGif
Sample.1_compare.png
com.github.takahirom
.roborazzi.sample.Co
mposeTest.wearCompos
able_2_compare.png
manual_compose_compa
re.png
com.github.takahirom
.roborazzi.sample.Ma
nualTest.captureScre
enWithMetadata_compa
re.png
com.github.takahirom
.roborazzi.sample.Co
mparisonStyleTest.si
mple_compare.png
manual_all_4_compare
.png
custom-com.github.ta
kahirom.roborazzi.sa
mple.RuleTestWithAll
Image.captureRoboGif
Sample.0_compare.png
manual_view_on_windo
w_compare.png
com.github.takahirom
.roborazzi.sample.Ru
leTestWithLastImage.
captureRoboGifSample
_compare.png
com.github.takahirom
.roborazzi.sample.Wi
ndowCaptureTest.dump
_compare.png
com.github.takahirom
.roborazzi.sample.Wi
ndowCaptureTest.andr
oidDialog_compare.pn
g
com.github.takahirom
.roborazzi.sample.Co
mposeTest.composable
_compare.png
com.github.takahirom
.roborazzi.sample.Wi
ndowCaptureTest.noDi
alog_compare.png
manual_all_2_compare
.png
com.github.takahirom
.roborazzi.sample.Fr
agmentActivityWindow
CaptureTest.bottomSh
eetDialog_compare.pn
g
com.github.takahirom
.roborazzi.sample.Co
mparisonStyleTest.di
ffNoSmallLineSpace_c
ompare.png
manual_all_0_compare
.png
com.github.takahirom
.roborazzi.sample.Ma
nualTest.captureScre
enLevelJapaneseWithE
spresso_compare.png
com.github.takahirom
.roborazzi.sample.Ma
nualTest.captureScre
enLevelImageWithEspr
essoAndScaleOptions_
compare.png
manual_small_view_bu
tton_compare.png
com.github.takahirom
.roborazzi.sample.Ma
nualTest.captureComp
oseImage_compare.png
com.github.takahirom
.roborazzi.sample.Co
mposeTest.roundTrans
parentResizeCompose_
compare.png
manual_view_a11y_dum
p_compare.png
custom-com.github.ta
kahirom.roborazzi.sa
mple.RuleTestWithAll
Image.captureRoboGif
Sample.3_compare.png
com.github.takahirom
.roborazzi.sample.Ma
nualTest.moveToNextP
ageWithEspresso_comp
are.png
com.github.takahirom
.roborazzi.sample.Co
mparisonStyleTest.di
ffNoBigLineSpace_com
pare.png
ios_button_compare.p
ng
ios_compare.png
com.github.takahirom
.roborazzi.sample.No
ComposeManualTest.ca
ptureRoboImageSample
_compare.png

takahirom added a commit that referenced this pull request Apr 18, 2024
github-merge-queue bot referenced this pull request in slackhq/circuit Apr 21, 2024
This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
|
[io.github.takahirom.roborazzi](https://togithub.com/takahirom/roborazzi)
| plugin | minor | `1.12.0` -> `1.13.0` |
|
[io.github.takahirom.roborazzi:roborazzi-junit-rule](https://togithub.com/takahirom/roborazzi)
| dependencies | minor | `1.12.0` -> `1.13.0` |
|
[io.github.takahirom.roborazzi:roborazzi-compose](https://togithub.com/takahirom/roborazzi)
| dependencies | minor | `1.12.0` -> `1.13.0` |
|
[io.github.takahirom.roborazzi:roborazzi](https://togithub.com/takahirom/roborazzi)
| dependencies | minor | `1.12.0` -> `1.13.0` |

---

> [!WARNING]
> Some dependencies could not be looked up. Check the Dependency
Dashboard for more information.

---

### Release Notes

<details>
<summary>takahirom/roborazzi (io.github.takahirom.roborazzi)</summary>

###
[`v1.13.0`](https://togithub.com/takahirom/roborazzi/releases/tag/1.13.0)

[Compare
Source](https://togithub.com/takahirom/roborazzi/compare/1.12.0...1.13.0)

##### Add experimental, minimal support for iOS in Compose Multiplatform

Roborazzi has supported Compose Desktop, but we hadn't yet supported
iOS, which could be a major use case for Compose Multiplatform.
Therefore, I have added support for iOS.
Now, you can record, compare, and verify just as you would with Android
support.

https://takahirom.github.io/roborazzi/compose-multiplatform.html#experimental-feature-ios-support


![ios_button_compare](https://togithub.com/takahirom/roborazzi/assets/1386930/2d7a8faf-f57f-4543-bade-b1aef82ed15a)

However, Roborazzi is fundamentally based on the JVM. Currently, we
offer only minimal features. If you are interested in helping to improve
these features, please take a look at these issues:

[https://github.com/takahirom/roborazzi/issues/302](https://togithub.com/takahirom/roborazzi/issues/302)
[https://github.com/takahirom/roborazzi/issues/305](https://togithub.com/takahirom/roborazzi/issues/305)

##### Changes Affecting Existing Support

To support iOS, we have made some dependency changes for Android and
Compose Desktop. I believe these changes will not affect existing
behavior. However, if you notice anything, please let me know.

All
Kotlin:
1.8.22 -> 1.9.21

dropbox/differ:
0.0.1 -> 0.0.2

##### Desktop Compose Version

Compose Multiplatform:
1.4.3 -> 1.6.1

##### What's Changed

- Add Minimal Compose Multiplatform iOS Support by
[@&#8203;takahirom](https://togithub.com/takahirom) in
[https://github.com/takahirom/roborazzi/pull/301](https://togithub.com/takahirom/roborazzi/pull/301)
- \[docs]Fix docs page by
[@&#8203;takahirom](https://togithub.com/takahirom) in
[https://github.com/takahirom/roborazzi/pull/303](https://togithub.com/takahirom/roborazzi/pull/303)
- \[docs]Fix docs by [@&#8203;takahirom](https://togithub.com/takahirom)
in
[https://github.com/takahirom/roborazzi/pull/304](https://togithub.com/takahirom/roborazzi/pull/304)
- Add roborazzi-painter ios file for building klib by
[@&#8203;takahirom](https://togithub.com/takahirom) in
[https://github.com/takahirom/roborazzi/pull/306](https://togithub.com/takahirom/roborazzi/pull/306)
- \[CI]Fix false positive diff in Roborazzi CI by using the same
platform for store and compare by
[@&#8203;takahirom](https://togithub.com/takahirom) in
[https://github.com/takahirom/roborazzi/pull/307](https://togithub.com/takahirom/roborazzi/pull/307)
- Remove unneeded jvm form compose ios by
[@&#8203;takahirom](https://togithub.com/takahirom) in
[https://github.com/takahirom/roborazzi/pull/308](https://togithub.com/takahirom/roborazzi/pull/308)
- Fix task and fix ios recording by
[@&#8203;takahirom](https://togithub.com/takahirom) in
[https://github.com/takahirom/roborazzi/pull/310](https://togithub.com/takahirom/roborazzi/pull/310)
- Fix iOS recording path by
[@&#8203;takahirom](https://togithub.com/takahirom) in
[https://github.com/takahirom/roborazzi/pull/311](https://togithub.com/takahirom/roborazzi/pull/311)
- Fix iOS golden path and report time logic by
[@&#8203;takahirom](https://togithub.com/takahirom) in
[https://github.com/takahirom/roborazzi/pull/312](https://togithub.com/takahirom/roborazzi/pull/312)
- Fix iOS comparison logic by
[@&#8203;takahirom](https://togithub.com/takahirom) in
[https://github.com/takahirom/roborazzi/pull/313](https://togithub.com/takahirom/roborazzi/pull/313)

**Full Changelog**:
takahirom/roborazzi@1.12.0...1.13.0

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about these
updates again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Renovate
Bot](https://togithub.com/renovatebot/renovate).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4zMTMuMSIsInVwZGF0ZWRJblZlciI6IjM3LjMxMy4xIiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6W119-->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant