Skip to content

chore: add debugging for failing test case, see COUCHDB-3413#518

Merged
janl merged 1 commit intomasterfrom
3413/test-debugging
May 16, 2017
Merged

chore: add debugging for failing test case, see COUCHDB-3413#518
janl merged 1 commit intomasterfrom
3413/test-debugging

Conversation

@janl
Copy link
Member

@janl janl commented May 14, 2017

Overview

Add test case to see what trips up this test case in COUCHDB 3413.

I assume it is one where the cluster returns a 500 error.

Testing recommendations

This needs to run in CI for a bit until it pops up again.

JIRA issue number

https://issues.apache.org/jira/browse/COUCHDB-3413

Checklist

  • Code is written and works correctly;
  • Changes are covered by tests;
  • Documentation reflects the changes;
  • I will not forget to update rebar.config.script
    with the correct commit hash once this PR get merged.

@nickva
Copy link
Contributor

nickva commented May 15, 2017

+1

At first I was about to mention that EUnit captures and hides all the stdout output but then noticed we send output directly to console with user in io:format. So that should work. However in general ?debugFmt is the recommended way to print during EUnit tests. Not big deal though just a minor nit.

@janl
Copy link
Member Author

janl commented May 15, 2017

re debugFmt same as in #517 (comment)

@nickva
Copy link
Contributor

nickva commented May 15, 2017

Alright, let's go with user then, minor nit anyway

@janl janl merged commit a2de7a0 into master May 16, 2017
@janl janl deleted the 3413/test-debugging branch May 16, 2017 07:37
nickva pushed a commit to nickva/couchdb that referenced this pull request Sep 7, 2022
The example result for _all_docs/queries did not match the example request and it was incorrect in a sense that the values `id` fields were not matching the respective values of `key` fields and there was a document returned that was not specified in the query.

This change fixes the issue described above by editing the ids so that they match the keys now and removing the element that was not specified in the request.
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.

2 participants