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

[stable28] Implement tests #2562

Merged
merged 7 commits into from
Mar 21, 2024
Merged

[stable28] Implement tests #2562

merged 7 commits into from
Mar 21, 2024

Conversation

backportbot-libresign[bot]
Copy link

@backportbot-libresign backportbot-libresign bot commented Mar 21, 2024

Backport of #2554

Warning, This backport's changes differ from the original and might be incomplete ⚠️

Todo

  • Review and resolve any conflicts
  • Amend HEAD commit to remove the line stating to skip CI

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

Signed-off-by: Vitor Mattos <[email protected]>
Signed-off-by: Vitor Mattos <[email protected]>
I identified that using OpenSSL and CFSSL this field never is array

Signed-off-by: Vitor Mattos <[email protected]>
The vfsStream get crazy behavior when we run all tests. If we execute
a test isolated, will work fine, if run together with all, the file
don't will exstists. I identified that the dataProvider is started
before all and maybe other tests that use vfs is affecting this tests
and to prevent the problem, I created the vfs to each test.

Signed-off-by: Vitor Mattos <[email protected]>
@vitormattos vitormattos force-pushed the backport/2554/stable28 branch from aa8b753 to a531394 Compare March 21, 2024 17:59
@vitormattos vitormattos marked this pull request as ready for review March 21, 2024 18:00
@vitormattos vitormattos merged commit 83de827 into stable28 Mar 21, 2024
37 checks passed
@vitormattos vitormattos deleted the backport/2554/stable28 branch March 21, 2024 18:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

1 participant