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

Headless mode Kotlin #367

Open
wants to merge 10 commits into
base: headless-mode
Choose a base branch
from

Conversation

Hello-zoka
Copy link
Collaborator

@Hello-zoka Hello-zoka commented Sep 18, 2024

Added Kotlin support and method-level generation in the headless mode. Coverage collection has to be implemented in a separate PR(see #379)

Closes #337, #366

Progress:

  • Kotlin test generation and compilation support
  • Method-level generation support

@Hello-zoka Hello-zoka added improvement New feature Urgent Urgant PR labels Sep 18, 2024
@Hello-zoka Hello-zoka added the Ready for review PR redy for review label Oct 7, 2024
@Vladislav0Art Vladislav0Art added In progress PR is in progress and removed Ready for review PR redy for review Urgent Urgant PR labels Oct 7, 2024
@Vladislav0Art
Copy link
Collaborator

I marked it as "in progress" because the "Kotlin coverage collection" task in the description is not ticked.

@Hello-zoka
Copy link
Collaborator Author

Hello-zoka commented Oct 7, 2024

I marked it as "in progress."

I was sure that we'd discussed releasing Kotlin headless without coverage support since it requires some work to understand why Kotlin class files don't appear at all after compilation, which is required for such a task and probably will take a lot of efforts

So, this PR is marked ready for review again, but it doesn't close #366. I've modified it's description to show the current status and problems

@Hello-zoka Hello-zoka added Ready for review PR redy for review and removed In progress PR is in progress labels Oct 7, 2024
@Vladislav0Art
Copy link
Collaborator

I marked it as "in progress."

I was sure that we'd discussed releasing Kotlin headless without coverage support since it requires some work to understand why Kotlin class files don't appear at all after compilation, which is required for such a task and probably will take a lot of efforts

So, this PR is marked ready for review again, but it doesn't close #366. I've modified it's description to show the current status and problems

@Hello-zoka
Do you mean that we want to merge this PR without the coverage for Kotlin (in the upcoming release, I assume)? In this case, I recommend creating a separate issue just for coverage and linking these two issues together (mentioning one in the other as a continuation).

@Hello-zoka Hello-zoka added In progress PR is in progress and removed Ready for review PR redy for review labels Oct 21, 2024
@Hello-zoka Hello-zoka changed the base branch from development to headless-mode November 8, 2024 15:53
@Hello-zoka
Copy link
Collaborator Author

Current state:

  • Headless mode was separated from development into separate branch
  • Headless mode for Kotlin fails after migration to V2. We are working on finding an alternative solution
  • After some bug fixes in the development branch, coverage for Kotlin should work and close Coverage collection for Kotlin in Headless mode #379. It has to be checked after the migration problem has been fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement New feature In progress PR is in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Headless mode for Kotlin(except coverage) Implement headless mode for methods and lines
3 participants