Bugzilla – Attachment 92930 Details for
Bug 23517
Add API route to update a holds priority
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 23517: (follow-up) More test cases
Bug-23517-follow-up-More-test-cases.patch (text/plain), 1.46 KB, created by
Josef Moravec
on 2019-09-18 04:41:37 UTC
(
hide
)
Description:
Bug 23517: (follow-up) More test cases
Filename:
MIME Type:
Creator:
Josef Moravec
Created:
2019-09-18 04:41:37 UTC
Size:
1.46 KB
patch
obsolete
>From 734c204e0cfa2d68e0e69226d9d50beee66c3051 Mon Sep 17 00:00:00 2001 >From: Tomas Cohen Arazi <tomascohen@theke.io> >Date: Mon, 16 Sep 2019 10:32:48 -0300 >Subject: [PATCH] Bug 23517: (follow-up) More test cases > >Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> > >Signed-off-by: Josef Moravec <josef.moravec@gmail.com> >--- > t/db_dependent/api/v1/holds.t | 10 +++++++++- > 1 file changed, 9 insertions(+), 1 deletion(-) > >diff --git a/t/db_dependent/api/v1/holds.t b/t/db_dependent/api/v1/holds.t >index 9bbe6c4d94..b0a5a191a7 100644 >--- a/t/db_dependent/api/v1/holds.t >+++ b/t/db_dependent/api/v1/holds.t >@@ -348,7 +348,7 @@ subtest 'suspend and resume tests' => sub { > > subtest 'PUT /holds/{hold_id}/priority tests' => sub { > >- plan tests => 8; >+ plan tests => 14; > > $schema->storage->txn_begin; > >@@ -429,5 +429,13 @@ subtest 'PUT /holds/{hold_id}/priority tests' => sub { > is( $hold_2->discard_changes->priority, 3, 'Priority adjusted correctly' ); > is( $hold_3->discard_changes->priority, 1, 'Priority adjusted correctly' ); > >+ $t->put_ok( "//$userid:$password@/api/v1/holds/" >+ . $hold_3->id >+ . "/priority" => json => 3 )->status_is(200)->json_is(3); >+ >+ is( $hold_1->discard_changes->priority, 1, 'Priority adjusted correctly' ); >+ is( $hold_2->discard_changes->priority, 2, 'Priority adjusted correctly' ); >+ is( $hold_3->discard_changes->priority, 3, 'Priority adjusted correctly' ); >+ > $schema->storage->txn_rollback; > }; >-- >2.11.0
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 23517
:
92588
|
92589
|
92590
|
92710
|
92711
|
92712
|
92836
|
92927
|
92928
|
92929
|
92930
|
92986
|
92987
|
92988
|
92989
|
93143