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

Add test with expected failure for #3503 #8280

Merged
merged 1 commit into from
Jul 21, 2021
Merged

Conversation

masih
Copy link
Member

@masih masih commented Jul 20, 2021

Write a sharness test that expects failure documenting the issue
raised in #3503. The issue may get resolved in the refactoring of
ipfs dag get command to support go-ipld-prime (e.g. #8171). For now
add a sharness test that expects failure. We will then flip the
expectation to success in the rewriting PR to check if #3503 gets
resolved.

Relates to:

Write a `sharness` test that expects failure documenting the issue
raised in #3503. The issue may get resolved in the refactoring of
`ipfs dag get` command to support go-ipld-prime (e.g. #8171). For now
add a `sharness` test that expects failure. We will then flip the
expectation in the success in the rewriting PR to check if #3503 gets
resolved.

Relates to:
- #3503
@Stebalien Stebalien merged commit 050a281 into master Jul 21, 2021
@Stebalien
Copy link
Member

Thanks!

@Stebalien Stebalien deleted the masih/test-issue-3503 branch July 21, 2021 02:47
@rvagg
Copy link
Member

rvagg commented Jul 22, 2021

to confirm, this is sorted out in feat/dag-put-ipld-prime including updated sharness tests in t0053-dag.sh which are similar to these

@aschmahmann aschmahmann mentioned this pull request Aug 23, 2021
62 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants