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

fix failed test #797

Open
wants to merge 12 commits into
base: main
Choose a base branch
from
+101 −47
Open

fix failed test #797

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
7a5f32c
introduce sourceSets
Yamada-Ika Aug 25, 2024
808eb1f
add test utility function for reading resource file
Yamada-Ika Aug 25, 2024
c829f31
enable passing tests when targeting iOS
Yamada-Ika Aug 25, 2024
7f6a43c
format
Yamada-Ika Aug 26, 2024
3d5a1c4
Merge branch 'main' into yamada-ika/fix-failed-test
Yamada-Ika Aug 26, 2024
c871954
Merge branch 'main' into yamada-ika/fix-failed-test
Yamada-Ika Aug 26, 2024
fceca5f
Merge branch 'main' into yamada-ika/fix-failed-test
Yamada-Ika Aug 27, 2024
3d3a17c
Merge branch 'main' into yamada-ika/fix-failed-test
Yamada-Ika Sep 1, 2024
4aad875
run test for iOS on CI
Yamada-Ika Sep 1, 2024
42e30e6
change machine type
Yamada-Ika Sep 1, 2024
1b0e1ae
merge test command
Yamada-Ika Sep 1, 2024
b431cc0
fix indent
Yamada-Ika Sep 1, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 6 additions & 0 deletions .github/workflows/unit-test.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,12 @@ jobs:

- run: ./gradlew testDevDebugUnitTest testDebugUnitTest --stacktrace

- run: |
./gradlew \
:core:data:iosSimulatorArm64Test \
:core:model:iosSimulatorArm64Test \
--stacktrace
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I apologize for delay. 🙇
There were another module for which iOS tests were not being run, so I specified them.
I am not similar with Gradle syntax, but is this acceptable?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! The iosSimulatorArm64Test might be skipped on ubuntu-latest. Could we change ubuntu-latest to macos-latest?
Additionally, we might need to modify the screenshot-comparison workflow to use macos-latest so that we have the same environment for the screenshots. I think we could merge the existing tests to speed up the process.

      - run: ./gradlew \
            testDevDebugUnitTest \
            testDebugUnitTest \
            :core:data:iosSimulatorArm64Test \
            :core:model:iosSimulatorArm64Test \
            --stacktrace

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for quick review! Fixed!
https://github.com/DroidKaigi/conference-app-2024/pull/797/files/3d3a17c7fce2823ec0333ccd02da600c69e50742..1b0e1aedfa7114ef281054b7271ea1416f51459b

Copy link
Contributor Author

@Yamada-Ika Yamada-Ika Sep 1, 2024 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, CI failed 🙇
I will try to fix


# Saving cost
# - run: ./gradlew koverHtmlReportDevDebug --stacktrace

Expand Down
Loading