Skip to content

Conversation

aozherelyeva
Copy link
Contributor

  1. Added tests for serialization and deserialization of requests to Bedrock from different sub-providers;
  2. Added AWS_SECRET_KEY and AWS_ACCESS_KEY_ID env variables to the environment variables of integration tests' run;
  3. Add a test for simpleBedrockExecutor.

Type of the change

  • New feature
  • Bug fix
  • Documentation fix
  • New tests

Checklist for all pull requests

  • The pull request has a description of the proposed change
  • I read the Contributing Guidelines before opening the pull request
  • The pull request uses develop as the base branch
  • Tests for the changes have been added
  • All new and existing tests passed
Additional steps for pull requests adding a new feature
  • An issue describing the proposed change exists
  • The pull request includes a link to the issue
  • The change was discussed and approved in the issue
  • Docs have been added / updated

@aozherelyeva aozherelyeva added the tests Add new tests or modify existing label Jul 10, 2025
Copy link

github-actions bot commented Jul 10, 2025

Qodana for JVM

345 new problems were found

Inspection name Severity Problems
Check Kotlin and Java source code coverage 🔶 Warning 251
Missing KDoc for public API declaration 🔶 Warning 77
Unused import directive 🔶 Warning 13
Vulnerable imported dependency 🔶 Warning 4
@@ Code coverage @@
+ 63% total lines covered
7323 lines analyzed, 4648 lines covered
# Calculated according to the filters of your coverage tool

☁️ View the detailed Qodana report

Contact Qodana team

Contact us at [email protected]

aozherelyeva and others added 6 commits July 11, 2025 12:54
…-bedrock-client/src/jvmTest/kotlin/ai/koog/prompt/executor/clients/bedrock/modelfamilies/ai21/BedrockAI21JambaSerializationTest.kt

Co-authored-by: Konstantin Pavlov <[email protected]>
…-bedrock-client/src/jvmTest/kotlin/ai/koog/prompt/executor/clients/bedrock/modelfamilies/ai21/BedrockAI21JambaSerializationTest.kt

Co-authored-by: Konstantin Pavlov <[email protected]>
…-bedrock-client/src/jvmTest/kotlin/ai/koog/prompt/executor/clients/bedrock/modelfamilies/ai21/JambaDataModelsTest.kt

Co-authored-by: Konstantin Pavlov <[email protected]>
@aozherelyeva aozherelyeva force-pushed the zarechneva/aws-bedrock-tests branch from 8b64997 to 716fd7f Compare July 11, 2025 10:54
@aozherelyeva aozherelyeva merged commit e1ee08e into develop Jul 11, 2025
5 checks passed
@aozherelyeva aozherelyeva deleted the zarechneva/aws-bedrock-tests branch July 11, 2025 11:50
@Ololoshechkin Ololoshechkin mentioned this pull request Jul 14, 2025
14 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tests Add new tests or modify existing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants