Skip to content

test: Add unit, component and integration tests for kyma companion #3847

Merged
OliwiaGowor merged 26 commits intokyma-project:mainfrom
chriskari:ai-tests
Apr 18, 2025
Merged

test: Add unit, component and integration tests for kyma companion #3847
OliwiaGowor merged 26 commits intokyma-project:mainfrom
chriskari:ai-tests

Conversation

@chriskari
Copy link
Contributor

@chriskari chriskari commented Apr 9, 2025

Description

Changes proposed in this pull request:

  • added unit test for formatMarkdown functions
  • added component tests for Bubbles, Message, ErrorMessage, TasksList, CodePanel
  • added integration test for companion's main functionalities

Related issue(s)

Definition of done

  • The PR's title starts with one of the following prefixes:
    • feat: A new feature
    • fix: A bug fix
    • docs: Documentation only changes
    • refactor: A code change that neither fixes a bug nor adds a feature
    • test: Adding tests
    • revert: Revert commit
    • chore: Maintainance changes to the build process or auxiliary tools, libraries, workflows, etc.
  • Related issues are linked. To link internal trackers, use the issue IDs like backlog#4567
  • Explain clearly why you created the PR and what changes it introduces
  • All necessary steps are delivered, for example, tests, documentation, merging

@chriskari chriskari changed the title test: add unit, component and integration tests for kyma companion test: Add unit, component and integration tests for kyma companion Apr 9, 2025
@chriskari chriskari linked an issue Apr 9, 2025 that may be closed by this pull request
12 tasks
Copy link
Contributor

@mrCherry97 mrCherry97 left a comment

Choose a reason for hiding this comment

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

I'll suggest to separate companion integration test to separate files, for example kompanion-suggestions, companion-error-handling etc, because this file is giant and problematic to debug in the future

@OliwiaGowor OliwiaGowor enabled auto-merge (squash) April 18, 2025 05:50
@OliwiaGowor OliwiaGowor merged commit 9614d14 into kyma-project:main Apr 18, 2025
19 checks passed
@chriskari chriskari deleted the ai-tests branch June 19, 2025 19:28
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.

Add tests for AI Assistant

3 participants