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

BE: Fix NPE on ptotobuf tests on Windows due to path separator mismatch #711

Open
4 tasks done
DimaVilda opened this issue Dec 15, 2024 · 3 comments
Open
4 tasks done
Assignees
Labels
scope/backend Related to backend changes status/blocked Further development process is blocked by something. Prevents merging for PRs. status/triage/completed Automatic triage completed type/chore Boring stuff, could be refactoring or tech debt

Comments

@DimaVilda
Copy link
Contributor

Issue submitter TODO list

  • I've looked up my issue in FAQ
  • I've searched for an already existing issues here
  • I've tried running main-labeled docker image and the issue still persists there
  • I'm running a supported version of the application which is listed here

Describe the bug (actual behavior)

  1. Run protobuf related tests on Windows OS ( i.e. ProtobufFileSerdeTest)
  2. See multiple NPEs on each test case related to incorrect 'language/language.proto' file location

Expected behavior

All tests should pass on Windows OS

Your installation details

Build and run current master branch.

Steps to reproduce

Build kafka-ui and run ProtobufFileSerdeTest. See NPE

Screenshots

Screenshot 2024-12-15 130433

Logs

No response

Additional context

No response

@DimaVilda DimaVilda added status/triage Issues pending maintainers triage type/bug Something isn't working labels Dec 15, 2024
@kapybro kapybro bot added status/triage/manual Manual triage in progress status/triage/completed Automatic triage completed and removed status/triage Issues pending maintainers triage labels Dec 15, 2024
Copy link

Hi DimaVilda! 👋

Welcome, and thank you for opening your first issue in the repo!

Please wait for triaging by our maintainers.

As development is carried out in our spare time, you can support us by sponsoring our activities or even funding the development of specific issues.
Sponsorship link

If you plan to raise a PR for this issue, please take a look at our contributing guide.

@DimaVilda
Copy link
Contributor Author

hey team
I'd like to be assigned to this issue since I already investigated the problem. However anybody can also take it also

@Haarolean Haarolean added scope/backend Related to backend changes type/chore Boring stuff, could be refactoring or tech debt and removed type/bug Something isn't working status/triage/manual Manual triage in progress labels Dec 15, 2024
@Haarolean Haarolean added the status/blocked Further development process is blocked by something. Prevents merging for PRs. label Dec 15, 2024
@Haarolean
Copy link
Member

Once square/wire#3215 is closed, a method from #723 can be nuked

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
scope/backend Related to backend changes status/blocked Further development process is blocked by something. Prevents merging for PRs. status/triage/completed Automatic triage completed type/chore Boring stuff, could be refactoring or tech debt
Projects
None yet
Development

No branches or pull requests

2 participants