chore: update go-path #8436
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,79 @@ | ||
| package test | ||
|
|
||
| import ( | ||
| "context" | ||
| "strconv" | ||
| "testing" | ||
| "time" | ||
|
|
||
| files "github.com/ipfs/go-ipfs-files" | ||
| "github.com/ipfs/go-merkledag" | ||
| uio "github.com/ipfs/go-unixfs/io" | ||
| "github.com/ipfs/interface-go-ipfs-core/options" | ||
| "github.com/ipfs/interface-go-ipfs-core/path" | ||
| "github.com/ipld/go-ipld-prime" | ||
| ) | ||
|
|
||
|
|
||
| func TestPathUnixFSHAMTPartial(t *testing.T) { | ||
| ctx, cancel := context.WithCancel(context.Background()) | ||
| defer cancel() | ||
|
|
||
| // Create a node | ||
| apis, err := NodeProvider{}.MakeAPISwarm(ctx, true, 1) | ||
| if err != nil { | ||
| t.Fatal(err) | ||
| } | ||
| a := apis[0] | ||
|
|
||
| // Setting this after instantiating the swarm so that it's not clobbered by loading the go-ipfs config | ||
| prevVal := uio.UseHAMTSharding | ||
| uio.UseHAMTSharding = true | ||
| defer func() { | ||
| uio.UseHAMTSharding = prevVal | ||
| }() | ||
|
|
||
| // Create and add a sharded directory | ||
| dir := make(map[string]files.Node) | ||
| // Make sure we have at least two levels of sharding | ||
| for i := 0; i < uio.DefaultShardWidth + 1; i++ { | ||
| dir[strconv.Itoa(i)] = files.NewBytesFile([]byte(strconv.Itoa(i))) | ||
| } | ||
|
|
||
| r, err := a.Unixfs().Add(ctx, files.NewMapDirectory(dir), options.Unixfs.Pin(false)) | ||
| if err != nil { | ||
| t.Fatal(err) | ||
| } | ||
|
|
||
| // Get the root of the directory | ||
| nd, err := a.Dag().Get(ctx, r.Cid()) | ||
| if err != nil { | ||
| t.Fatal(err) | ||
| } | ||
|
|
||
| // Make sure the root is a DagPB node (this API might change in the future to account for ADLs) | ||
| _ = nd.(ipld.Node) | ||
| pbNode := nd.(*merkledag.ProtoNode) | ||
|
|
||
| // Remove one of the sharded directory blocks | ||
| if err := a.Block().Rm(ctx, path.IpfsPath(pbNode.Links()[0].Cid)); err != nil { | ||
| t.Fatal(err) | ||
| } | ||
|
|
||
| // Try and resolve each of the entries in the sharded directory which will result in pathing over the missing block | ||
| // | ||
| // Note: we could just check a particular path here, but it would require either greater use of the HAMT internals | ||
| // or some hard coded values in the test both of which would be a pain to follow. | ||
| for k := range dir { | ||
| // The node will go out to the (non-existent) network looking for the missing block. Make sure we're erroring | ||
| // because we exceeded the timeout on our query | ||
| timeoutCtx, timeoutCancel := context.WithTimeout(ctx, time.Second * 1) | ||
| _, err := a.ResolveNode(timeoutCtx, path.Join(r, k)) | ||
| if err != nil { | ||
| if timeoutCtx.Err() == nil { | ||
| t.Fatal(err) | ||
| } | ||
|
Comment on lines
+73
to
+75
There was a problem hiding this comment. This is one of the reasons I chose to do this instead of a sharness test (the other being the sharness tests are less portable which is overall just no fun). The equivalent sharness test would have to process text outputs like "context deadline exceeded" as good and "context cancelled" as bad which is a little too close for comfort and wouldn't detect other regressions where we returned "context deadline exceeded" in error. |
||
| } | ||
| timeoutCancel() | ||
| } | ||
| } | ||
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This makes the test take a couple of seconds to run, which is a bit annoying if we were going to run many of these although not so bad by itself.