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

Integration tests #345

Merged
merged 31 commits into from
Nov 13, 2023
Merged

Integration tests #345

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
31 commits
Select commit Hold shift + click to select a range
1bf56b2
Introduced integration tests for JVM and MPP projects
mvicsokolova Sep 11, 2023
3653759
Minor fixups
mvicsokolova Oct 23, 2023
4e25e4e
WIP: minor review fixes
mvicsokolova Oct 24, 2023
e095162
Fix test names in MppProjectTest
mvicsokolova Nov 1, 2023
4820ad1
Unit tests WIP
mvicsokolova Nov 1, 2023
ccdbec7
Minor review fixes
mvicsokolova Nov 2, 2023
f67b99d
Added unit tests for Artifact and dependency checker
mvicsokolova Nov 2, 2023
f3b0170
Add all the native targets to the mpp-sample project config
mvicsokolova Nov 6, 2023
eed2954
Composite build setup:
mvicsokolova Nov 7, 2023
ecde8a9
fixup
mvicsokolova Nov 7, 2023
8e0fd64
Retrieve values of project properties from the global provider
mvicsokolova Nov 8, 2023
8197b01
Pass versions to sample projects via version catalogs
mvicsokolova Nov 8, 2023
7efebc6
Fix mavenTest problem WIP: added dependency KotlinCompile to local pu…
mvicsokolova Nov 8, 2023
0f4c4e5
mavenTest depends on `atomicfu:publishToMavenLocal` to be able to che…
mvicsokolova Nov 8, 2023
3fb1cce
Make functionalTest dependOn atomicfu publication
mvicsokolova Nov 8, 2023
80709ea
functionalTest should depend on the global publishToMavenLocal task
mvicsokolova Nov 9, 2023
7b99c56
Pass dependencies to the funcitonalTest sourceSet
mvicsokolova Nov 9, 2023
b74544f
Revert "Pass dependencies to the funcitonalTest sourceSet"
mvicsokolova Nov 9, 2023
3890abc
Revert the last changes and depend only on atomicfu-gradle-plugin
mvicsokolova Nov 9, 2023
8d31501
fixes
shanshin Nov 9, 2023
d00033b
atomicfu and atomicfu-transformer artifacts should also be published …
mvicsokolova Nov 10, 2023
48f93df
Minor fix: add new lines at the end of all files
mvicsokolova Nov 10, 2023
8118b85
Minor fix: in case the name of the group starts with a digit
mvicsokolova Nov 10, 2023
2abbb53
fixup
mvicsokolova Nov 10, 2023
b69c7d9
Update integration-testing/examples/mpp-sample/src/commonTest/kotlin/…
mvicsokolova Nov 10, 2023
ea02a32
Do not cache functionalTest outputs
mvicsokolova Nov 11, 2023
191ac44
Import kotlin.test.* to the test class in mpp-sample
mvicsokolova Nov 13, 2023
77121f8
Add kotlin-test dependency to commonMain sourceSet
mvicsokolova Nov 13, 2023
f4253f9
Remove kotlin-stdlib dependency
mvicsokolova Nov 13, 2023
1ba8e2b
Trying to fix unresoved kotlin.Any problem: changed the order of repo…
mvicsokolova Nov 13, 2023
f3af8ee
Final fix of the repo order in sample projects
mvicsokolova Nov 13, 2023
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
2 changes: 1 addition & 1 deletion integration-testing/examples/jvm-sample/build.gradle.kts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -19,9 +19,9 @@ plugins {
apply(plugin = "kotlinx-atomicfu")

repositories {
mavenLocal()
mavenCentral()
maven("https://maven.pkg.jetbrains.space/kotlin/p/kotlin/dev")
mavenLocal()
}

dependencies {
Expand Down
1 change: 1 addition & 0 deletions integration-testing/examples/mpp-sample/build.gradle.kts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,7 @@ apply(plugin = "kotlinx-atomicfu")

repositories {
mavenCentral()
maven("https://maven.pkg.jetbrains.space/kotlin/p/kotlin/dev")
Copy link
Contributor

Choose a reason for hiding this comment

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

Is there a reason to have these repos ordered differently in jvm- and mpp-sample projects?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

The problem with unresolved classes from kotlin-stdlib may've been caused by the maven local repository being corrupted by some other process, and it's priority caused gradle to search for stdlib classes there first. As gradle can not work properly with mavenLocal, it's better not to set it first in the repo list .

mavenLocal()
}

Expand Down