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

Fix mempool.space ios tests #622

Merged
merged 3 commits into from
Mar 27, 2024
Merged

Conversation

sstone
Copy link
Member

@sstone sstone commented Mar 26, 2024

These tests include https calls which won't work if the ios simulator is started in standalone mode, which is the default mode.

These tests include https calls which won't work if the ios simulator is started in standalone mode, which is the default mode.
@sstone sstone force-pushed the snapshot/fix-mempool-ios-tests branch from cc00511 to 3a3d8de Compare March 26, 2024 14:42
@sstone sstone marked this pull request as ready for review March 26, 2024 15:22
@sstone sstone requested review from pm47 and dpad85 March 26, 2024 15:22
@sstone
Copy link
Member Author

sstone commented Mar 26, 2024

See https://youtrack.jetbrains.com/issue/KT-38317 for a relevant issue, and the workaround I'm using here.

@pm47
Copy link
Member

pm47 commented Mar 26, 2024

Wow, the fix is pretty involved. For the CI it's probably fine, but I'm a bit worried about the implication on a normal dev machine, would such global calls not have side effects and break the environment? Let's wait for @robbiehanson's opinion

It is needed to run mempool.space tests but the fix may breaks things on developer's machine, here it's limited to github CI.
@sstone
Copy link
Member Author

sstone commented Mar 26, 2024

Wow, the fix is pretty involved. For the CI it's probably fine, but I'm a bit worried about the implication on a normal dev machine, would such global calls not have side effects and break the environment? Let's wait for @robbiehanson's opinion

There's another workaround in https://youtrack.jetbrains.com/issue/KT-61470/Kotlin-Native-Keychain-operations-fail-in-iOS-tests-due-to-missing-entitlements but it's even more complex... it also mentions that the underlying issue will be fixed in Kotlin 2.1.0.

I agree that this fix may break things on dev machine, so now it will only be used on github's CI,

@pm47
Copy link
Member

pm47 commented Mar 26, 2024

it will only be used on github's CI,

So the tests will keep failing on dev machine, that's right? There is no filtering to only run test on CI. I think it's good that way, just making sure.

@sstone
Copy link
Member Author

sstone commented Mar 26, 2024

So the tests will keep failing on dev machine, that's right? There is no filtering to only run test on CI. I think it's good that way, just making sure.

Yes they will run and fail on dev machines.

@robbiehanson
Copy link
Contributor

robbiehanson commented Mar 26, 2024

I can confirm that the various MempoolSpace tests fail on my machine (macOS) as expected. But all other tests succeed. And a publishToMavenLocal also works.

@sstone sstone merged commit cb35855 into fee-credit Mar 27, 2024
4 checks passed
@sstone sstone deleted the snapshot/fix-mempool-ios-tests branch March 27, 2024 08:01
pm47 pushed a commit that referenced this pull request Apr 30, 2024
* Fix mempool.space ios tests on github CI

These tests include https calls which won't work if the ios simulator is started in standalone mode, which is the default mode.
We modified our build and github CI workflow to run tests with `./gradlew -PiosSimulatorMode=standalone` and start the emulator "manually". 
Builds on dev machine remains the same and mempool.space will still fail on dev machine (unless users set `-PiosSimulatorMode=standalone` )

* Set version to 1.6.2-FEECREDIT-5
pm47 pushed a commit that referenced this pull request May 15, 2024
* Fix mempool.space ios tests on github CI

These tests include https calls which won't work if the ios simulator is started in standalone mode, which is the default mode.
We modified our build and github CI workflow to run tests with `./gradlew -PiosSimulatorMode=standalone` and start the emulator "manually". 
Builds on dev machine remains the same and mempool.space will still fail on dev machine (unless users set `-PiosSimulatorMode=standalone` )

* Set version to 1.6.2-FEECREDIT-5
pm47 pushed a commit that referenced this pull request May 15, 2024
* Fix mempool.space ios tests on github CI

These tests include https calls which won't work if the ios simulator is started in standalone mode, which is the default mode.
We modified our build and github CI workflow to run tests with `./gradlew -PiosSimulatorMode=standalone` and start the emulator "manually". 
Builds on dev machine remains the same and mempool.space will still fail on dev machine (unless users set `-PiosSimulatorMode=standalone` )

* Set version to 1.6.2-FEECREDIT-5
pm47 pushed a commit that referenced this pull request May 23, 2024
* Fix mempool.space ios tests on github CI

These tests include https calls which won't work if the ios simulator is started in standalone mode, which is the default mode.
We modified our build and github CI workflow to run tests with `./gradlew -PiosSimulatorMode=standalone` and start the emulator "manually". 
Builds on dev machine remains the same and mempool.space will still fail on dev machine (unless users set `-PiosSimulatorMode=standalone` )

* Set version to 1.6.2-FEECREDIT-5
pm47 pushed a commit that referenced this pull request May 31, 2024
* Fix mempool.space ios tests on github CI

These tests include https calls which won't work if the ios simulator is started in standalone mode, which is the default mode.
We modified our build and github CI workflow to run tests with `./gradlew -PiosSimulatorMode=standalone` and start the emulator "manually". 
Builds on dev machine remains the same and mempool.space will still fail on dev machine (unless users set `-PiosSimulatorMode=standalone` )

* Set version to 1.6.2-FEECREDIT-5
pm47 pushed a commit that referenced this pull request May 31, 2024
* Fix mempool.space ios tests on github CI

These tests include https calls which won't work if the ios simulator is started in standalone mode, which is the default mode.
We modified our build and github CI workflow to run tests with `./gradlew -PiosSimulatorMode=standalone` and start the emulator "manually".
Builds on dev machine remains the same and mempool.space will still fail on dev machine (unless users set `-PiosSimulatorMode=standalone` )
pm47 added a commit that referenced this pull request Jun 3, 2024
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.

3 participants