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

t0141: swap test_sort_cmp() arguments #1790

Closed
wants to merge 1 commit into from
Closed

Conversation

chriscool
Copy link
Contributor

License: MIT
Signed-off-by: Christian Couder chriscool@tuxfamily.org

Unverified

This commit is not signed, but one or more authors requires that any commit attributed to them is signed.
License: MIT
Signed-off-by: Christian Couder <chriscool@tuxfamily.org>
@jbenet jbenet added the status/in-progress In progress label Oct 4, 2015
@@ -27,7 +27,7 @@ test_swarm_filter_cmd() {
'

test_expect_success "'ipfs swarm filters' output looks good" '
test_sort_cmp list_actual list_expected
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This can be all killed in 1 commit.
From grep:

t0141-addfilter.sh
30:     test_sort_cmp list_actual list_expected

t0120-bootstrap.sh
37:    test_cmp list_actual list_expected
45:    test_cmp list2_actual list_expected
66:    test_cmp add_actual add_expected
78:    test_cmp rm_actual rm_expected
97:    test_cmp add2_actual add2_expected
116:    test_cmp rm2_actual rm2_expected

t0081-repo-pinning.sh
104:    test_cmp actual1 expected1
234:    test_cmp actual8 expected8

xt0130-multinode.sh
33: test_cmp actual1 expected1

x0045-add-cat-iptb.sh
33: test_cmp actual1 expected1

@jbenet
Copy link
Member

jbenet commented Oct 4, 2015

@chriscool @rht yeah let's do it all in one commit

@chriscool chriscool mentioned this pull request Oct 4, 2015
@chriscool
Copy link
Contributor Author

Ok, I replaced and completed this PR and the previous related PRs that are not already merged with PR #1792 (Fix test cmp args). The last commit in PR #1792 should swap all the test_*cmd args that should be swapped and other commits are various small cleanups.

@chriscool chriscool closed this Oct 4, 2015
@jbenet jbenet removed the status/in-progress In progress label Oct 4, 2015
@chriscool chriscool deleted the improve-t0141 branch April 30, 2016 05:47
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.

None yet

3 participants