Add LinterCheck and LinterCheckRun

This commit is contained in:
Sebastian Serth
2020-10-26 21:08:43 +01:00
parent 66110fd57b
commit ee7729110b
4 changed files with 75 additions and 1 deletions

View File

@ -0,0 +1,5 @@
# frozen_string_literal: true
class LinterCheck < ApplicationRecord
has_many :linter_check_runs
end

View File

@ -0,0 +1,30 @@
# frozen_string_literal: true
class LinterCheckRun < ApplicationRecord
belongs_to :linter_check
belongs_to :testrun
belongs_to :file, class_name: 'CodeOcean::File'
def self.create_from(testrun, assessment)
assessment[:detailed_linter_results].each do |linter_result|
check = LinterCheck.find_or_create_by!(code: linter_result[:code]) do |new_check|
new_check.name = linter_result[:name]
new_check.severity = linter_result[:severity]
end
file = testrun.submission.file_by_name(linter_result[:file_name])
LinterCheckRun.create!(
linter_check: check,
result: linter_result[:result],
line: linter_result[:line],
scope: linter_result[:scope],
testrun: testrun,
file: file
)
end
rescue ActiveRecord::RecordInvalid
# Something bad happened. Probably, the RegEx in lib/py_lint_adapter.rb didn't work.
Raven.extra_context(testrun: testrun, linter_result: linter_result)
end
end

View File

@ -0,0 +1,19 @@
class CreateLinterChecks < ActiveRecord::Migration[5.2]
def change
create_table :linter_checks do |t|
t.string :name, null: false
t.string :code, null: false
t.string :severity
end
create_table :linter_check_runs do |t|
t.references :linter_check, null: false
t.string :scope
t.integer :line
t.text :result
t.references :testrun, null: false
t.references :file, null: false
t.timestamps
end
end
end

View File

@ -10,7 +10,7 @@
# #
# It's strongly recommended that you check this file into your version control system. # It's strongly recommended that you check this file into your version control system.
ActiveRecord::Schema.define(version: 2020_10_19_090123) do ActiveRecord::Schema.define(version: 2020_10_26_184633) do
# These are extensions that must be enabled in order to support this database # These are extensions that must be enabled in order to support this database
enable_extension "plpgsql" enable_extension "plpgsql"
@ -265,6 +265,26 @@ ActiveRecord::Schema.define(version: 2020_10_19_090123) do
t.datetime "updated_at" t.datetime "updated_at"
end end
create_table "linter_check_runs", force: :cascade do |t|
t.bigint "linter_check_id", null: false
t.string "scope"
t.integer "line"
t.text "result"
t.bigint "testrun_id", null: false
t.bigint "file_id", null: false
t.datetime "created_at", null: false
t.datetime "updated_at", null: false
t.index ["file_id"], name: "index_linter_check_runs_on_file_id"
t.index ["linter_check_id"], name: "index_linter_check_runs_on_linter_check_id"
t.index ["testrun_id"], name: "index_linter_check_runs_on_testrun_id"
end
create_table "linter_checks", force: :cascade do |t|
t.string "name", null: false
t.string "code", null: false
t.string "severity"
end
create_table "lti_parameters", id: :serial, force: :cascade do |t| create_table "lti_parameters", id: :serial, force: :cascade do |t|
t.integer "external_users_id" t.integer "external_users_id"
t.integer "consumers_id" t.integer "consumers_id"