[#139] test: Add test storage implementation #160

Closed
ale64bit wants to merge 1 commit from ale64bit/frostfs-node:fix/139-unit_test_storage into master
Member

This aims to reduce the usage of chmod hackery to induce or simulate
OS-related failures.

Signed-off-by: Alejandro Lopez a.lopez@yadro.com

This aims to reduce the usage of chmod hackery to induce or simulate OS-related failures. Signed-off-by: Alejandro Lopez <a.lopez@yadro.com>
ale64bit added 1 commit 2023-03-22 08:19:27 +00:00
[#139] test: Add test storage implementation
All checks were successful
ci/woodpecker/pr/pre-commit Pipeline was successful
3fb064649f
This aims to reduce the usage of chmod hackery to induce or simulate
OS-related failures.

Signed-off-by: Alejandro Lopez <a.lopez@yadro.com>
ale64bit requested review from storage-core-committers 2023-03-22 08:19:51 +00:00
ale64bit requested review from storage-core-developers 2023-03-22 08:19:51 +00:00
fyrchik reviewed 2023-03-23 06:14:52 +00:00
@ -67,1 +57,3 @@
t.Cleanup(func() { require.NoError(t, os.Chmod(dir, 0777)) })
t.Run("corrupt directory", func(t *testing.T) {
largeFileStorage.ExistsPassthrough.Store(false)
largeFileStorage.On("Exists", mock.Anything).Return(common.ExistsRes{}, teststore.ErrDiskExploded)
Owner

We try to avoid mocks in our codebase.

We try to avoid mocks in our codebase.
Author
Member

I see.

I see.
Author
Member

No point since mocks are avoided.

No point since mocks are avoided.
ale64bit closed this pull request 2023-03-23 06:55:31 +00:00
All checks were successful
ci/woodpecker/pr/pre-commit Pipeline was successful
Required
Details

Pull request closed

Sign in to join this conversation.
No reviewers
TrueCloudLab/storage-core-committers
TrueCloudLab/storage-core-developers
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: TrueCloudLab/frostfs-node#160
No description provided.