Merge branch 'master' into toshifumi/write-warning

Author: ShinyaKato

app/controllers/admin/contests_controller.rb

@@ -73,7 +73,7 @@ def update_from_json
         name_en: json[:contest_name] || @contest.name_en
       )
       @contest.problems.destroy_all
-      json[:problems].each.with_index(1) do |problem_json, i|
+      json[:problems].each.with_index do |problem_json, i|
         problem = Problem.create(
           contest_id: @contest.id,
           name_ja: problem_json[:title],
@@ -82,7 +82,7 @@ def update_from_json
           description_en: problem_json[:statement],
           order: i
         )
-        problem_json[:data_sets].each.with_index(1) do |data_set_json, j|
+        problem_json[:data_sets].each.with_index do |data_set_json, j|
           DataSet.create(
             problem_id: problem.id,
             label: data_set_json[:label],

app/models/problem.rb

@@ -1,6 +1,6 @@
 class Problem < ApplicationRecord
   belongs_to :contest
-  has_many :data_sets, dependent: :destroy
+  has_many :data_sets, -> { order(:order) }, dependent: :destroy
   has_many :submissions, through: :data_sets
 
   def name

spec/controllers/admin/contests_controller_spec.rb

@@ -105,10 +105,11 @@
         updated_contest = Contest.find_by(id: contest.id)
         expect(response).to redirect_to admin_contest_path(contest)
         expect(updated_contest.name_ja).to eq json[:contest_name]
-        expect(updated_contest.problems.size).to eq 1
-        expect(updated_contest.problems.first.name_ja).to eq 'A + B'
-        expect(updated_contest.problems.first.data_sets.size).to eq 1
-        expect(updated_contest.problems.first.data_sets.first.label).to eq 'Small'
+        expect(updated_contest.problems.size).to eq json[:problems].size
+        expect(updated_contest.problems.first.name_ja).to eq json[:problems].first[:title]
+        expect(updated_contest.problems.first.order).to eq 0
+        expect(updated_contest.problems.first.data_sets.size).to eq json[:problems].first[:data_sets].size
+        expect(updated_contest.problems.first.data_sets.first.label).to eq json[:problems].first[:data_sets].first[:label]
       end
     end
 

src/contests/Problem.tsx

@@ -134,10 +134,9 @@ export default class Problem extends React.Component<ProblemProps, ProblemState>
                     <textarea
                       className="problem--answer"
                       name="answer"
-                      placeholder={`このテキストボックスに解答（作成したプログラムにデータセットを入力して得られた実行結果）を貼り付けて、右下の提出ボタンを押してください。
-
-入力となるデータセットはフォームの右上のリンクからダウンロードできます。
-データセットは2種類あるので、SmallもしくはLargeを押して選択してから、ダウンロードしてください。`}
+                      placeholder={`このテキストボックスに解答（作成したプログラムにデータセットを入力して得られた実行結果）を貼り付けて、右下の提出ボタンを押してください。\n\n${
+                        this.props.problem.dataSets.length >= 2 ? `データセットの種類を選択してから、` : ``
+                      }入力となるデータをフォームの右上のリンクからダウンロードしてください。`}
                       value={this.selectedDataSet().answer}
                       rows={Math.max(6, Math.min(18, this.selectedDataSet().answer.split('\n').length))}
                       onChange={(e: React.ChangeEvent<HTMLTextAreaElement>): void => {