-
Notifications
You must be signed in to change notification settings - Fork 32
Conversation
end | ||
it 'should display the options' do | ||
expect(page).to have_text("Grading: #{mission.title}") | ||
expect(page).to have_selector('.auto-grading-keyword', count: 2) |
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.
Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
8bc9eff
to
ded7d99
Compare
as_enum :auto_grading_type, none: 0, exact: 1, keyword: 2 | ||
|
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.
Line is too long. [112/80]
9c1c986
to
a4fa463
Compare
it 'should display the options' do | ||
expect(page).to have_text("Grading: #{mission.title}") | ||
expect(page).to have_selector('.auto-grading-exact', count: 2) | ||
expect(page).to have_selector('.auto-grading-keyword', count: 2) |
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.
Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
@@ -317,6 +344,11 @@ | |||
explanation 'Some explanation' | |||
end | |||
|
|||
factory :auto_grading_keyword_option, class: Assessment::AutoGradingKeywordOption do | |||
keyword 'keyword1' |
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.
Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
a4fa463
to
9766668
Compare
@@ -1,7 +1,7 @@ | |||
<%= simple_form_for @grading, url: (@grading.persisted? ? | |||
course_assessment_submission_grading_path(@course, @assessment, @submission, @grading) : | |||
course_assessment_submission_gradings_path(@course, @assessment, @submission)), | |||
html: { class: "form-horizontal" } do |f| %> | |||
htm: { class: "form-horizontal" } do |f| %> |
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.
Why changed to htm
?
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.
Mistake, my bad.
It is made to be less than the current max grade and positive. The validations go both ways, triggering when the max grade is changed, and when any score is changed. The latter is done because the score fields are dynamically created.
A number of forms have buttons that add associations and are rendered as links, due to `link_to_add_association` being called with incorrect parameters.
9766668
to
2b7c78d
Compare
Auto-grading with keyword answers
Implements feature 4 in Shuqun's feature requests. Depends on #427.