Skip to content

Initial driver-kotlin-sync implementation #1090

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

Closed
wants to merge 16 commits into from
Closed

Initial driver-kotlin-sync implementation #1090

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
70f7694
Initial driver-kotlin-sync implementation
rozza Feb 20, 2023
b5d6846
Code review updates
rozza Feb 21, 2023
3c8bdbd
Added UnifiedCrudTest
rozza Feb 28, 2023
5f1e693
Make syncadapter classes internal
rozza Feb 28, 2023
47db59b
Update visibility modifiers for wrapped variables
rozza Feb 28, 2023
7fdcfdf
Simplify ClientSession
rozza Feb 28, 2023
d5bb8b5
Remove all remaining @since annotations
rozza Feb 28, 2023
d3a8036
Update MongoChangeStreamCursor kdoc
rozza Feb 28, 2023
6e02f84
Updated evergreen display name
rozza Feb 28, 2023
5160319
CrudTest back
rozza Feb 28, 2023
0e6bc56
PR updates - remove nullable & add toList
rozza Mar 8, 2023
95a0fce
Removed deprecated MapReduce support
rozza Mar 8, 2023
f9acb10
Kotlin coding convention: Prefer properties for cheap, non throwing f…
rozza Mar 9, 2023
0cda074
Sealed MongoCursor
rozza Mar 17, 2023
0520fc6
Seal MongoChangeStreamCursor
rozza Mar 17, 2023
f513ae8
Test the Implementation as that hasw the declared functions
rozza Mar 17, 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
24 changes: 24 additions & 0 deletions .evergreen/.evg.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -289,6 +289,16 @@ functions:
${PREPARE_SHELL}
SCALA="${SCALA}" AUTH="${AUTH}" SSL="${SSL}" MONGODB_URI="${MONGODB_URI}" SAFE_FOR_MULTI_MONGOS="${SAFE_FOR_MULTI_MONGOS}" TOPOLOGY="${TOPOLOGY}" JAVA_VERSION="${JAVA_VERSION}" .evergreen/run-scala-tests.sh

"run kotlin tests":
- command: shell.exec
type: test
params:
working_dir: "src"
script: |
${PREPARE_SHELL}
AUTH="${AUTH}" SSL="${SSL}" MONGODB_URI="${MONGODB_URI}" SAFE_FOR_MULTI_MONGOS="${SAFE_FOR_MULTI_MONGOS}" TOPOLOGY="${TOPOLOGY}" JAVA_VERSION="${JAVA_VERSION}" .evergreen/run-kotlin-tests.sh


"run socket tests":
- command: shell.exec
type: test
Expand Down Expand Up @@ -904,6 +914,11 @@ tasks:
- func: "bootstrap mongo-orchestration"
- func: "run scala tests"

- name: "kotlin-tests"
commands:
- func: "bootstrap mongo-orchestration"
- func: "run kotlin tests"

- name: "reactive-streams-tck-test"
commands:
- func: "bootstrap mongo-orchestration"
Expand Down Expand Up @@ -1437,6 +1452,8 @@ tasks:
name: "plain-auth-test"
- variant: ".test-scala-variant"
name: "scala-tests"
- variant: ".test-kotlin-variant"
name: "kotlin-tests"
- variant: ".tests-netty-variant"
name: "netty-test"
commands:
Expand Down Expand Up @@ -2052,6 +2069,13 @@ buildvariants:
tasks:
- name: "scala-tests"

- matrix_name: "kotlin-tests"
matrix_spec: { auth: "noauth", ssl: "nossl", jdk: ["jdk8", "jdk17"], version: ["6.0"], topology: "replicaset", os: "ubuntu" }
display_name: "Kotlin: ${jdk} ${version} ${topology} ${os}"
tags: ["test-kotlin-variant"]
tasks:
- name: "kotlin-tests"

- name: publish-snapshot
display_name: "Publish Snapshot"
run_on: ubuntu1804-test
Expand Down
37 changes: 37 additions & 0 deletions .evergreen/run-kotlin-tests.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,37 @@
#!/bin/bash

set -o xtrace # Write all commands first to stderr
set -o errexit # Exit the script with error if any of the commands fail


AUTH=${AUTH:-noauth}
SSL=${SSL:-nossl}
MONGODB_URI=${MONGODB_URI:-}
TOPOLOGY=${TOPOLOGY:-standalone}
SAFE_FOR_MULTI_MONGOS=${SAFE_FOR_MULTI_MONGOS:-}

############################################
# Main Program #
############################################
RELATIVE_DIR_PATH="$(dirname "${BASH_SOURCE:-$0}")"
. "${RELATIVE_DIR_PATH}/javaConfig.bash"


if [ "$SSL" != "nossl" ]; then
echo -e "\nSSL support not configured for Kotlin tests"
exit 1
fi

if [ "$AUTH" != "noauth" ]; then
echo -e "\nAuth support not configured for Kotlin tests"
exit 1
fi

if [ "$SAFE_FOR_MULTI_MONGOS" == "true" ]; then
export MULTI_MONGOS_URI_SYSTEM_PROPERTY="-Dorg.mongodb.test.multi.mongos.uri=${MONGODB_URI}"
fi

echo "Running Kotlin tests"

./gradlew -version
./gradlew :driver-kotlin-sync:kCheck -Dorg.mongodb.test.uri=${MONGODB_URI} ${MULTI_MONGOS_URI_SYSTEM_PROPERTY}
15 changes: 15 additions & 0 deletions config/detekt/baseline.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,15 @@
<?xml version='1.0' encoding='UTF-8'?>
<SmellBaseline>
<ManuallySuppressedIssues/>
<CurrentIssues>
<ID>IteratorNotThrowingNoSuchElementException:MongoCursor.kt$MongoCursor&lt;T : Any> : IteratorCloseable</ID>
<ID>LargeClass:MongoCollectionTest.kt$MongoCollectionTest</ID>
<ID>LongMethod:FindIterableTest.kt$FindIterableTest$@Suppress("DEPRECATION") @Test fun shouldCallTheUnderlyingMethods()</ID>
<ID>MaxLineLength:MapReduceIterable.kt$MapReduceIterable$*</ID>
<ID>SwallowedException:MockitoHelper.kt$MockitoHelper.DeepReflectionEqMatcher$e: Throwable</ID>
<ID>TooManyFunctions:ClientSession.kt$ClientSession : jClientSession</ID>
<ID>TooManyFunctions:FindIterable.kt$FindIterable&lt;T : Any> : MongoIterable</ID>
<ID>TooManyFunctions:MongoCollection.kt$MongoCollection&lt;T : Any></ID>
<ID>TooManyFunctions:MongoDatabase.kt$MongoDatabase</ID>
</CurrentIssues>
</SmellBaseline>
Loading