remove obsolete test for a removed Setting
refs INTEROP-8632 Change-Id: Iff716d774f416c48fa2a034e922fa8c7f4d2a0b5 Reviewed-on: https://gerrit.instructure.com/c/canvas-lms/+/354256 Tested-by: Service Cloud Jenkins <svc.cloudjenkins@instructure.com> Reviewed-by: Xander Moffatt <xmoffatt@instructure.com> QA-Review: Xander Moffatt <xmoffatt@instructure.com> Product-Review: Steven McGee <steve.mcgee@instructure.com>
This commit is contained in:
parent
7c9c2e7aa0
commit
43fe6c9b05
|
@ -46,15 +46,6 @@ describe "page views" do
|
|||
expect(pv.http_method).to eq "get"
|
||||
end
|
||||
|
||||
it "does not record gets for api request when setting disabled" do
|
||||
Setting.set("create_get_api_page_views", "false")
|
||||
course_with_teacher(active_all: 1, user: user_with_pseudonym)
|
||||
@topic = @course.discussion_topics.create!
|
||||
expect do
|
||||
get "/api/v1/courses/#{@course.id}/discussion_topics/#{@topic.id}/entries", params: { access_token: @user.access_tokens.create!.full_token }
|
||||
end.not_to change(PageView, :count)
|
||||
end
|
||||
|
||||
it "records the developer key when an access token was used" do
|
||||
user_with_pseudonym(active_all: 1)
|
||||
course_with_teacher(active_all: 1, user: @user)
|
||||
|
|
Loading…
Reference in New Issue