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

Enable random_channel_buffer_tests to run on Linux #10863

Merged
merged 2 commits into from May 1, 2024

Conversation

kain88-de
Copy link
Contributor

@kain88-de kain88-de commented Apr 22, 2024

With this patch applied I can see the following line in the test output:

test tests::random_channel_buffer_tests::test_random_channel_buffers has been running for over 60 seconds

Without it I run in a failure with 'Invalid keystroke cmk-,

Release Notes:

  • N/A

Copy link

cla-bot bot commented Apr 22, 2024

We require contributors to sign our Contributor License Agreement, and we don't have @testmaxli on file. You can sign our CLA at https://zed.dev/cla. Once you've signed, post a comment here that says '@cla-bot check'.

@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Apr 22, 2024
@kain88-de
Copy link
Contributor Author

When do you plan to run the unit tests on linux?

@maxdeviant
Copy link
Member

When do you plan to run the unit tests on linux?

Probably as we get closer to a full Linux release.

I'm not sure what the current state of the tests is.

kain88-de and others added 2 commits May 1, 2024 21:13
With this patch applied I can see the following line in the test output:

   test tests::random_channel_buffer_tests::test_random_channel_buffers has been running for over 60 seconds

Without it I run in a failure wit 'Invalid keystroke `cmk-,`
Co-authored-by: Marshall Bowers <elliott.codes@gmail.com>
@kain88-de
Copy link
Contributor Author

Currently 38 tests fail with this patch applied. I assume most of those are similar issues. I can work a bit on fixing them. I would feel better about that if the CI runs the tests to ensure they are not broken again in the future. Ideally all new test would also pass on Linux.

However, do you want this at the moment?

@maxdeviant
Copy link
Member

Currently 38 tests fail with this patch applied. I assume most of those are similar issues. I can work a bit on fixing them. I would feel better about that if the CI runs the tests to ensure they are not broken again in the future. Ideally all new test would also pass on Linux.

However, do you want this at the moment?

I'm not sure that we're ready to enable Linux tests in CI quite yet.

@maxdeviant maxdeviant merged commit dad3cbb into zed-industries:main May 1, 2024
8 checks passed
luckydye pushed a commit to luckydye/zed that referenced this pull request May 2, 2024
)

With this patch applied I can see the following line in the test output:

test tests::random_channel_buffer_tests::test_random_channel_buffers has
been running for over 60 seconds

Without it I run in a failure with 'Invalid keystroke `cmk-,`



Release Notes:

- N/A

---------

Co-authored-by: Max Linke <kain88-de@users.noreply.github.com>
Co-authored-by: Marshall Bowers <elliott.codes@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed The user has signed the Contributor License Agreement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants