v3 fix existing specs for un/star endpoints and starred filter
This commit is contained in:
parent
086932df11
commit
8d3acfe9bb
|
@ -16,9 +16,6 @@ module Travis::API::V3
|
||||||
end
|
end
|
||||||
|
|
||||||
def filter(list, user: nil)
|
def filter(list, user: nil)
|
||||||
p 'list'
|
|
||||||
p list.inspect
|
|
||||||
|
|
||||||
list = list.where(invalidated_at: nil)
|
list = list.where(invalidated_at: nil)
|
||||||
list = list.where(active: bool(active)) unless active.nil?
|
list = list.where(active: bool(active)) unless active.nil?
|
||||||
list = list.where(private: bool(private)) unless private.nil?
|
list = list.where(private: bool(private)) unless private.nil?
|
||||||
|
|
|
@ -4,7 +4,7 @@ module Travis::API::V3
|
||||||
paginate(default_limit: 100)
|
paginate(default_limit: 100)
|
||||||
|
|
||||||
def run!
|
def run!
|
||||||
unfiltered = query.for_owner(find(:owner), user: acccess_control.user)
|
unfiltered = query.for_owner(find(:owner), user: access_control.user)
|
||||||
access_control.visible_repositories(unfiltered)
|
access_control.visible_repositories(unfiltered)
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
|
@ -42,6 +42,8 @@ describe Travis::API::V3::Services::Repositories::ForOwner do
|
||||||
"read" => true,
|
"read" => true,
|
||||||
"enable" => false,
|
"enable" => false,
|
||||||
"disable" => false,
|
"disable" => false,
|
||||||
|
"star" => false,
|
||||||
|
"unstar" => false,
|
||||||
"create_request" => false},
|
"create_request" => false},
|
||||||
"id" => repo.id,
|
"id" => repo.id,
|
||||||
"name" => "minimal",
|
"name" => "minimal",
|
||||||
|
@ -59,7 +61,9 @@ describe Travis::API::V3::Services::Repositories::ForOwner do
|
||||||
"@type" => "branch",
|
"@type" => "branch",
|
||||||
"@href" => "/v3/repo/#{repo.id}/branch/master",
|
"@href" => "/v3/repo/#{repo.id}/branch/master",
|
||||||
"@representation" => "minimal",
|
"@representation" => "minimal",
|
||||||
"name" => "master"}}]}}
|
"name" => "master"},
|
||||||
|
"starred" => false
|
||||||
|
}]}}
|
||||||
end
|
end
|
||||||
|
|
||||||
describe "filter: private=false" do
|
describe "filter: private=false" do
|
||||||
|
|
Loading…
Reference in New Issue
Block a user