Makes the course wizard not popup automatically on initial course creation.
Test Plan: - Create a course - Go to said course, no wizard should appear - Click the course checklist button, a wizard should appear Change-Id: Ie4672daa7068b518f1eb8dc0f1da29e3b1c1d8e4 Reviewed-on: https://gerrit.instructure.com/46979 Reviewed-by: Dan Minkevitch <dan@instructure.com> Tested-by: Jenkins <jenkins@instructure.com> QA-Review: Clare Strong <clare@instructure.com> Product-Review: Clay Diffrient <cdiffrient@instructure.com>
This commit is contained in:
parent
c69e8ff120
commit
fbec7fa7b5
|
@ -21,7 +21,10 @@ require [
|
|||
}), $wizard_box[0])
|
||||
)
|
||||
|
||||
setTimeout( ->
|
||||
if (!userSettings.get('hide_wizard_' + pathname))
|
||||
$(".wizard_popup_link.auto_open:first").click()
|
||||
, 500)
|
||||
# We are currently not allowing the wizard to popup automatically,
|
||||
# uncommenting the following code will re-enable that functionality.
|
||||
#
|
||||
# setTimeout( ->
|
||||
# if (!userSettings.get('hide_wizard_' + pathname))
|
||||
# $(".wizard_popup_link.auto_open:first").click()
|
||||
# , 500)
|
|
@ -77,6 +77,9 @@ describe "courses" do
|
|||
end
|
||||
|
||||
it "should properly hide the wizard and remember its hidden state" do
|
||||
# For now we are not allowing the wizard to popup automatically
|
||||
# so this spec doesn't apply, it may in the future though.
|
||||
pending
|
||||
course_with_teacher_logged_in
|
||||
|
||||
create_new_course
|
||||
|
@ -95,6 +98,9 @@ describe "courses" do
|
|||
end
|
||||
|
||||
it "should open and close wizard after initial close" do
|
||||
# For now we are not allowing the wizard to popup automatically
|
||||
# so this spec doesn't apply, it may in the future though.
|
||||
pending
|
||||
def find_wizard_box
|
||||
wizard_box = keep_trying_until do
|
||||
wizard_box = f(".ic-wizard-box")
|
||||
|
@ -129,9 +135,14 @@ describe "courses" do
|
|||
course_with_teacher_logged_in
|
||||
create_new_course
|
||||
|
||||
# Because of the specs about automatically opening are currently
|
||||
# pending, we need to cause the wizard to open by way of click. When
|
||||
# those specs are no longer pendings, the click line should be removed.
|
||||
f(".wizard_popup_link").click()
|
||||
wizard_box = f(".ic-wizard-box")
|
||||
keep_trying_until { expect(wizard_box).to be_displayed }
|
||||
|
||||
|
||||
f("#wizard_home_page").click
|
||||
f(".ic-wizard-box__message-button a").click
|
||||
wait_for_ajaximations
|
||||
|
|
Loading…
Reference in New Issue