New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
survey: Return survey results for single-page assessments too. #12299
Conversation
Is this meant to be |
LGTM but echoing Asher's comments - this should be against staging-next |
should there be a corresponding change to unit tests for this? |
@@ -360,6 +360,113 @@ def make_progress_in_section(script) | |||
assert_equal expected_response, JSON.parse(@response.body) | |||
end | |||
|
|||
test "should get surveys for section with script with single page anonymous level_group assessment" do |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👏
With the change to make level groups with pages "long assessments" even if they have a single page, we could no longer have a stage in allthethings.script that had two of these. The single page assessment is moved to a new stage. Also, some improvements were made to the tests to make them run more reliably on localhost with Chromedriver.
@@ -767,10 +767,11 @@ def generate_teacher_student(name, teacher_authorized) | |||
steps %Q{ | |||
Then I am on "http://code.org/teacher-dashboard#/sections" | |||
And I wait to see ".jumbotron" | |||
And I wait for 3 seconds |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Bjvanminnen I made a few changes that seemed to help here.
@@ -767,10 +767,11 @@ def generate_teacher_student(name, teacher_authorized) | |||
steps %Q{ | |||
Then I am on "http://code.org/teacher-dashboard#/sections" | |||
And I wait to see ".jumbotron" | |||
And I wait for 3 seconds | |||
And I click selector ".btn-white:contains('New section')" once I see it |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Bjvanminnen Any idea why a 3 second delay is needed despite having once I see it
here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nope.
No description provided.