TrueCloudLab/frostfs-node
20
1
Fork 27
Code Issues 101 Pull requests 4 Projects Releases 1 Activity Actions

[#668] shard/test: Do not alter rootPath option

Browse source 
Supposedly, this was added to allow creating 2 different shards without
subtest. Now we use t.TempDir() everywhere, so this should not be a
problem.

Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>
This commit is contained in:
Evgenii Stratonikov 2023-09-06 18:08:23 +03:00 committed by Evgenii Stratonikov
parent 36759f8434
commit a716db99db
1 changed files with 0 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
pkg/local_object_storage/shard/shard_test.go
View file

@ -54,7 +54,6 @@ func newCustomShard(t testing.TB, enableWriteCache bool, o shardOptions) *Shard
var sh *Shard var sh *Shard
if enableWriteCache { if enableWriteCache {
o.rootPath = filepath.Join(o.rootPath, "wc")
switch o.wcOpts.Type { switch o.wcOpts.Type {
case writecacheconfig.TypeBBolt: case writecacheconfig.TypeBBolt:
o.wcOpts.BBoltOptions = append( o.wcOpts.BBoltOptions = append(
@ -65,8 +64,6 @@ func newCustomShard(t testing.TB, enableWriteCache bool, o shardOptions) *Shard
[]writecachebadger.Option{writecachebadger.WithPath(filepath.Join(o.rootPath, "wcache"))}, []writecachebadger.Option{writecachebadger.WithPath(filepath.Join(o.rootPath, "wcache"))},
o.wcOpts.BadgerOptions...) o.wcOpts.BadgerOptions...)
} }
} else {
o.rootPath = filepath.Join(o.rootPath, "nowc")
} }
if o.bsOpts == nil { if o.bsOpts == nil {