Project

General

Profile

Actions

Bug #2744

closed

Developer doc bug-fixes

Added by Tom Clegg almost 10 years ago. Updated almost 10 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
-
Category:
-
Story points:
1.0

Subtasks 5 (0 open5 closed)

Task #2696: Fix install doc bugsResolvedTom Clegg12/02/2013Actions
Task #2729: Building python egg should go in sdk/python/python-sdk.html and install/client.html should just tell you to use "pip install"ResolvedWard Vandewege12/02/2013Actions
Task #2737: Document "filters" parameterResolvedTom Clegg12/02/2013Actions
Task #2738: Call new log records "log entry" instead of "log" on log methods pageResolvedTom Clegg12/02/2013Actions
Task #2747: Review 2744-update-dev-docs branchResolvedMisha Zatsman12/02/2013Actions
Actions #1

Updated by Misha Zatsman almost 10 years ago

Looks good, just a few minor points:

test "search jobs by uuid with 'not in' query":
Can you provide multiple real uuids for the set which the uuids are not in? Otherwise this test looks just like the test that follows.
test "search jobs by output with '!= nil' query":
Are there any jobs with output = nil?
If not, then this test will always pass regardless of the correctness of the filter (unless it gets really weird), right?
If there are can you please add a test for output = nul as well?
Actions #2

Updated by Tom Clegg almost 10 years ago

Misha Zatsman wrote:

test "search jobs by uuid with 'not in' query":
Can you provide multiple real uuids for the set which the uuids are not in? Otherwise this test looks just like the test that follows.

Indeed. Fixed.

Are there any jobs with output = nil?
If there are can you please add a test for output = nul as well?

Yes. Rearranged tests as matched pairs of "=" and "!=", each with an assert_not_empty to ensure something actually has to get filtered out in the other.

Actions #3

Updated by Anonymous almost 10 years ago

  • Status changed from New to Resolved

Applied in changeset arvados|commit:8fcba3783d9a79bb18a00d339f9b3222f1eccd7f.

Actions

Also available in: Atom PDF