rubocop-issues.txt

Inspecting 151 files
..C..CWWWCC.WC.CWCCWCW..WCWWCCCCCWWCCCCCCCCCCCC..WCC..WCWCWCC.CCCWCCC....EC.CCCWCCCC.C..W................W..............W...........................CCC

Offenses:

app/controllers/admins_controller.rb:10:3: C: Assignment Branch Condition size for email_instructors is too high. [19.13/15]
  def email_instructors
  ^^^
app/controllers/admins_controller.rb:12:53: C: Do not use DateTime.[:now] without zone. Use #Time.zone.[:now] instead.
            .where("courses.end_date > ?", DateTime.now).where(instructor: true)
                                                    ^^^
app/controllers/application_controller.rb:9:1: C: Class definition is too long. [242/100]
class ApplicationController < ActionController::Base
^^^^^
app/controllers/application_controller.rb:43:3: C: Replace class var @@global_whitelist with a class instance var.
  @@global_whitelist = {}
  ^^^^^^^^^^^^^^^^^^
app/controllers/application_controller.rb:44:3: C: Assignment Branch Condition size for action_auth_level is too high. [22.23/15]
  def self.action_auth_level(action, level)
  ^^^
app/controllers/application_controller.rb:44:3: C: Cyclomatic complexity for action_auth_level is too high. [8/6]
  def self.action_auth_level(action, level)
  ^^^
app/controllers/application_controller.rb:44:3: C: Perceived complexity for action_auth_level is too high. [8/7]
  def self.action_auth_level(action, level)
  ^^^
app/controllers/application_controller.rb:102:3: C: Assignment Branch Condition size for authenticate_for_action is too high. [16.88/15]
  def authenticate_for_action
  ^^^
app/controllers/application_controller.rb:132:3: C: Assignment Branch Condition size for set_course is too high. [19.54/15]
  def set_course
  ^^^
app/controllers/application_controller.rb:132:3: C: Cyclomatic complexity for set_course is too high. [8/6]
  def set_course
  ^^^
app/controllers/application_controller.rb:132:3: C: Perceived complexity for set_course is too high. [8/7]
  def set_course
  ^^^
app/controllers/application_controller.rb:151:3: C: Assignment Branch Condition size for authorize_user_for_course is too high. [67.31/15]
  def authorize_user_for_course
  ^^^
app/controllers/application_controller.rb:151:3: C: Cyclomatic complexity for authorize_user_for_course is too high. [22/6]
  def authorize_user_for_course
  ^^^
app/controllers/application_controller.rb:151:3: C: Method has too many lines. [39/20]
  def authorize_user_for_course
  ^^^
app/controllers/application_controller.rb:151:3: C: Perceived complexity for authorize_user_for_course is too high. [21/7]
  def authorize_user_for_course
  ^^^
app/controllers/application_controller.rb:211:3: C: Assignment Branch Condition size for set_assessment is too high. [17.12/15]
  def set_assessment
  ^^^
app/controllers/application_controller.rb:211:3: C: Cyclomatic complexity for set_assessment is too high. [7/6]
  def set_assessment
  ^^^
app/controllers/application_controller.rb:227:3: C: Assignment Branch Condition size for set_submission is too high. [31.8/15]
  def set_submission
  ^^^
app/controllers/application_controller.rb:227:3: C: Cyclomatic complexity for set_submission is too high. [14/6]
  def set_submission
  ^^^
app/controllers/application_controller.rb:227:3: C: Perceived complexity for set_submission is too high. [14/7]
  def set_submission
  ^^^
app/controllers/application_controller.rb:250:3: C: Assignment Branch Condition size for run_scheduler is too high. [33.39/15]
  def run_scheduler
  ^^^
app/controllers/application_controller.rb:250:3: C: Method has too many lines. [21/20]
  def run_scheduler
  ^^^
app/controllers/application_controller.rb:251:48: C: Do not use Time.[:now] without zone. Use #Time.zone.[:now] instead.
    actions = Scheduler.where("next < ?", Time.now)
                                               ^^^
app/controllers/application_controller.rb:253:26: C: Do not use Time.[:now] without zone. Use #Time.zone.[:now] instead.
      action.next = Time.now + action.interval
                         ^^^
app/controllers/application_controller.rb:316:3: C: Assignment Branch Condition size for render_error is too high. [26.42/15]
  def render_error(exception)
  ^^^
app/controllers/application_controller.rb:316:3: C: Cyclomatic complexity for render_error is too high. [9/6]
  def render_error(exception)
  ^^^
app/controllers/application_controller.rb:316:3: C: Perceived complexity for render_error is too high. [9/7]
  def render_error(exception)
  ^^^
app/controllers/assessment/autograde.rb:13:3: C: Assignment Branch Condition size for autograde_done is too high. [32.89/15]
  def autograde_done
  ^^^
app/controllers/assessment/autograde.rb:13:3: C: Cyclomatic complexity for autograde_done is too high. [8/6]
  def autograde_done
  ^^^
app/controllers/assessment/autograde.rb:13:3: C: Perceived complexity for autograde_done is too high. [9/7]
  def autograde_done
  ^^^
app/controllers/assessment/autograde.rb:63:3: C: Assignment Branch Condition size for regradeBatch is too high. [28.34/15]
  def regradeBatch
  ^^^
app/controllers/assessment/autograde.rb:63:3: C: Cyclomatic complexity for regradeBatch is too high. [8/6]
  def regradeBatch
  ^^^
app/controllers/assessment/autograde.rb:63:3: C: Method has too many lines. [25/20]
  def regradeBatch
  ^^^
app/controllers/assessment/autograde.rb:63:3: C: Perceived complexity for regradeBatch is too high. [10/7]
  def regradeBatch
  ^^^
app/controllers/assessment/autograde.rb:63:7: C: Use snake_case for method names.
  def regradeBatch
      ^^^^^^^^^^^^
app/controllers/assessment/autograde.rb:76:64: W: Non-local exit from iterator, without return value. next, break, Array#find, Array#any?, etc. is preferred.
          redirect_to([@course, @assessment, :submissions]) && return
                                                               ^^^^^^
app/controllers/assessment/autograde.rb:87:101: C: Line is too long. [115/100]
    flash[:error] = "Warning: Could not regrade #{failed_jobs} submission(s):<br>" + failed_list if failed_jobs > 0
                                                                                                    ^^^^^^^^^^^^^^^
app/controllers/assessment/autograde.rb:104:3: C: Assignment Branch Condition size for regradeAll is too high. [28.34/15]
  def regradeAll
  ^^^
app/controllers/assessment/autograde.rb:104:3: C: Cyclomatic complexity for regradeAll is too high. [8/6]
  def regradeAll
  ^^^
app/controllers/assessment/autograde.rb:104:3: C: Method has too many lines. [26/20]
  def regradeAll
  ^^^
app/controllers/assessment/autograde.rb:104:3: C: Perceived complexity for regradeAll is too high. [10/7]
  def regradeAll
  ^^^
app/controllers/assessment/autograde.rb:104:7: C: Use snake_case for method names.
  def regradeAll
      ^^^^^^^^^^
app/controllers/assessment/autograde.rb:119:64: W: Non-local exit from iterator, without return value. next, break, Array#find, Array#any?, etc. is preferred.
          redirect_to([@course, @assessment, :submissions]) && return
                                                               ^^^^^^
app/controllers/assessment/autograde.rb:130:101: C: Line is too long. [115/100]
    flash[:error] = "Warning: Could not regrade #{failed_jobs} submission(s):<br>" + failed_list if failed_jobs > 0
                                                                                                    ^^^^^^^^^^^^^^^
app/controllers/assessment/autograde.rb:145:3: C: Assignment Branch Condition size for autogradeSubmissions is too high. [42.25/15]
  def autogradeSubmissions(course, assessment, submissions)
  ^^^
app/controllers/assessment/autograde.rb:145:3: C: Cyclomatic complexity for autogradeSubmissions is too high. [12/6]
  def autogradeSubmissions(course, assessment, submissions)
  ^^^
app/controllers/assessment/autograde.rb:145:3: C: Method has too many lines. [30/20]
  def autogradeSubmissions(course, assessment, submissions)
  ^^^
app/controllers/assessment/autograde.rb:145:3: C: Perceived complexity for autogradeSubmissions is too high. [14/7]
  def autogradeSubmissions(course, assessment, submissions)
  ^^^
app/controllers/assessment/autograde.rb:145:7: C: Use snake_case for method names.
  def autogradeSubmissions(course, assessment, submissions)
      ^^^^^^^^^^^^^^^^^^^^
app/controllers/assessment/autograde.rb:147:101: C: Line is too long. [116/100]
    flash[:error] = "Submission could not be autograded due to an error in creation" && return if submissions.blank?
                                                                                                    ^^^^^^^^^^^^^^^^
app/controllers/assessment/autograde.rb:153:101: C: Line is too long. [101/100]
        link = (view_context.link_to "Autograder Settings", [:edit, course, assessment, :autograder])
                                                                                                    ^
app/controllers/assessment/autograde.rb:167:101: C: Line is too long. [101/100]
        link = (view_context.link_to "Autograder Settings", [:edit, course, assessment, :autograder])
                                                                                                    ^
app/controllers/assessment/autograde.rb:173:101: C: Line is too long. [104/100]
      link = "<a href=\"#{url_for(controller: 'jobs', action: 'getjob', id: job)}\">Job ID = #{job}</a>"
                                                                                                    ^^^^
app/controllers/assessment/autograde.rb:184:3: C: Assignment Branch Condition size for tango_upload is too high. [35.86/15]
  def tango_upload(course, assessment, submission, existing_files)
  ^^^
app/controllers/assessment/autograde.rb:184:3: C: Method has too many lines. [29/20]
  def tango_upload(course, assessment, submission, existing_files)
  ^^^
app/controllers/assessment/autograde.rb:263:101: C: Line is too long. [132/100]
      "#{hostname}/courses/#{course.name}/assessments/#{assessment.name}/autograde_done?dave=#{dave}&submission_id=#{submission.id}"
                                                                                                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
app/controllers/assessment/autograde.rb:280:3: C: Assignment Branch Condition size for tango_add_job is too high. [16.31/15]
  def tango_add_job(course, assessment, upload_file_list, callback_url, job_name, output_file)
  ^^^
app/controllers/assessment/autograde.rb:280:20: C: Avoid parameter lists longer than 5 parameters.
  def tango_add_job(course, assessment, upload_file_list, callback_url, job_name, output_file)
                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
app/controllers/assessment/autograde.rb:303:3: C: Assignment Branch Condition size for tango_poll is too high. [18.87/15]
  def tango_poll(course, assessment, submissions, output_file)
  ^^^
app/controllers/assessment/autograde.rb:303:3: C: Method has too many lines. [28/20]
  def tango_poll(course, assessment, submissions, output_file)
  ^^^
app/controllers/assessment/autograde.rb:308:101: C: Line is too long. [104/100]
          response = TangoClient.poll("#{course.name}-#{assessment.name}", "#{URI.encode(output_file)}")
                                                                                                    ^^^^
app/controllers/assessment/autograde.rb:346:3: C: Assignment Branch Condition size for sendJob is too high. [32.16/15]
  def sendJob(course, assessment, submissions, cud)
  ^^^
app/controllers/assessment/autograde.rb:346:3: C: Cyclomatic complexity for sendJob is too high. [8/6]
  def sendJob(course, assessment, submissions, cud)
  ^^^
app/controllers/assessment/autograde.rb:346:3: C: Method has too many lines. [24/20]
  def sendJob(course, assessment, submissions, cud)
  ^^^
app/controllers/assessment/autograde.rb:346:3: C: Perceived complexity for sendJob is too high. [8/7]
  def sendJob(course, assessment, submissions, cud)
  ^^^
app/controllers/assessment/autograde.rb:346:7: C: Use snake_case for method names.
  def sendJob(course, assessment, submissions, cud)
      ^^^^^^^
app/controllers/assessment/autograde.rb:389:7: C: Use snake_case for method names.
  def autogradeInputFiles(ass_dir, assessment, submission)
      ^^^^^^^^^^^^^^^^^^^
app/controllers/assessment/autograde.rb:415:7: C: Use snake_case for method names.
  def autogradeDone(submissions, feedback)
      ^^^^^^^^^^^^^
app/controllers/assessment/autograde.rb:438:3: C: Assignment Branch Condition size for saveAutograde is too high. [66.77/15]
  def saveAutograde(submissions, feedback)
  ^^^
app/controllers/assessment/autograde.rb:438:3: C: Cyclomatic complexity for saveAutograde is too high. [9/6]
  def saveAutograde(submissions, feedback)
  ^^^
app/controllers/assessment/autograde.rb:438:3: C: Method has too many lines. [50/20]
  def saveAutograde(submissions, feedback)
  ^^^
app/controllers/assessment/autograde.rb:438:3: C: Perceived complexity for saveAutograde is too high. [10/7]
  def saveAutograde(submissions, feedback)
  ^^^
app/controllers/assessment/autograde.rb:438:7: C: Use snake_case for method names.
  def saveAutograde(submissions, feedback)
      ^^^^^^^^^^^^^
app/controllers/assessment/autograde.rb:498:101: C: Line is too long. [112/100]
      logger.add(Logger::INFO) { "#{submission.course_user_datum.email}, #{submission.version}, #{autoresult}" }
                                                                                                    ^^^^^^^^^^^^
app/controllers/assessment/autograde.rb:507:7: C: Use snake_case for method names.
  def parseAutoresult(autoresult, _isOfficial)
      ^^^^^^^^^^^^^^^
app/controllers/assessment/autograde.rb:514:3: C: Assignment Branch Condition size for extend_config_module is too high. [40.41/15]
  def extend_config_module(assessment, submission, cud)
  ^^^
app/controllers/assessment/autograde.rb:514:3: C: Cyclomatic complexity for extend_config_module is too high. [7/6]
  def extend_config_module(assessment, submission, cud)
  ^^^
app/controllers/assessment/autograde.rb:514:3: C: Method has too many lines. [42/20]
  def extend_config_module(assessment, submission, cud)
  ^^^
app/controllers/assessment/grading.rb:4:1: C: Missing top-level module documentation comment.
module AssessmentGrading
^^^^^^
app/controllers/assessment/grading.rb:6:7: C: Use snake_case for method names.
  def bulkExport
      ^^^^^^^^^^
app/controllers/assessment/grading.rb:11:22: C: Do not use Time.[:now] without zone. Use #Time.zone.[:now] instead.
    timestamp = Time.now.strftime "%Y%m%d%H%M"
                     ^^^
app/controllers/assessment/grading.rb:17:3: C: Assignment Branch Condition size for bulkGrade is too high. [25.96/15]
  def bulkGrade
  ^^^
app/controllers/assessment/grading.rb:17:3: C: Cyclomatic complexity for bulkGrade is too high. [10/6]
  def bulkGrade
  ^^^
app/controllers/assessment/grading.rb:17:3: C: Method has too many lines. [22/20]
  def bulkGrade
  ^^^
app/controllers/assessment/grading.rb:17:3: C: Perceived complexity for bulkGrade is too high. [12/7]
  def bulkGrade
  ^^^
app/controllers/assessment/grading.rb:17:7: C: Use snake_case for method names.
  def bulkGrade
      ^^^^^^^^^
app/controllers/assessment/grading.rb:21:5: C: Use a guard clause instead of wrapping the code inside a conditional expression.
    if params[:upload]
    ^^
app/controllers/assessment/grading.rb:50:3: C: Assignment Branch Condition size for bulkGrade_complete is too high. [29.58/15]
  def bulkGrade_complete
  ^^^
app/controllers/assessment/grading.rb:50:3: C: Cyclomatic complexity for bulkGrade_complete is too high. [12/6]
  def bulkGrade_complete
  ^^^
app/controllers/assessment/grading.rb:50:3: C: Perceived complexity for bulkGrade_complete is too high. [13/7]
  def bulkGrade_complete
  ^^^
app/controllers/assessment/grading.rb:50:7: C: Use snake_case for method names.
  def bulkGrade_complete
      ^^^^^^^^^^^^^^^^^^
app/controllers/assessment/grading.rb:71:5: C: Use a guard clause instead of wrapping the code inside a conditional expression.
    unless save_entries @entries, data_type
    ^^^^^^
app/controllers/assessment/grading.rb:88:7: C: Indent when as deep as case.
      when Hash
      ^^^^
app/controllers/assessment/grading.rb:96:3: C: Assignment Branch Condition size for save_entries is too high. [40.63/15]
  def save_entries(entries, data_type)
  ^^^
app/controllers/assessment/grading.rb:96:3: C: Cyclomatic complexity for save_entries is too high. [8/6]
  def save_entries(entries, data_type)
  ^^^
app/controllers/assessment/grading.rb:96:3: C: Method has too many lines. [43/20]
  def save_entries(entries, data_type)
  ^^^
app/controllers/assessment/grading.rb:111:22: W: Assignment in condition - you probably meant to use ==.
          unless sub = aud.latest_submission
                     ^
app/controllers/assessment/grading.rb:155:89: W: Ambiguous block operator. Parenthesize the method arguments if it's surely a block operator, or add a whitespace to the right of the & if it should be a binary AND.
    emails = Set.new(CourseUserDatum.joins(:user).where(course: @assessment.course).map &:email)
                                                                                        ^
app/controllers/assessment/grading.rb:172:3: C: Assignment Branch Condition size for parse_csv_row is too high. [33.62/15]
  def parse_csv_row(row, kind, problems, emails)
  ^^^
app/controllers/assessment/grading.rb:172:3: C: Cyclomatic complexity for parse_csv_row is too high. [10/6]
  def parse_csv_row(row, kind, problems, emails)
  ^^^
app/controllers/assessment/grading.rb:172:3: C: Method has too many lines. [36/20]
  def parse_csv_row(row, kind, problems, emails)
  ^^^
app/controllers/assessment/grading.rb:172:3: C: Perceived complexity for parse_csv_row is too high. [12/7]
  def parse_csv_row(row, kind, problems, emails)
  ^^^
app/controllers/assessment/grading.rb:190:5: W: end at 190, 4 is not aligned with if at 184, 24
    end
    ^^^
app/controllers/assessment/grading.rb:199:11: C: Avoid using rescue in its modifier form.
          Float(datum) rescue({ error: datum })
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
app/controllers/assessment/grading.rb:219:5: W: end at 219, 4 is not aligned with if at 213, 29
    end
    ^^^
app/controllers/assessment/grading.rb:226:3: C: Assignment Branch Condition size for quickSetScore is too high. [27.66/15]
  def quickSetScore
  ^^^
app/controllers/assessment/grading.rb:226:7: C: Use snake_case for method names.
  def quickSetScore
      ^^^^^^^^^^^^^
app/controllers/assessment/grading.rb:255:3: C: Assignment Branch Condition size for quickSetScoreDetails is too high. [29.55/15]
  def quickSetScoreDetails
  ^^^
app/controllers/assessment/grading.rb:255:7: C: Use snake_case for method names.
  def quickSetScoreDetails
      ^^^^^^^^^^^^^^^^^^^^
app/controllers/assessment/grading.rb:291:15: C: Favor the ternary operator (?:) over if/then/else/end constructs.
    grader = (if score then score.grader else nil end)
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
app/controllers/assessment/grading.rb:291:42: C: Redundant else-clause.
    grader = (if score then score.grader else nil end)
                                         ^^^^
app/controllers/assessment/grading.rb:299:7: C: Use snake_case for method names.
  def viewGradesheet
      ^^^^^^^^^^^^^^
app/controllers/assessment/grading.rb:303:7: C: Use snake_case for method names.
  def quickGetTotal
      ^^^^^^^^^^^^^
app/controllers/assessment/grading.rb:312:3: C: Assignment Branch Condition size for statistics is too high. [41.05/15]
  def statistics
  ^^^
app/controllers/assessment/grading.rb:312:3: C: Method has too many lines. [21/20]
  def statistics
  ^^^
app/controllers/assessment/grading.rb:355:7: W: The use of eval is a serious security risk.
      eval("extend(Course#{course.camelize})")
      ^^^^
app/controllers/assessment/grading.rb:356:5: W: Avoid rescuing the Exception class. Perhaps you meant to rescue StandardError?
    rescue Exception
    ^^^^^^^^^^^^^^^^
app/controllers/assessment/grading.rb:359:69: C: Use && instead of and.
        "admin/reload'>here</a> to reload the file, and try again") and
                                                                    ^^^
app/controllers/assessment/grading.rb:365:3: C: Assignment Branch Condition size for stats_for_grouping is too high. [37.39/15]
  def stats_for_grouping(grouping)
  ^^^
app/controllers/assessment/grading.rb:365:3: C: Method has too many lines. [25/20]
  def stats_for_grouping(grouping)
  ^^^
app/controllers/assessment/grading.rb:367:5: W: Useless assignment to variable - problem_id_to_name.
    problem_id_to_name = @assessment.problem_id_to_name
    ^^^^^^^^^^^^^^^^^^
app/controllers/assessment/grading.rb:407:3: C: Assignment Branch Condition size for stats_for_grader is too high. [52.33/15]
  def stats_for_grader(submissions)
  ^^^
app/controllers/assessment/grading.rb:407:3: C: Method has too many lines. [48/20]
  def stats_for_grader(submissions)
  ^^^
app/controllers/assessment/grading.rb:409:5: W: Useless assignment to variable - problem_id_to_name.
    problem_id_to_name = @assessment.problem_id_to_name
    ^^^^^^^^^^^^^^^^^^
app/controllers/assessment/grading.rb:467:3: C: Assignment Branch Condition size for load_gradesheet_data is too high. [17.12/15]
  def load_gradesheet_data
  ^^^
app/controllers/assessment/grading.rb:468:19: C: Do not use Time.[:now] without zone. Use #Time.zone.[:now] instead.
    @start = Time.now
                  ^^^
app/controllers/assessment/grading.rb:472:9: C: Avoid multi-line ?: (the ternary operator); use if/unless instead.
    o = params[:section] ? {
        ^^^^^^^^^^^^^^^^^^^^
app/controllers/assessment/grading.rb:480:47: W: Do not use prefix _ for a variable that is used.
    cache = AssociationCache.new(@course) do |_|
                                              ^
app/controllers/assessment/handin.rb:21:3: C: Assignment Branch Condition size for handin is too high. [21.12/15]
  def handin
  ^^^
app/controllers/assessment/handin.rb:21:3: C: Cyclomatic complexity for handin is too high. [10/6]
  def handin
  ^^^
app/controllers/assessment/handin.rb:21:3: C: Perceived complexity for handin is too high. [10/7]
  def handin
  ^^^
app/controllers/assessment/handin.rb:49:3: C: Assignment Branch Condition size for local_submit is too high. [89.69/15]
  def local_submit
  ^^^
app/controllers/assessment/handin.rb:49:3: C: Cyclomatic complexity for local_submit is too high. [31/6]
  def local_submit
  ^^^
app/controllers/assessment/handin.rb:49:3: C: Method has too many lines. [71/20]
  def local_submit
  ^^^
app/controllers/assessment/handin.rb:49:3: C: Perceived complexity for local_submit is too high. [34/7]
  def local_submit
  ^^^
app/controllers/assessment/handin.rb:151:3: C: Assignment Branch Condition size for log_submit is too high. [77.76/15]
  def log_submit
  ^^^
app/controllers/assessment/handin.rb:151:3: C: Cyclomatic complexity for log_submit is too high. [15/6]
  def log_submit
  ^^^
app/controllers/assessment/handin.rb:151:3: C: Method has too many lines. [49/20]
  def log_submit
  ^^^
app/controllers/assessment/handin.rb:151:3: C: Perceived complexity for log_submit is too high. [17/7]
  def log_submit
  ^^^
app/controllers/assessment/handin.rb:178:5: W: The use of eval is a serious security risk.
    eval("extend #{mod_name.camelcase}")
    ^^^^
app/controllers/assessment/handin.rb:202:38: C: Do not use Time.[:now] without zone. Use #Time.zone.[:now] instead.
        submission.created_at = Time.now
                                     ^^^
app/controllers/assessment/handin.rb:229:3: C: Assignment Branch Condition size for validateHandin is too high. [37.34/15]
  def validateHandin
  ^^^
app/controllers/assessment/handin.rb:229:3: C: Method has too many lines. [21/20]
  def validateHandin
  ^^^
app/controllers/assessment/handin.rb:229:7: C: Use snake_case for method names.
  def validateHandin
      ^^^^^^^^^^^^^^
app/controllers/assessment/handin.rb:266:3: C: Assignment Branch Condition size for validate_for_groups is too high. [20.15/15]
  def validate_for_groups
  ^^^
app/controllers/assessment/handin.rb:266:3: C: Cyclomatic complexity for validate_for_groups is too high. [7/6]
  def validate_for_groups
  ^^^
app/controllers/assessment/handin.rb:295:3: C: Assignment Branch Condition size for saveHandin is too high. [27.73/15]
  def saveHandin(sub)
  ^^^
app/controllers/assessment/handin.rb:295:3: C: Method has too many lines. [24/20]
  def saveHandin(sub)
  ^^^
app/controllers/assessment/handin.rb:295:7: C: Use snake_case for method names.
  def saveHandin(sub)
      ^^^^^^^^^^
app/controllers/assessment/handin.rb:324:3: C: Assignment Branch Condition size for set_handin is too high. [17.49/15]
  def set_handin
  ^^^
app/controllers/assessment/handin.rb:328:51: C: Do not use Time.[:now] without zone. Use #Time.zone.[:now] instead.
    @can_submit, @why_not = @aud.can_submit? Time.now
                                                  ^^^
app/controllers/assessment/handout.rb:13:3: C: Assignment Branch Condition size for handout is too high. [16.61/15]
  def handout
  ^^^
app/controllers/assessment/svn.rb:25:3: C: Assignment Branch Condition size for import_svn is too high. [18.68/15]
  def import_svn
  ^^^
app/controllers/assessment/svn.rb:53:3: C: Assignment Branch Condition size for svn_save_handin is too high. [27.82/15]
  def svn_save_handin
  ^^^
app/controllers/assessment/svn.rb:53:3: C: Method has too many lines. [25/20]
  def svn_save_handin
  ^^^
app/controllers/assessments_controller.rb:8:1: C: Class definition is too long. [583/100]
class AssessmentsController < ApplicationController
^^^^^
app/controllers/assessments_controller.rb:8:1: C: Missing top-level class documentation comment.
class AssessmentsController < ApplicationController
^^^^^
app/controllers/assessments_controller.rb:64:101: C: Line is too long. [166/100]
    @announcements = Announcement.where("start_date<? and end_date>? and (course_id=? or system) and !persistent", Time.now, Time.now, @course.id).order("start_date")
                                                                                                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
app/controllers/assessments_controller.rb:64:121: C: Do not use Time.[:now] without zone. Use #Time.zone.[:now] instead.
    @announcements = Announcement.where("start_date<? and end_date>? and (course_id=? or system) and !persistent", Time.now, Time.now, @course.id).order("start_date")
                                                                                                                        ^^^
app/controllers/assessments_controller.rb:64:131: C: Do not use Time.[:now] without zone. Use #Time.zone.[:now] instead.
    @announcements = Announcement.where("start_date<? and end_date>? and (course_id=? or system) and !persistent", Time.now, Time.now, @course.id).order("start_date")
                                                                                                                                  ^^^
app/controllers/assessments_controller.rb:65:101: C: Line is too long. [103/100]
    @attachments = (@cud.instructor?) ? @course.attachments : @course.attachments.where(released: true)
                                                                                                    ^^^
app/controllers/assessments_controller.rb:78:3: C: Assignment Branch Condition size for installQuiz is too high. [52.76/15]
  def installQuiz
  ^^^
app/controllers/assessments_controller.rb:78:3: C: Method has too many lines. [34/20]
  def installQuiz
  ^^^
app/controllers/assessments_controller.rb:78:7: C: Use snake_case for method names.
  def installQuiz
      ^^^^^^^^^^^
app/controllers/assessments_controller.rb:83:7: C: Use snake_case for variable names.
      quizJSON = params[:quiz]
      ^^^^^^^^
app/controllers/assessments_controller.rb:84:7: C: Use snake_case for variable names.
      quizDisplayName = params[:quizName]
      ^^^^^^^^^^^^^^^
app/controllers/assessments_controller.rb:85:7: C: Use snake_case for variable names.
      quizName = quizDisplayName.downcase.gsub(/[^a-z0-9]/, "")
      ^^^^^^^^
app/controllers/assessments_controller.rb:94:37: C: Do not use Time.[:now] without zone. Use #Time.zone.[:now] instead.
      @assessment.visible_at = Time.now
                                    ^^^
app/controllers/assessments_controller.rb:95:35: C: Do not use Time.[:now] without zone. Use #Time.zone.[:now] instead.
      @assessment.start_at = Time.now
                                  ^^^
app/controllers/assessments_controller.rb:96:33: C: Do not use Time.[:now] without zone. Use #Time.zone.[:now] instead.
      @assessment.due_at = Time.now
                                ^^^
app/controllers/assessments_controller.rb:97:43: C: Do not use Time.[:now] without zone. Use #Time.zone.[:now] instead.
      @assessment.grading_deadline = Time.now
                                          ^^^
app/controllers/assessments_controller.rb:98:33: C: Do not use Time.[:now] without zone. Use #Time.zone.[:now] instead.
      @assessment.end_at = Time.now
                                ^^^
app/controllers/assessments_controller.rb:101:101: C: Line is too long. [101/100]
      @assessment.max_submissions = params.include?(:max_submissions) ? params[:max_submissions] : -1
                                                                                                    ^
app/controllers/assessments_controller.rb:107:7: C: Use snake_case for variable names.
      quizData = JSON.parse(quizJSON)
      ^^^^^^^^
app/controllers/assessments_controller.rb:123:3: C: Assignment Branch Condition size for takeQuiz is too high. [16.03/15]
  def takeQuiz
  ^^^
app/controllers/assessments_controller.rb:123:7: C: Use snake_case for method names.
  def takeQuiz
      ^^^^^^^^
app/controllers/assessments_controller.rb:129:5: C: Use snake_case for variable names.
    @quizData = JSON.parse(@assessment.quizData)
    ^^^^^^^^^
app/controllers/assessments_controller.rb:130:5: C: Use snake_case for variable names.
    @submitPath = submitQuiz_course_assessment_path(@course, @assessment)
    ^^^^^^^^^^^
app/controllers/assessments_controller.rb:135:3: C: Assignment Branch Condition size for submitQuiz is too high. [36.24/15]
  def submitQuiz
  ^^^
app/controllers/assessments_controller.rb:135:3: C: Cyclomatic complexity for submitQuiz is too high. [7/6]
  def submitQuiz
  ^^^
app/controllers/assessments_controller.rb:135:3: C: Method has too many lines. [23/20]
  def submitQuiz
  ^^^
app/controllers/assessments_controller.rb:135:7: C: Use snake_case for method names.
  def submitQuiz
      ^^^^^^^^^^
app/controllers/assessments_controller.rb:141:5: C: Use snake_case for variable names.
    @quizData = JSON.parse(@assessment.quizData)
    ^^^^^^^^^
app/controllers/assessments_controller.rb:145:7: C: Use snake_case for variable names.
      actualAnswer = @quizData[i]["answer"]
      ^^^^^^^^^^^^
app/controllers/assessments_controller.rb:151:5: C: Use snake_case for variable names.
    quizScore = Score.new(score: score,
    ^^^^^^^^^
app/controllers/assessments_controller.rb:166:7: C: Use snake_case for method names.
  def installAssessment
      ^^^^^^^^^^^^^^^^^
app/controllers/assessments_controller.rb:182:3: C: Assignment Branch Condition size for importAsmtFromTar is too high. [66.68/15]
  def importAsmtFromTar
  ^^^
app/controllers/assessments_controller.rb:182:3: C: Cyclomatic complexity for importAsmtFromTar is too high. [12/6]
  def importAsmtFromTar
  ^^^
app/controllers/assessments_controller.rb:182:3: C: Method has too many lines. [53/20]
  def importAsmtFromTar
  ^^^
app/controllers/assessments_controller.rb:182:3: C: Perceived complexity for importAsmtFromTar is too high. [13/7]
  def importAsmtFromTar
  ^^^
app/controllers/assessments_controller.rb:182:7: C: Use snake_case for method names.
  def importAsmtFromTar
      ^^^^^^^^^^^^^^^^^
app/controllers/assessments_controller.rb:183:5: C: Use snake_case for variable names.
    tarFile = params["tarFile"]
    ^^^^^^^
app/controllers/assessments_controller.rb:191:7: C: Use snake_case for variable names.
      tarFile = File.new(tarFile.open, "rb")
      ^^^^^^^
app/controllers/assessments_controller.rb:209:101: C: Line is too long. [116/100]
      flash[:error] = "An assessment with the same name already exists for the course. Please use a different name."
                                                                                                    ^^^^^^^^^^^^^^^^
app/controllers/assessments_controller.rb:248:7: C: Use snake_case for method names.
  def importAssessment
      ^^^^^^^^^^^^^^^^
app/controllers/assessments_controller.rb:259:3: C: Assignment Branch Condition size for create is too high. [54.78/15]
  def create
  ^^^
app/controllers/assessments_controller.rb:259:3: C: Cyclomatic complexity for create is too high. [8/6]
  def create
  ^^^
app/controllers/assessments_controller.rb:259:3: C: Method has too many lines. [44/20]
  def create
  ^^^
app/controllers/assessments_controller.rb:259:3: C: Perceived complexity for create is too high. [8/7]
  def create
  ^^^
app/controllers/assessments_controller.rb:280:35: C: Do not use Time.[:now] without zone. Use #Time.zone.[:now] instead.
    @assessment.visible_at = Time.now
                                  ^^^
app/controllers/assessments_controller.rb:281:33: C: Do not use Time.[:now] without zone. Use #Time.zone.[:now] instead.
    @assessment.start_at = Time.now
                                ^^^
app/controllers/assessments_controller.rb:282:31: C: Do not use Time.[:now] without zone. Use #Time.zone.[:now] instead.
    @assessment.due_at = Time.now
                              ^^^
app/controllers/assessments_controller.rb:283:41: C: Do not use Time.[:now] without zone. Use #Time.zone.[:now] instead.
    @assessment.grading_deadline = Time.now
                                        ^^^
app/controllers/assessments_controller.rb:284:31: C: Do not use Time.[:now] without zone. Use #Time.zone.[:now] instead.
    @assessment.end_at = Time.now
                              ^^^
app/controllers/assessments_controller.rb:321:3: C: Assignment Branch Condition size for assessmentInitialize is too high. [16.4/15]
  def assessmentInitialize(assignName)
  ^^^
app/controllers/assessments_controller.rb:321:7: C: Use snake_case for method names.
  def assessmentInitialize(assignName)
      ^^^^^^^^^^^^^^^^^^^^
app/controllers/assessments_controller.rb:347:3: C: Assignment Branch Condition size for installProblems is too high. [16.28/15]
  def installProblems
  ^^^
app/controllers/assessments_controller.rb:347:7: C: Use snake_case for method names.
  def installProblems
      ^^^^^^^^^^^^^^^
app/controllers/assessments_controller.rb:378:3: C: Assignment Branch Condition size for grade is too high. [19.95/15]
  def grade
  ^^^
app/controllers/assessments_controller.rb:383:7: C: Use snake_case for variable names.
      subFile = File.join(Rails.root, "courses",
      ^^^^^^^
app/controllers/assessments_controller.rb:387:7: C: Use snake_case for variable names.
      @submissionData = File.read(subFile)
      ^^^^^^^^^^^^^^^
app/controllers/assessments_controller.rb:389:7: C: Use snake_case for variable names.
      @submissionData = "Could not read #{subFile}"
      ^^^^^^^^^^^^^^^
app/controllers/assessments_controller.rb:394:7: C: Use snake_case for method names.
  def getAssessmentVariable(key)
      ^^^^^^^^^^^^^^^^^^^^^
app/controllers/assessments_controller.rb:405:3: C: Assignment Branch Condition size for export is too high. [47.61/15]
  def export
  ^^^
app/controllers/assessments_controller.rb:405:3: C: Method has too many lines. [31/20]
  def export
  ^^^
app/controllers/assessments_controller.rb:412:7: C: Use snake_case for variable names.
      tarStream = StringIO.new("")
      ^^^^^^^^^
app/controllers/assessments_controller.rb:417:37: C: Use %r around regular expression.
          relative_path = file.sub((/^#{Regexp.escape base_path}\/?/), "")
                                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
app/controllers/assessments_controller.rb:421:101: C: Line is too long. [101/100]
          elsif !relative_path.starts_with? File.join(@assessment.name, @assessment.handin_directory)
                                                                                                    ^
app/controllers/assessments_controller.rb:430:98: C: Do not use Time.[:now] without zone. Use #Time.zone.[:now] instead.
      send_data tarStream.string.force_encoding("binary"), filename: "#{@assessment.name}_#{Time.now.strftime('%Y%m%d')}.tar", content_type: "application/x-tar"
                                                                                                 ^^^
app/controllers/assessments_controller.rb:430:101: C: Line is too long. [160/100]
      send_data tarStream.string.force_encoding("binary"), filename: "#{@assessment.name}_#{Time.now.strftime('%Y%m%d')}.tar", content_type: "application/x-tar"
                                                                                                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
app/controllers/assessments_controller.rb:444:5: C: Prefer each over for.
    for submission in @assessment.submissions do
    ^^^
app/controllers/assessments_controller.rb:448:5: C: Prefer each over for.
    for attachment in @assessment.attachments do
    ^^^
app/controllers/assessments_controller.rb:459:3: C: Assignment Branch Condition size for show is too high. [51.89/15]
  def show
  ^^^
app/controllers/assessments_controller.rb:459:3: C: Cyclomatic complexity for show is too high. [7/6]
  def show
  ^^^
app/controllers/assessments_controller.rb:459:3: C: Method has too many lines. [40/20]
  def show
  ^^^
app/controllers/assessments_controller.rb:459:3: C: Perceived complexity for show is too high. [8/7]
  def show
  ^^^
app/controllers/assessments_controller.rb:476:5: C: Use snake_case for variable names.
    @startTime = Time.now
    ^^^^^^^^^^
app/controllers/assessments_controller.rb:476:23: C: Do not use Time.[:now] without zone. Use #Time.zone.[:now] instead.
    @startTime = Time.now
                      ^^^
app/controllers/assessments_controller.rb:478:7: C: Use snake_case for variable names.
      @effectiveCud = @course.course_user_data.find(params[:cud_id])
      ^^^^^^^^^^^^^
app/controllers/assessments_controller.rb:480:7: C: Use snake_case for variable names.
      @effectiveCud = @cud
      ^^^^^^^^^^^^^
app/controllers/assessments_controller.rb:482:101: C: Line is too long. [110/100]
    @submissions = @assessment.submissions.where(course_user_datum_id: @effectiveCud.id).order("version DESC")
                                                                                                    ^^^^^^^^^^
app/controllers/assessments_controller.rb:498:5: C: Prefer each over for.
    for result in results do
    ^^^
app/controllers/assessments_controller.rb:499:7: C: Use snake_case for variable names.
      subId = result["submission_id"].to_i
      ^^^^^
app/controllers/assessments_controller.rb:515:3: C: Assignment Branch Condition size for history is too high. [47.1/15]
  def history
  ^^^
app/controllers/assessments_controller.rb:515:3: C: Cyclomatic complexity for history is too high. [8/6]
  def history
  ^^^
app/controllers/assessments_controller.rb:515:3: C: Method has too many lines. [35/20]
  def history
  ^^^
app/controllers/assessments_controller.rb:515:3: C: Perceived complexity for history is too high. [9/7]
  def history
  ^^^
app/controllers/assessments_controller.rb:519:5: C: Use snake_case for variable names.
    @startTime = Time.now
    ^^^^^^^^^^
app/controllers/assessments_controller.rb:519:23: C: Do not use Time.[:now] without zone. Use #Time.zone.[:now] instead.
    @startTime = Time.now
                      ^^^
app/controllers/assessments_controller.rb:521:7: C: Use snake_case for variable names.
      @effectiveCud = @course.course_user_data.find(params[:cud_id])
      ^^^^^^^^^^^^^
app/controllers/assessments_controller.rb:523:7: C: Use snake_case for variable names.
      @effectiveCud = @cud
      ^^^^^^^^^^^^^
app/controllers/assessments_controller.rb:525:101: C: Line is too long. [110/100]
    @submissions = @assessment.submissions.where(course_user_datum_id: @effectiveCud.id).order("version DESC")
                                                                                                    ^^^^^^^^^^
app/controllers/assessments_controller.rb:541:5: C: Prefer each over for.
    for result in results do
    ^^^
app/controllers/assessments_controller.rb:542:7: C: Use snake_case for variable names.
      subId = result["submission_id"].to_i
      ^^^^^
app/controllers/assessments_controller.rb:556:5: C: Use a guard clause instead of wrapping the code inside a conditional expression.
    if params[:partial]
    ^^
app/controllers/assessments_controller.rb:563:7: C: Use snake_case for method names.
  def viewFeedback
      ^^^^^^^^^^^^
app/controllers/assessments_controller.rb:569:5: C: Use a guard clause instead of wrapping the code inside a conditional expression.
    if Archive.archive? @submission.handin_file_path
    ^^
app/controllers/assessments_controller.rb:577:3: W: Do not suppress exceptions.
  rescue StandardError => @error
  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
app/controllers/assessments_controller.rb:607:7: C: Use snake_case for method names.
  def releaseAllGrades
      ^^^^^^^^^^^^^^^^
app/controllers/assessments_controller.rb:612:43: C: Favor format over String#%.
      flash[:success] = "%d %s released." % [num_released, (num_released > 1 ? "grades were" : "grade was")]
                                          ^
app/controllers/assessments_controller.rb:612:101: C: Line is too long. [108/100]
      flash[:success] = "%d %s released." % [num_released, (num_released > 1 ? "grades were" : "grade was")]
                                                                                                    ^^^^^^^^
app/controllers/assessments_controller.rb:620:3: C: Assignment Branch Condition size for releaseSectionGrades is too high. [22.83/15]
  def releaseSectionGrades
  ^^^
app/controllers/assessments_controller.rb:620:3: C: Cyclomatic complexity for releaseSectionGrades is too high. [7/6]
  def releaseSectionGrades
  ^^^
app/controllers/assessments_controller.rb:620:3: C: Perceived complexity for releaseSectionGrades is too high. [8/7]
  def releaseSectionGrades
  ^^^
app/controllers/assessments_controller.rb:620:7: C: Use snake_case for method names.
  def releaseSectionGrades
      ^^^^^^^^^^^^^^^^^^^^
app/controllers/assessments_controller.rb:622:101: C: Line is too long. [110/100]
      flash[:error] = "You haven't been assigned to a lecture and/or section. Please contact your instructor."
                                                                                                    ^^^^^^^^^^
app/controllers/assessments_controller.rb:627:101: C: Line is too long. [101/100]
    num_released = releaseMatchingGrades { |submission, _| @cud.CA_of? submission.course_user_datum }
                                                                                                    ^
app/controllers/assessments_controller.rb:630:43: C: Favor format over String#%.
      flash[:success] = "%d %s released." % [num_released, (num_released > 1 ? "grades were" : "grade was")]
                                          ^
app/controllers/assessments_controller.rb:630:101: C: Line is too long. [108/100]
      flash[:success] = "%d %s released." % [num_released, (num_released > 1 ? "grades were" : "grade was")]
                                                                                                    ^^^^^^^^
app/controllers/assessments_controller.rb:633:101: C: Line is too long. [102/100]
                      "Either they were all already released or you might be assigned to a lecture " \
                                                                                                    ^^
app/controllers/assessments_controller.rb:640:7: C: Use snake_case for method names.
  def withdrawAllGrades
      ^^^^^^^^^^^^^^^^^
app/controllers/assessments_controller.rb:683:7: C: Use snake_case for method names.
  def updateScore(_user, score)
      ^^^^^^^^^^^
app/controllers/assessments_controller.rb:689:7: C: Use snake_case for method names.
  def loadHandinPage
      ^^^^^^^^^^^^^^
app/controllers/assessments_controller.rb:692:3: C: Assignment Branch Condition size for releaseMatchingGrades is too high. [15.94/15]
  def releaseMatchingGrades
  ^^^
app/controllers/assessments_controller.rb:692:7: C: Use snake_case for method names.
  def releaseMatchingGrades
      ^^^^^^^^^^^^^^^^^^^^^
app/controllers/assessments_controller.rb:731:3: C: Assignment Branch Condition size for edit_assessment_params is too high. [27.91/15]
  def edit_assessment_params
  ^^^
app/controllers/assessments_controller.rb:731:3: C: Cyclomatic complexity for edit_assessment_params is too high. [8/6]
  def edit_assessment_params
  ^^^
app/controllers/assessments_controller.rb:731:3: C: Perceived complexity for edit_assessment_params is too high. [8/7]
  def edit_assessment_params
  ^^^
app/controllers/assessments_controller.rb:749:3: C: Assignment Branch Condition size for valid_asmt_tar is too high. [15.17/15]
  def valid_asmt_tar(tar_extract)
  ^^^
app/controllers/assessments_controller.rb:749:3: C: Cyclomatic complexity for valid_asmt_tar is too high. [10/6]
  def valid_asmt_tar(tar_extract)
  ^^^
app/controllers/assessments_controller.rb:749:3: C: Perceived complexity for valid_asmt_tar is too high. [11/7]
  def valid_asmt_tar(tar_extract)
  ^^^
app/controllers/attachments_controller.rb:33:3: C: Assignment Branch Condition size for show is too high. [16.79/15]
  def show
  ^^^
app/controllers/attachments_controller.rb:52:3: C: Cyclomatic complexity for update is too high. [8/6]
  def update
  ^^^
app/controllers/attachments_controller.rb:52:3: C: Perceived complexity for update is too high. [11/7]
  def update
  ^^^
app/controllers/course_user_data_controller.rb:1:1: C: Class definition is too long. [188/100]
class CourseUserDataController < ApplicationController
^^^^^
app/controllers/course_user_data_controller.rb:1:1: C: Missing top-level class documentation comment.
class CourseUserDataController < ApplicationController
^^^^^
app/controllers/course_user_data_controller.rb:6:5: C: Use snake_case for variable names.
    @requestedUser = @cud
    ^^^^^^^^^^^^^^
app/controllers/course_user_data_controller.rb:12:5: C: Use snake_case for variable names.
    @newCUD = @course.course_user_data.new
    ^^^^^^^
app/controllers/course_user_data_controller.rb:18:3: C: Assignment Branch Condition size for create is too high. [37.7/15]
  def create
  ^^^
app/controllers/course_user_data_controller.rb:18:3: C: Cyclomatic complexity for create is too high. [11/6]
  def create
  ^^^
app/controllers/course_user_data_controller.rb:18:3: C: Method has too many lines. [33/20]
  def create
  ^^^
app/controllers/course_user_data_controller.rb:18:3: C: Perceived complexity for create is too high. [15/7]
  def create
  ^^^
app/controllers/course_user_data_controller.rb:20:5: C: Use snake_case for variable names.
    @newCUD = @course.course_user_data.new(cud_parameters)
    ^^^^^^^
app/controllers/course_user_data_controller.rb:63:5: C: Use snake_case for variable names.
    @requestedUser = @cud.course.course_user_data.find(params[:id])
    ^^^^^^^^^^^^^^
app/controllers/course_user_data_controller.rb:75:3: C: Assignment Branch Condition size for edit is too high. [21.93/15]
  def edit
  ^^^
app/controllers/course_user_data_controller.rb:75:3: C: Cyclomatic complexity for edit is too high. [7/6]
  def edit
  ^^^
app/controllers/course_user_data_controller.rb:76:5: C: Use snake_case for variable names.
    @editCUD = @course.course_user_data.find(params[:id])
    ^^^^^^^^
app/controllers/course_user_data_controller.rb:92:3: C: Assignment Branch Condition size for update is too high. [45.02/15]
  def update
  ^^^
app/controllers/course_user_data_controller.rb:92:3: C: Cyclomatic complexity for update is too high. [14/6]
  def update
  ^^^
app/controllers/course_user_data_controller.rb:92:3: C: Method has too many lines. [29/20]
  def update
  ^^^
app/controllers/course_user_data_controller.rb:92:3: C: Perceived complexity for update is too high. [17/7]
  def update
  ^^^
app/controllers/course_user_data_controller.rb:96:5: C: Use snake_case for variable names.
    @editCUD = @course.course_user_data.find(params[:id])
    ^^^^^^^^
app/controllers/course_user_data_controller.rb:134:3: C: Cyclomatic complexity for destroy is too high. [7/6]
  def destroy
  ^^^
app/controllers/course_user_data_controller.rb:135:5: C: Use snake_case for variable names.
    @destroyCUD = @course.course_user_data.find(params[:id])
    ^^^^^^^^^^^
app/controllers/course_user_data_controller.rb:146:7: C: Use snake_case for method names.
  def destroyConfirm
      ^^^^^^^^^^^^^^
app/controllers/course_user_data_controller.rb:147:5: C: Use snake_case for variable names.
    @destroyCUD = @course.course_user_data.find(params[:id])
    ^^^^^^^^^^^
app/controllers/course_user_data_controller.rb:151:3: C: Assignment Branch Condition size for sudo is too high. [59.47/15]
  def sudo
  ^^^
app/controllers/course_user_data_controller.rb:151:3: C: Cyclomatic complexity for sudo is too high. [14/6]
  def sudo
  ^^^
app/controllers/course_user_data_controller.rb:151:3: C: Method has too many lines. [26/20]
  def sudo
  ^^^
app/controllers/course_user_data_controller.rb:151:3: C: Perceived complexity for sudo is too high. [14/7]
  def sudo
  ^^^
app/controllers/course_user_data_controller.rb:198:5: C: Use a guard clause instead of wrapping the code inside a conditional expression.
    if @cud.instructor
    ^^
app/controllers/course_user_data_controller.rb:206:101: C: Line is too long. [120/100]
                                                :lecture, :section, :instructor, :dropped, :nickname, :course_assistant,
                                                                                                    ^^^^^^^^^^^^^^^^^^^^
app/controllers/course_user_data_controller.rb:211:101: C: Line is too long. [120/100]
                                                :lecture, :section, :instructor, :dropped, :nickname, :course_assistant,
                                                                                                    ^^^^^^^^^^^^^^^^^^^^
app/controllers/course_user_data_controller.rb:223:101: C: Line is too long. [120/100]
                                                :lecture, :section, :instructor, :dropped, :nickname, :course_assistant,
                                                                                                    ^^^^^^^^^^^^^^^^^^^^
app/controllers/course_user_data_controller.rb:224:101: C: Line is too long. [104/100]
                                                user_attributes: [:id, :email, :first_name, :last_name],
                                                                                                    ^^^^
app/controllers/course_user_data_controller.rb:228:101: C: Line is too long. [120/100]
                                                :lecture, :section, :instructor, :dropped, :nickname, :course_assistant,
                                                                                                    ^^^^^^^^^^^^^^^^^^^^
app/controllers/course_user_data_controller.rb:229:101: C: Line is too long. [104/100]
                                                user_attributes: [:id, :email, :first_name, :last_name],
                                                                                                    ^^^^
app/controllers/course_user_data_controller.rb:232:101: C: Line is too long. [104/100]
      params.require(:course_user_datum).permit(:nickname) # user_attributes: [:first_name, :last_name])
                                                                                                    ^^^^
app/controllers/courses_controller.rb:6:1: C: Class definition is too long. [524/100]
class CoursesController < ApplicationController
^^^^^
app/controllers/courses_controller.rb:6:1: C: Missing top-level class documentation comment.
class CoursesController < ApplicationController
^^^^^
app/controllers/courses_controller.rb:33:3: C: Assignment Branch Condition size for manage is too high. [22.89/15]
  def manage
  ^^^
app/controllers/courses_controller.rb:70:5: C: Use snake_case for variable names.
    @newCourse = Course.new
    ^^^^^^^^^^
app/controllers/courses_controller.rb:75:3: C: Assignment Branch Condition size for create is too high. [66.29/15]
  def create
  ^^^
app/controllers/courses_controller.rb:75:3: C: Cyclomatic complexity for create is too high. [13/6]
  def create
  ^^^
app/controllers/courses_controller.rb:75:3: C: Method has too many lines. [49/20]
  def create
  ^^^
app/controllers/courses_controller.rb:75:3: C: Perceived complexity for create is too high. [16/7]
  def create
  ^^^
app/controllers/courses_controller.rb:82:5: C: Use snake_case for variable names.
    @newCourse = Course.new(new_course_params)
    ^^^^^^^^^^
app/controllers/courses_controller.rb:88:34: C: Do not use Time.[:now] without zone. Use #Time.zone.[:now] instead.
    @newCourse.start_date = Time.now
                                 ^^^
app/controllers/courses_controller.rb:89:32: C: Do not use Time.[:now] without zone. Use #Time.zone.[:now] instead.
    @newCourse.end_date = Time.now
                               ^^^
app/controllers/courses_controller.rb:107:9: W: Avoid rescuing the Exception class. Perhaps you meant to rescue StandardError?
        rescue Exception => e
        ^^^^^^^^^^^^^^^^^^^^^
app/controllers/courses_controller.rb:114:7: C: Use snake_case for variable names.
      newCUD = @newCourse.course_user_data.new
      ^^^^^^
app/controllers/courses_controller.rb:158:3: C: Assignment Branch Condition size for destroy is too high. [19.67/15]
  def destroy
  ^^^
app/controllers/courses_controller.rb:178:5: C: Use a guard clause instead of wrapping the code inside a conditional expression.
    if request.post?
    ^^
app/controllers/courses_controller.rb:191:3: C: Assignment Branch Condition size for userLookup is too high. [17.58/15]
  def userLookup
  ^^^
app/controllers/courses_controller.rb:191:7: C: Use snake_case for method names.
  def userLookup
      ^^^^^^^^^^
app/controllers/courses_controller.rb:213:101: C: Line is too long. [135/100]
      @cuds = @course.course_user_data.joins(:user).order("users.email ASC").where(CourseUserDatum.conditions_by_like(params[:search]))
                                                                                                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
app/controllers/courses_controller.rb:235:3: C: Assignment Branch Condition size for uploadRoster is too high. [21.21/15]
  def uploadRoster
  ^^^
app/controllers/courses_controller.rb:235:3: C: Cyclomatic complexity for uploadRoster is too high. [8/6]
  def uploadRoster
  ^^^
app/controllers/courses_controller.rb:235:3: C: Perceived complexity for uploadRoster is too high. [9/7]
  def uploadRoster
  ^^^
app/controllers/courses_controller.rb:235:7: C: Use snake_case for method names.
  def uploadRoster
      ^^^^^^^^^^^^
app/controllers/courses_controller.rb:247:7: W: Avoid rescuing the Exception class. Perhaps you meant to rescue StandardError?
      rescue Exception => e
      ^^^^^^^^^^^^^^^^^^^^^
app/controllers/courses_controller.rb:258:3: C: Assignment Branch Condition size for downloadRoster is too high. [16.19/15]
  def downloadRoster
  ^^^
app/controllers/courses_controller.rb:258:7: C: Use snake_case for method names.
  def downloadRoster
      ^^^^^^^^^^^^^^
app/controllers/courses_controller.rb:263:5: C: Prefer each over for.
    for cud in @cuds do
    ^^^
app/controllers/courses_controller.rb:276:7: C: Use snake_case for method names.
  def installAssessment
      ^^^^^^^^^^^^^^^^^
app/controllers/courses_controller.rb:277:5: C: Use snake_case for variable names.
    @assignDir = File.join(Rails.root, "courses", @course.name)
    ^^^^^^^^^^
app/controllers/courses_controller.rb:278:5: C: Use snake_case for variable names.
    @availableAssessments = []
    ^^^^^^^^^^^^^^^^^^^^^
app/controllers/courses_controller.rb:290:7: C: Use snake_case for variable names.
      @availableAssessments = @availableAssessments.sort
      ^^^^^^^^^^^^^^^^^^^^^
app/controllers/courses_controller.rb:291:5: W: Avoid rescuing the Exception class. Perhaps you meant to rescue StandardError?
    rescue Exception => error
    ^^^^^^^^^^^^^^^^^^^^^^^^^
app/controllers/courses_controller.rb:299:3: C: Assignment Branch Condition size for email is too high. [22.05/15]
  def email
  ^^^
app/controllers/courses_controller.rb:299:3: C: Method has too many lines. [21/20]
  def email
  ^^^
app/controllers/courses_controller.rb:300:5: C: Use a guard clause instead of wrapping the code inside a conditional expression.
    if request.post?
    ^^
app/controllers/courses_controller.rb:314:7: C: Use snake_case for variable names.
      bccString = make_dlist(@cuds)
      ^^^^^^^^^
app/controllers/courses_controller.rb:333:3: C: Assignment Branch Condition size for runMoss is too high. [43.61/15]
  def runMoss
  ^^^
app/controllers/courses_controller.rb:333:3: C: Cyclomatic complexity for runMoss is too high. [11/6]
  def runMoss
  ^^^
app/controllers/courses_controller.rb:333:3: C: Method has too many lines. [30/20]
  def runMoss
  ^^^
app/controllers/courses_controller.rb:333:3: C: Perceived complexity for runMoss is too high. [11/7]
  def runMoss
  ^^^
app/controllers/courses_controller.rb:333:7: C: Use snake_case for method names.
  def runMoss
      ^^^^^^^
app/controllers/courses_controller.rb:339:5: C: Use snake_case for variable names.
    assessmentIDs = params[:assessments]
    ^^^^^^^^^^^^^
app/controllers/courses_controller.rb:344:7: C: Prefer each over for.
      for aID in assessmentIDs.keys do
      ^^^
app/controllers/courses_controller.rb:344:11: C: Use snake_case for variable names.
      for aID in assessmentIDs.keys do
          ^^^
app/controllers/courses_controller.rb:350:9: C: Use snake_case for variable names.
        assessmentCUD = assessment.course.course_user_data.joins(:user).find_by(users: { email: current_user.email }, instructor: true)
        ^^^^^^^^^^^^^
app/controllers/courses_controller.rb:350:101: C: Line is too long. [135/100]
        assessmentCUD = assessment.course.course_user_data.joins(:user).find_by(users: { email: current_user.email }, instructor: true)
                                                                                                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
app/controllers/courses_controller.rb:359:5: C: Use snake_case for variable names.
    @mossCmd = [Rails.root.join("vendor", "mossnet -d")]
    ^^^^^^^^
app/controllers/courses_controller.rb:370:5: C: Use snake_case for variable names.
    @mossCmdString = @mossCmd.join(" ")
    ^^^^^^^^^^^^^^
app/controllers/courses_controller.rb:371:5: C: Use snake_case for variable names.
    @mossExit = $CHILD_STATUS
    ^^^^^^^^^
app/controllers/courses_controller.rb:372:5: C: Use snake_case for variable names.
    @mossOutput = `#{@mossCmdString} 2>&1`
    ^^^^^^^^^^^
app/controllers/courses_controller.rb:385:101: C: Line is too long. [121/100]
    params.require(:editCourse).permit(:name, :semester, :late_slack, :grace_days, :display_name, :start_date, :end_date,
                                                                                                    ^^^^^^^^^^^^^^^^^^^^^
app/controllers/courses_controller.rb:386:101: C: Line is too long. [101/100]
                                       :disabled, :exam_in_progress, :version_threshold, :gb_message,
                                                                                                    ^
app/controllers/courses_controller.rb:412:3: C: Assignment Branch Condition size for save_uploaded_roster is too high. [90.87/15]
  def save_uploaded_roster
  ^^^
app/controllers/courses_controller.rb:412:3: C: Cyclomatic complexity for save_uploaded_roster is too high. [10/6]
  def save_uploaded_roster
  ^^^
app/controllers/courses_controller.rb:412:3: C: Method has too many lines. [68/20]
  def save_uploaded_roster
  ^^^
app/controllers/courses_controller.rb:412:3: C: Perceived complexity for save_uploaded_roster is too high. [12/7]
  def save_uploaded_roster
  ^^^
app/controllers/courses_controller.rb:414:7: C: Use snake_case for variable names.
      rowNum = 0
      ^^^^^^
app/controllers/courses_controller.rb:417:9: C: Use snake_case for variable names.
        newCUD = params["cuds"][rowNum.to_s]
        ^^^^^^
app/controllers/courses_controller.rb:433:13: C: Avoid more than 3 levels of block nesting.
            fail "New user cannot be created in uploadRoster." if user.nil?
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
app/controllers/courses_controller.rb:468:101: C: Line is too long. [107/100]
          existing = @course.course_user_data.includes(:user).where(users: { email: newCUD[:email] }).first
                                                                                                    ^^^^^^^
app/controllers/courses_controller.rb:477:101: C: Line is too long. [107/100]
          existing = @course.course_user_data.includes(:user).where(users: { email: newCUD[:email] }).first
                                                                                                    ^^^^^^^
app/controllers/courses_controller.rb:508:9: C: Use snake_case for variable names.
        rowNum += 1
        ^^^^^^
app/controllers/courses_controller.rb:513:3: C: Assignment Branch Condition size for parse_roster_csv is too high. [86.29/15]
  def parse_roster_csv
  ^^^
app/controllers/courses_controller.rb:513:3: C: Cyclomatic complexity for parse_roster_csv is too high. [12/6]
  def parse_roster_csv
  ^^^
app/controllers/courses_controller.rb:513:3: C: Method has too many lines. [53/20]
  def parse_roster_csv
  ^^^
app/controllers/courses_controller.rb:513:3: C: Perceived complexity for parse_roster_csv is too high. [13/7]
  def parse_roster_csv
  ^^^
app/controllers/courses_controller.rb:516:5: C: Use snake_case for variable names.
    @currentCUDs = @course.course_user_data.all.to_a
    ^^^^^^^^^^^^
app/controllers/courses_controller.rb:517:5: C: Use snake_case for variable names.
    @newCUDs = []
    ^^^^^^^^
app/controllers/courses_controller.rb:523:9: C: Use snake_case for variable names.
        newCUD = { email: row[1].to_s,
        ^^^^^^
app/controllers/courses_controller.rb:532:37: W: Shadowing outer local variable - cud.
        cud = @currentCUDs.find do |cud|
                                    ^^^
app/controllers/courses_controller.rb:545:5: W: Avoid rescuing the Exception class. Perhaps you meant to rescue StandardError?
    rescue Exception => e
    ^^^^^^^^^^^^^^^^^^^^^
app/controllers/courses_controller.rb:547:7: W: Unreachable code detected.
      flash[:error] = "Error uploading the CSV file!: " +
      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
app/controllers/courses_controller.rb:553:5: C: Use a guard clause instead of wrapping the code inside a conditional expression.
    if params[:upload][:dropMissing] == "1"
    ^^
app/controllers/courses_controller.rb:558:7: C: Prefer each over for.
      for cud in @currentCUDs do # These are the drops
      ^^^
app/controllers/courses_controller.rb:559:9: C: Use snake_case for variable names.
        newCUD = { email: cud.user.email,
        ^^^^^^
app/controllers/courses_controller.rb:576:3: C: Assignment Branch Condition size for detectAndConvertRoster is too high. [78.39/15]
  def detectAndConvertRoster(roster)
  ^^^
app/controllers/courses_controller.rb:576:3: C: Method has too many lines. [32/20]
  def detectAndConvertRoster(roster)
  ^^^
app/controllers/courses_controller.rb:576:3: C: Perceived complexity for detectAndConvertRoster is too high. [9/7]
  def detectAndConvertRoster(roster)
  ^^^
app/controllers/courses_controller.rb:576:7: C: Use snake_case for method names.
  def detectAndConvertRoster(roster)
      ^^^^^^^^^^^^^^^^^^^^^^
app/controllers/courses_controller.rb:577:5: C: Use snake_case for variable names.
    parsedRoster = CSV.parse(roster)
    ^^^^^^^^^^^^
app/controllers/courses_controller.rb:594:7: C: Use snake_case for variable names.
      numRows = parsedRoster.length - offset
      ^^^^^^^
app/controllers/courses_controller.rb:595:7: C: Use snake_case for variable names.
      convertedRoster = Array.new(numRows) { Array.new(11) }
      ^^^^^^^^^^^^^^^
app/controllers/courses_controller.rb:597:7: C: Prefer each over for.
      for i in 0..(numRows - 1)
      ^^^
app/controllers/courses_controller.rb:623:3: C: Assignment Branch Condition size for extract_asmt_for_moss is too high. [46.01/15]
  def extract_asmt_for_moss(tmp_dir, assessments)
  ^^^
app/controllers/courses_controller.rb:623:3: C: Cyclomatic complexity for extract_asmt_for_moss is too high. [10/6]
  def extract_asmt_for_moss(tmp_dir, assessments)
  ^^^
app/controllers/courses_controller.rb:623:3: C: Method has too many lines. [33/20]
  def extract_asmt_for_moss(tmp_dir, assessments)
  ^^^
app/controllers/courses_controller.rb:623:3: C: Perceived complexity for extract_asmt_for_moss is too high. [10/7]
  def extract_asmt_for_moss(tmp_dir, assessments)
  ^^^
app/controllers/courses_controller.rb:625:5: C: Prefer each over for.
    for ass in assessments do
    ^^^
app/controllers/courses_controller.rb:627:7: C: Use snake_case for variable names.
      assDir = File.join(tmp_dir, "#{ass.name}-#{ass.course.name}")
      ^^^^^^
app/controllers/courses_controller.rb:631:7: C: Use snake_case for variable names.
      isArchive = params[:isArchive] && params[:isArchive][ass.id.to_s]
      ^^^^^^^^^
app/controllers/courses_controller.rb:634:7: C: Prefer each over for.
      for sub in ass.submissions.latest do
      ^^^
app/controllers/courses_controller.rb:634:7: C: Use next to skip iteration.
      for sub in ass.submissions.latest do
      ^^^
app/controllers/courses_controller.rb:635:9: C: Use snake_case for variable names.
        subFile = sub.handin_file_path
        ^^^^^^^
app/controllers/courses_controller.rb:639:9: C: Use snake_case for variable names.
        stuDir = File.join(assDir, sub.course_user_datum.email)
        ^^^^^^
app/controllers/courses_controller.rb:652:29: C: Use next to skip iteration.
            archive_extract.each do |entry|
                            ^^^^
app/controllers/courses_controller.rb:654:15: C: Avoid more than 3 levels of block nesting.
              unless Archive.looks_like_directory?(pathname)
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
app/controllers/courses_controller.rb:655:32: C: Use %r around regular expression.
                pathname.gsub!(/\//, "-")
                               ^^^^
app/controllers/courses_controller.rb:661:19: C: Avoid using rescue in its modifier form.
                  out.fsync rescue nil # for filesystems without fsync(2)
                  ^^^^^^^^^^^^^^^^^^^^
app/controllers/courses_controller.rb:665:11: C: Avoid more than 3 levels of block nesting.
          rescue
          ^^^^^^
app/controllers/courses_controller.rb:676:3: C: Assignment Branch Condition size for extract_tar_for_moss is too high. [24.1/15]
  def extract_tar_for_moss(tmp_dir, external_tar)
  ^^^
app/controllers/courses_controller.rb:676:3: C: Method has too many lines. [26/20]
  def extract_tar_for_moss(tmp_dir, external_tar)
  ^^^
app/controllers/courses_controller.rb:679:5: C: Use snake_case for variable names.
    extTarDir = File.join(tmp_dir, "external_input")
    ^^^^^^^^^
app/controllers/courses_controller.rb:683:5: C: Use snake_case for variable names.
    extTarPath = File.join(extTarDir, "input_file")
    ^^^^^^^^^^
app/controllers/courses_controller.rb:688:5: C: Use snake_case for variable names.
    extFilesDir = File.join(extTarDir, "submissions")
    ^^^^^^^^^^^
app/controllers/courses_controller.rb:697:23: C: Use next to skip iteration.
      archive_extract.each do |entry|
                      ^^^^
app/controllers/courses_controller.rb:700:26: C: Use %r around regular expression.
          pathname.gsub!(/\//, "-")
                         ^^^^
app/controllers/courses_controller.rb:705:13: C: Avoid using rescue in its modifier form.
            out.fsync rescue nil # for filesystems without fsync(2)
            ^^^^^^^^^^^^^^^^^^^^
app/controllers/extensions_controller.rb:20:3: C: Assignment Branch Condition size for create is too high. [17.29/15]
  def create
  ^^^
app/controllers/gradebooks_controller.rb:6:1: C: Missing top-level class documentation comment.
class GradebooksController < ApplicationController
^^^^^
app/controllers/gradebooks_controller.rb:31:5: C: Use a guard clause instead of wrapping the code inside a conditional expression.
    unless @cud.has_auth_level?(:instructor) || @section == @cud.section
    ^^^^^^
app/controllers/gradebooks_controller.rb:38:3: C: Assignment Branch Condition size for student is too high. [24.43/15]
  def student
  ^^^
app/controllers/gradebooks_controller.rb:38:3: C: Cyclomatic complexity for student is too high. [9/6]
  def student
  ^^^
app/controllers/gradebooks_controller.rb:38:3: C: Perceived complexity for student is too high. [10/7]
  def student
  ^^^
app/controllers/gradebooks_controller.rb:39:13: C: Avoid multi-line ?: (the ternary operator); use if/unless instead.
    @_cud = params[:id] ?
            ^^^^^^^^^^^^^
app/controllers/gradebooks_controller.rb:74:3: C: Assignment Branch Condition size for statistics is too high. [22.89/15]
  def statistics
  ^^^
app/controllers/gradebooks_controller.rb:106:7: C: Use snake_case for method names.
  def bulkRelease
      ^^^^^^^^^^^
app/controllers/gradebooks_controller.rb:107:5: C: Prefer each over for.
    for assessment in @course.assessments do
    ^^^
app/controllers/gradebooks_controller.rb:108:7: C: Prefer each over for.
      for problem in assessment.problems do
      ^^^
app/controllers/gradebooks_controller.rb:110:9: C: Prefer each over for.
        for score in scores do
        ^^^
app/controllers/groups_controller.rb:1:1: C: Class definition is too long. [217/100]
class GroupsController < ApplicationController
^^^^^
app/controllers/groups_controller.rb:1:1: C: Missing top-level class documentation comment.
class GroupsController < ApplicationController
^^^^^
app/controllers/groups_controller.rb:14:3: C: Assignment Branch Condition size for index is too high. [16.97/15]
  def index
  ^^^
app/controllers/groups_controller.rb:25:5: C: Use snake_case for variable names.
    @groupAssessments = @course.assessments
    ^^^^^^^^^^^^^^^^^
app/controllers/groups_controller.rb:26:101: C: Line is too long. [126/100]
                        .where("`group_size` > 1 AND `group_size` <= ?", @assessment.group_size).where.not(id: @assessment.id)
                                                                                                    ^^^^^^^^^^^^^^^^^^^^^^^^^^
app/controllers/groups_controller.rb:27:5: C: Use snake_case for variable names.
    @grouplessCUDs = @assessment.grouplessCUDs
    ^^^^^^^^^^^^^^
app/controllers/groups_controller.rb:36:3: C: Assignment Branch Condition size for show is too high. [20.35/15]
  def show
  ^^^
app/controllers/groups_controller.rb:36:3: C: Cyclomatic complexity for show is too high. [8/6]
  def show
  ^^^
app/controllers/groups_controller.rb:36:3: C: Perceived complexity for show is too high. [9/7]
  def show
  ^^^
app/controllers/groups_controller.rb:47:7: C: Use snake_case for variable names.
      @grouplessCUDs = @assessment.grouplessCUDs
      ^^^^^^^^^^^^^^
app/controllers/groups_controller.rb:56:3: C: Assignment Branch Condition size for new is too high. [15.81/15]
  def new
  ^^^
app/controllers/groups_controller.rb:63:5: C: Use snake_case for variable names.
    @grouplessCUDs = @assessment.grouplessCUDs
    ^^^^^^^^^^^^^^
app/controllers/groups_controller.rb:64:5: C: Use snake_case for variable names.
    @unfullGroups = @assessment.groups.all.select { |g| g.assessment_user_data.size < @assessment.group_size }
    ^^^^^^^^^^^^^
app/controllers/groups_controller.rb:64:101: C: Line is too long. [110/100]
    @unfullGroups = @assessment.groups.all.select { |g| g.assessment_user_data.size < @assessment.group_size }
                                                                                                    ^^^^^^^^^^
app/controllers/groups_controller.rb:74:3: C: Assignment Branch Condition size for create is too high. [33.97/15]
  def create
  ^^^
app/controllers/groups_controller.rb:74:3: C: Cyclomatic complexity for create is too high. [8/6]
  def create
  ^^^
app/controllers/groups_controller.rb:74:3: C: Method has too many lines. [25/20]
  def create
  ^^^
app/controllers/groups_controller.rb:74:3: C: Perceived complexity for create is too high. [9/7]
  def create
  ^^^
app/controllers/groups_controller.rb:75:17: W: Assignment in condition - you probably meant to use ==.
    unless cud2 = get_member_cud
                ^
app/controllers/groups_controller.rb:142:3: C: Assignment Branch Condition size for import is too high. [40.87/15]
  def import
  ^^^
app/controllers/groups_controller.rb:142:3: C: Cyclomatic complexity for import is too high. [11/6]
  def import
  ^^^
app/controllers/groups_controller.rb:142:3: C: Method has too many lines. [25/20]
  def import
  ^^^
app/controllers/groups_controller.rb:142:3: C: Perceived complexity for import is too high. [12/7]
  def import
  ^^^
app/controllers/groups_controller.rb:156:30: C: Use next to skip iteration.
      g.assessment_user_data.each do |a|
                             ^^^^
app/controllers/groups_controller.rb:180:3: C: Assignment Branch Condition size for add is too high. [26.74/15]
  def add
  ^^^
app/controllers/groups_controller.rb:180:3: C: Cyclomatic complexity for add is too high. [10/6]
  def add
  ^^^
app/controllers/groups_controller.rb:180:3: C: Perceived complexity for add is too high. [10/7]
  def add
  ^^^
app/controllers/groups_controller.rb:184:16: W: Assignment in condition - you probably meant to use ==.
    unless cud = get_member_cud
               ^
app/controllers/groups_controller.rb:187:5: C: Use snake_case for variable names.
    newMemberAUD = @assessment.aud_for cud.id
    ^^^^^^^^^^^^
app/controllers/groups_controller.rb:189:101: C: Line is too long. [111/100]
    # if we're adding a new member, and not group-confirming someone, make sure that the group is not too large
                                                                                                    ^^^^^^^^^^^
app/controllers/groups_controller.rb:195:101: C: Line is too long. [104/100]
    # if the new member has no previous group or was already in this group, group-confirm the new member
                                                                                                    ^^^^
app/controllers/groups_controller.rb:196:101: C: Line is too long. [111/100]
    if newMemberAUD.membership_status == AssessmentUserDatum::UNCONFIRMED || newMemberAUD.group_id == @group.id
                                                                                                    ^^^^^^^^^^^
app/controllers/groups_controller.rb:211:3: C: Assignment Branch Condition size for join is too high. [18.55/15]
  def join
  ^^^
app/controllers/groups_controller.rb:212:5: C: Use snake_case for variable names.
    newMemberAUD = @assessment.aud_for(@cud.id)
    ^^^^^^^^^^^^
app/controllers/groups_controller.rb:220:101: C: Line is too long. [104/100]
    # if the new member has no previous group or was already in this group, group-confirm the new member
                                                                                                    ^^^^
app/controllers/groups_controller.rb:221:101: C: Line is too long. [111/100]
    if newMemberAUD.membership_status == AssessmentUserDatum::UNCONFIRMED || newMemberAUD.group_id == @group.id
                                                                                                    ^^^^^^^^^^^
app/controllers/groups_controller.rb:238:3: C: Assignment Branch Condition size for leave is too high. [30.1/15]
  def leave
  ^^^
app/controllers/groups_controller.rb:238:3: C: Cyclomatic complexity for leave is too high. [8/6]
  def leave
  ^^^
app/controllers/groups_controller.rb:238:3: C: Method has too many lines. [23/20]
  def leave
  ^^^
app/controllers/groups_controller.rb:238:3: C: Perceived complexity for leave is too high. [9/7]
  def leave
  ^^^
app/controllers/groups_controller.rb:267:5: C: Use a guard clause instead of wrapping the code inside a conditional expression.
    unless @assessment.has_groups?
    ^^^^^^
app/controllers/groups_controller.rb:285:3: C: Assignment Branch Condition size for get_member_cud is too high. [22.38/15]
  def get_member_cud
  ^^^
app/controllers/groups_controller.rb:285:7: C: Do not prefix reader method names with get_.
  def get_member_cud
      ^^^^^^^^^^^^^^
app/controllers/groups_controller.rb:290:11: C: Redundant else-clause.
          else
          ^^^^
app/controllers/groups_controller.rb:292:5: W: end at 292, 4 is not aligned with if at 286, 10
    end
    ^^^
app/controllers/home_controller.rb:28:3: C: Assignment Branch Condition size for publicSignUp is too high. [27.09/15]
  def publicSignUp
  ^^^
app/controllers/home_controller.rb:28:3: C: Cyclomatic complexity for publicSignUp is too high. [8/6]
  def publicSignUp
  ^^^
app/controllers/home_controller.rb:28:3: C: Perceived complexity for publicSignUp is too high. [9/7]
  def publicSignUp
  ^^^
app/controllers/home_controller.rb:28:7: C: Use snake_case for method names.
  def publicSignUp
      ^^^^^^^^^^^^
app/controllers/jobs_controller.rb:8:1: C: Class definition is too long. [256/100]
class JobsController < ApplicationController
^^^^^
app/controllers/jobs_controller.rb:14:3: C: Assignment Branch Condition size for index is too high. [34.94/15]
  def index
  ^^^
app/controllers/jobs_controller.rb:14:3: C: Cyclomatic complexity for index is too high. [9/6]
  def index
  ^^^
app/controllers/jobs_controller.rb:14:3: C: Method has too many lines. [28/20]
  def index
  ^^^
app/controllers/jobs_controller.rb:14:3: C: Perceived complexity for index is too high. [10/7]
  def index
  ^^^
app/controllers/jobs_controller.rb:30:40: W: Useless assignment to variable - deadjobs.
      raw_dead_jobs = TangoClient.jobs(deadjobs = 1)
                                       ^^^^^^^^
app/controllers/jobs_controller.rb:65:3: C: Assignment Branch Condition size for getjob is too high. [66.88/15]
  def getjob
  ^^^
app/controllers/jobs_controller.rb:65:3: C: Cyclomatic complexity for getjob is too high. [23/6]
  def getjob
  ^^^
app/controllers/jobs_controller.rb:65:3: C: Method has too many lines. [66/20]
  def getjob
  ^^^
app/controllers/jobs_controller.rb:65:3: C: Perceived complexity for getjob is too high. [25/7]
  def getjob
  ^^^
app/controllers/jobs_controller.rb:77:40: W: Useless assignment to variable - deadjobs.
      raw_dead_jobs = TangoClient.jobs(deadjobs = 1)
                                       ^^^^^^^^
app/controllers/jobs_controller.rb:161:3: C: Assignment Branch Condition size for tango_status is too high. [28.79/15]
  def tango_status
  ^^^
app/controllers/jobs_controller.rb:176:41: W: Useless assignment to variable - deadjobs. Did you mean dead_jobs?
    @tango_dead_jobs = TangoClient.jobs(deadjobs = 1)
                                        ^^^^^^^^
app/controllers/jobs_controller.rb:177:34: W: Useless assignment to variable - live_jobs.
    @plot_data = tango_plot_data(live_jobs = @tango_live_jobs, dead_jobs = @tango_dead_jobs)
                                 ^^^^^^^^^
app/controllers/jobs_controller.rb:177:64: W: Useless assignment to variable - dead_jobs. Did you mean deadjobs?
    @plot_data = tango_plot_data(live_jobs = @tango_live_jobs, dead_jobs = @tango_dead_jobs)
                                                               ^^^^^^^^^
app/controllers/jobs_controller.rb:181:76: C: Do not use Time.[:now] without zone. Use #Time.zone.[:now] instead.
      @upcoming_asmt << asmt if asmt.has_autograder? && asmt.due_at > Time.now
                                                                           ^^^
app/controllers/jobs_controller.rb:196:3: C: Assignment Branch Condition size for formatRawJob is too high. [77.11/15]
  def formatRawJob(rjob, is_live)
  ^^^
app/controllers/jobs_controller.rb:196:3: C: Cyclomatic complexity for formatRawJob is too high. [12/6]
  def formatRawJob(rjob, is_live)
  ^^^
app/controllers/jobs_controller.rb:196:3: C: Method has too many lines. [42/20]
  def formatRawJob(rjob, is_live)
  ^^^
app/controllers/jobs_controller.rb:196:3: C: Perceived complexity for formatRawJob is too high. [16/7]
  def formatRawJob(rjob, is_live)
  ^^^
app/controllers/jobs_controller.rb:196:7: C: Use snake_case for method names.
  def formatRawJob(rjob, is_live)
      ^^^^^^^^^^^^
app/controllers/jobs_controller.rb:227:21: C: Do not use DateTime.[:parse] without zone. Use #Time.zone.[:parse] instead.
      t1 = DateTime.parse(job[:first]).to_time
                    ^^^^^
app/controllers/jobs_controller.rb:227:40: C: Do not use to_time on Date objects,because it knows nothing about time zone in use.
      t1 = DateTime.parse(job[:first]).to_time
                                       ^^^^^^^
app/controllers/jobs_controller.rb:229:21: C: Do not use Time.[:now] without zone. Use #Time.zone.[:now] instead.
        snow = Time.now.localtime.to_s
                    ^^^
app/controllers/jobs_controller.rb:230:23: C: Do not use DateTime.[:parse] without zone. Use #Time.zone.[:parse] instead.
        t2 = DateTime.parse(snow).to_time
                      ^^^^^
app/controllers/jobs_controller.rb:230:35: C: Do not use to_time on Date objects,because it knows nothing about time zone in use.
        t2 = DateTime.parse(snow).to_time
                                  ^^^^^^^
app/controllers/jobs_controller.rb:232:23: C: Do not use DateTime.[:parse] without zone. Use #Time.zone.[:parse] instead.
        t2 = DateTime.parse(job[:last]).to_time
                      ^^^^^
app/controllers/jobs_controller.rb:232:41: C: Do not use to_time on Date objects,because it knows nothing about time zone in use.
        t2 = DateTime.parse(job[:last]).to_time
                                        ^^^^^^^
app/controllers/jobs_controller.rb:255:3: C: Assignment Branch Condition size for tango_plot_data is too high. [196.6/15]
  def tango_plot_data(live_jobs = nil, dead_jobs = nil)
  ^^^
app/controllers/jobs_controller.rb:255:3: C: Cyclomatic complexity for tango_plot_data is too high. [18/6]
  def tango_plot_data(live_jobs = nil, dead_jobs = nil)
  ^^^
app/controllers/jobs_controller.rb:255:3: C: Method has too many lines. [82/20]
  def tango_plot_data(live_jobs = nil, dead_jobs = nil)
  ^^^
app/controllers/jobs_controller.rb:255:3: C: Perceived complexity for tango_plot_data is too high. [19/7]
  def tango_plot_data(live_jobs = nil, dead_jobs = nil)
  ^^^
app/controllers/jobs_controller.rb:257:36: W: Useless assignment to variable - deadjobs. Did you mean dead_jobs?
    dead_jobs ||= TangoClient.jobs(deadjobs = 1)
                                   ^^^^^^^^
app/controllers/jobs_controller.rb:273:23: C: Do not use Time.[:parse] without zone. Use #Time.zone.[:parse] instead.
      duration = Time.parse(j["trace"].last.split("|")[0]).to_i - Time.parse(j["trace"].first.split("|")[0]).to_i
                      ^^^^^
app/controllers/jobs_controller.rb:273:72: C: Do not use Time.[:parse] without zone. Use #Time.zone.[:parse] instead.
      duration = Time.parse(j["trace"].last.split("|")[0]).to_i - Time.parse(j["trace"].first.split("|")[0]).to_i
                                                                       ^^^^^
app/controllers/jobs_controller.rb:273:101: C: Line is too long. [113/100]
      duration = Time.parse(j["trace"].last.split("|")[0]).to_i - Time.parse(j["trace"].first.split("|")[0]).to_i
                                                                                                    ^^^^^^^^^^^^^
app/controllers/jobs_controller.rb:303:23: C: Do not use Time.[:parse] without zone. Use #Time.zone.[:parse] instead.
      duration = Time.parse(j["trace"].last.split("|")[0]).to_i - Time.parse(j["trace"].first.split("|")[0]).to_i
                      ^^^^^
app/controllers/jobs_controller.rb:303:72: C: Do not use Time.[:parse] without zone. Use #Time.zone.[:parse] instead.
      duration = Time.parse(j["trace"].last.split("|")[0]).to_i - Time.parse(j["trace"].first.split("|")[0]).to_i
                                                                       ^^^^^
app/controllers/jobs_controller.rb:303:101: C: Line is too long. [113/100]
      duration = Time.parse(j["trace"].last.split("|")[0]).to_i - Time.parse(j["trace"].first.split("|")[0]).to_i
                                                                                                    ^^^^^^^^^^^^^
app/controllers/scoreboards_controller.rb:4:1: C: Class definition is too long. [240/100]
class ScoreboardsController < ApplicationController
^^^^^
app/controllers/scoreboards_controller.rb:21:3: C: Assignment Branch Condition size for show is too high. [84.22/15]
  def show
  ^^^
app/controllers/scoreboards_controller.rb:21:3: C: Cyclomatic complexity for show is too high. [13/6]
  def show
  ^^^
app/controllers/scoreboards_controller.rb:21:3: C: Method has too many lines. [77/20]
  def show
  ^^^
app/controllers/scoreboards_controller.rb:21:3: C: Perceived complexity for show is too high. [15/7]
  def show
  ^^^
app/controllers/scoreboards_controller.rb:24:5: C: Use snake_case for variable names.
    problemQuery = "SELECT scores.score AS score,
    ^^^^^^^^^^^^
app/controllers/scoreboards_controller.rb:78:11: C: Use snake_case for variable names.
          @errorMessage = "An error occurred while calling " \
          ^^^^^^^^^^^^^
app/controllers/scoreboards_controller.rb:82:45: W: Non-local exit from iterator, without return value. next, break, Array#find, Array#any?, etc. is preferred.
          render([@course, @assessment]) && return
                                            ^^^^^^
app/controllers/scoreboards_controller.rb:96:5: C: Use snake_case for variable names.
    @sortedGrades = @grades.values.sort do |a, b|
    ^^^^^^^^^^^^^
app/controllers/scoreboards_controller.rb:107:11: C: Use snake_case for variable names.
          @errorMessage = "An error occurred while calling "\
          ^^^^^^^^^^^^^
app/controllers/scoreboards_controller.rb:111:45: W: Non-local exit from iterator, without return value. next, break, Array#find, Array#any?, etc. is preferred.
          render([@course, @assessment]) && return
                                            ^^^^^^
app/controllers/scoreboards_controller.rb:118:5: C: Use a guard clause instead of wrapping the code inside a conditional expression.
    unless @scoreboard.colspec.blank?
    ^^^^^^
app/controllers/scoreboards_controller.rb:162:3: C: Assignment Branch Condition size for emitColSpec is too high. [24.82/15]
  def emitColSpec(colspec)
  ^^^
app/controllers/scoreboards_controller.rb:162:3: C: Cyclomatic complexity for emitColSpec is too high. [7/6]
  def emitColSpec(colspec)
  ^^^
app/controllers/scoreboards_controller.rb:162:3: C: Method has too many lines. [28/20]
  def emitColSpec(colspec)
  ^^^
app/controllers/scoreboards_controller.rb:162:3: C: Perceived complexity for emitColSpec is too high. [8/7]
  def emitColSpec(colspec)
  ^^^
app/controllers/scoreboards_controller.rb:162:7: C: Use snake_case for method names.
  def emitColSpec(colspec)
      ^^^^^^^^^^^
app/controllers/scoreboards_controller.rb:169:29: W: Useless assignment to variable - e.
    rescue StandardError => e
                            ^
app/controllers/scoreboards_controller.rb:208:3: C: Assignment Branch Condition size for createScoreboardEntry is too high. [35.07/15]
  def createScoreboardEntry(scores, autoresult)
  ^^^
app/controllers/scoreboards_controller.rb:208:3: C: Cyclomatic complexity for createScoreboardEntry is too high. [14/6]
  def createScoreboardEntry(scores, autoresult)
  ^^^
app/controllers/scoreboards_controller.rb:208:3: C: Method has too many lines. [32/20]
  def createScoreboardEntry(scores, autoresult)
  ^^^
app/controllers/scoreboards_controller.rb:208:3: C: Perceived complexity for createScoreboardEntry is too high. [14/7]
  def createScoreboardEntry(scores, autoresult)
  ^^^
app/controllers/scoreboards_controller.rb:208:7: C: Use snake_case for method names.
  def createScoreboardEntry(scores, autoresult)
      ^^^^^^^^^^^^^^^^^^^^^
app/controllers/scoreboards_controller.rb:219:7: C: Prefer each over for.
      for problem in @assessment.problems do
      ^^^
app/controllers/scoreboards_controller.rb:226:7: C: Prefer each over for.
      for problem in @assessment.problems do
      ^^^
app/controllers/scoreboards_controller.rb:247:9: C: Prefer each over for.
        for item in parsed["scoreboard"] do
        ^^^
app/controllers/scoreboards_controller.rb:247:13: W: Useless assignment to variable - item.
        for item in parsed["scoreboard"] do
            ^^^^
app/controllers/scoreboards_controller.rb:276:3: C: Assignment Branch Condition size for scoreboardOrderSubmissions is too high. [78.7/15]
  def scoreboardOrderSubmissions(a, b)
  ^^^
app/controllers/scoreboards_controller.rb:276:3: C: Cyclomatic complexity for scoreboardOrderSubmissions is too high. [21/6]
  def scoreboardOrderSubmissions(a, b)
  ^^^
app/controllers/scoreboards_controller.rb:276:3: C: Method has too many lines. [53/20]
  def scoreboardOrderSubmissions(a, b)
  ^^^
app/controllers/scoreboards_controller.rb:276:3: C: Perceived complexity for scoreboardOrderSubmissions is too high. [27/7]
  def scoreboardOrderSubmissions(a, b)
  ^^^
app/controllers/scoreboards_controller.rb:276:7: C: Use snake_case for method names.
  def scoreboardOrderSubmissions(a, b)
      ^^^^^^^^^^^^^^^^^^^^^^^^^^
app/controllers/scoreboards_controller.rb:282:7: C: Use snake_case for variable names.
      aSum = 0; bSum = 0
      ^^^^
app/controllers/scoreboards_controller.rb:282:15: C: Do not use semicolons to terminate expressions.
      aSum = 0; bSum = 0
              ^
app/controllers/scoreboards_controller.rb:282:17: C: Use snake_case for variable names.
      aSum = 0; bSum = 0
                ^^^^
app/controllers/scoreboards_controller.rb:284:9: C: Use snake_case for variable names.
        aSum += a[:problems][key].to_f
        ^^^^
app/controllers/scoreboards_controller.rb:287:9: C: Use snake_case for variable names.
        bSum += b[:problems][key].to_f
        ^^^^
app/controllers/scoreboards_controller.rb:310:7: W: Do not suppress exceptions.
      rescue
      ^^^^^^
app/controllers/scores_controller.rb:11:3: C: Assignment Branch Condition size for create is too high. [21.05/15]
  def create
  ^^^
app/controllers/scores_controller.rb:33:3: C: Assignment Branch Condition size for update is too high. [15.13/15]
  def update
  ^^^
app/controllers/submissions_controller.rb:4:1: C: Class definition is too long. [250/100]
class SubmissionsController < ApplicationController
^^^^^
app/controllers/submissions_controller.rb:4:1: C: Missing top-level class documentation comment.
class SubmissionsController < ApplicationController
^^^^^
app/controllers/submissions_controller.rb:7:101: C: Line is too long. [114/100]
  before_action :set_submission, only: [:destroy, :destroyConfirm, :download, :edit, :listArchive, :update, :view]
                                                                                                    ^^^^^^^^^^^^^^
app/controllers/submissions_controller.rb:16:5: W: Useless assignment to variable - modName.
    modName = (assign + (@course.name).gsub(/[^A-Za-z0-9]/, "")).camelize
    ^^^^^^^
app/controllers/submissions_controller.rb:16:5: C: Use snake_case for variable names.
    modName = (assign + (@course.name).gsub(/[^A-Za-z0-9]/, "")).camelize
    ^^^^^^^
app/controllers/submissions_controller.rb:22:3: C: Assignment Branch Condition size for new is too high. [26.02/15]
  def new
  ^^^
app/controllers/submissions_controller.rb:29:9: C: Use snake_case for variable names.
        @errorMessage = "Couldn't find all course_user_data in #{cuds_ids}. " \
        ^^^^^^^^^^^^^
app/controllers/submissions_controller.rb:37:7: C: Prefer each over for.
      for cud in @course.course_user_data.joins(:user).order("email ASC") do
      ^^^
app/controllers/submissions_controller.rb:45:3: C: Assignment Branch Condition size for create is too high. [55.77/15]
  def create
  ^^^
app/controllers/submissions_controller.rb:45:3: C: Cyclomatic complexity for create is too high. [8/6]
  def create
  ^^^
app/controllers/submissions_controller.rb:45:3: C: Method has too many lines. [25/20]
  def create
  ^^^
app/controllers/submissions_controller.rb:45:3: C: Perceived complexity for create is too high. [8/7]
  def create
  ^^^
app/controllers/submissions_controller.rb:52:7: C: Use snake_case for variable names.
      @errorMessage = "Invalid CourseUserDatum ID in #{cud_ids}"
      ^^^^^^^^^^^^^
app/controllers/submissions_controller.rb:55:5: C: Prefer each over for.
    for cud_id in cud_ids do
    ^^^
app/controllers/submissions_controller.rb:55:5: C: Use next to skip iteration.
    for cud_id in cud_ids do
    ^^^
app/controllers/submissions_controller.rb:60:101: C: Line is too long. [112/100]
        @submission.tweak = Tweak.new(params[:submission][:tweak_attributes].permit([:value, :kind, :_destroy]))
                                                                                                    ^^^^^^^^^^^^
app/controllers/submissions_controller.rb:77:5: W: Useless assignment to variable - submission.
    submission = Submission.find(params[:id])
    ^^^^^^^^^^
app/controllers/submissions_controller.rb:107:3: C: Assignment Branch Condition size for update is too high. [19.42/15]
  def update
  ^^^
app/controllers/submissions_controller.rb:112:101: C: Line is too long. [110/100]
      redirect_to(history_course_assessment_path(@submission.course_user_datum.course, @assessment)) && return
                                                                                                    ^^^^^^^^^^
app/controllers/submissions_controller.rb:114:101: C: Line is too long. [131/100]
      redirect_to(edit_course_assessment_submission_path(@submission.course_user_datum.course, @assessment, @submission)) && return
                                                                                                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
app/controllers/submissions_controller.rb:126:101: C: Line is too long. [123/100]
    redirect_to(course_assessment_submissions_path(@submission.course_user_datum.course, @submission.assessment)) && return
                                                                                                    ^^^^^^^^^^^^^^^^^^^^^^^
app/controllers/submissions_controller.rb:131:7: C: Use snake_case for method names.
  def destroyConfirm
      ^^^^^^^^^^^^^^
app/controllers/submissions_controller.rb:140:3: C: Assignment Branch Condition size for missing is too high. [18.47/15]
  def missing
  ^^^
app/controllers/submissions_controller.rb:146:5: C: Prefer each over for.
    for submission in @submissions do
    ^^^
app/controllers/submissions_controller.rb:160:3: C: Assignment Branch Condition size for downloadAll is too high. [30.68/15]
  def downloadAll
  ^^^
app/controllers/submissions_controller.rb:160:3: C: Cyclomatic complexity for downloadAll is too high. [9/6]
  def downloadAll
  ^^^
app/controllers/submissions_controller.rb:160:3: C: Method has too many lines. [21/20]
  def downloadAll
  ^^^
app/controllers/submissions_controller.rb:160:3: C: Perceived complexity for downloadAll is too high. [10/7]
  def downloadAll
  ^^^
app/controllers/submissions_controller.rb:160:7: C: Use snake_case for method names.
  def downloadAll
      ^^^^^^^^^^^
app/controllers/submissions_controller.rb:193:3: C: Assignment Branch Condition size for download is too high. [15.39/15]
  def download
  ^^^
app/controllers/submissions_controller.rb:198:44: C: Use && instead of and.
        redirect_to [@course, @assessment] and return false
                                           ^^^
app/controllers/submissions_controller.rb:205:7: W: Useless assignment to variable - mime.
      mime = params[:forceMime] || @submission.detected_mime_type
      ^^^^
app/controllers/submissions_controller.rb:218:3: C: Assignment Branch Condition size for view is too high. [86.63/15]
  def view
  ^^^
app/controllers/submissions_controller.rb:218:3: C: Cyclomatic complexity for view is too high. [25/6]
  def view
  ^^^
app/controllers/submissions_controller.rb:218:3: C: Method has too many lines. [63/20]
  def view
  ^^^
app/controllers/submissions_controller.rb:218:3: C: Perceived complexity for view is too high. [28/7]
  def view
  ^^^
app/controllers/submissions_controller.rb:220:101: C: Line is too long. [104/100]
      file, pathname = Archive.get_nth_file(@submission.handin_file_path, params[:header_position].to_i)
                                                                                                    ^^^^
app/controllers/submissions_controller.rb:223:44: C: Use && instead of and.
        redirect_to [@course, @assessment] and return false
                                           ^^^
app/controllers/submissions_controller.rb:226:7: C: Use snake_case for variable names.
      @displayFilename = pathname
      ^^^^^^^^^^^^^^^^
app/controllers/submissions_controller.rb:227:101: C: Line is too long. [111/100]
      @breadcrumbs << (view_context.link_to "View Archive", [:list_archive, @course, @assessment, @submission])
                                                                                                    ^^^^^^^^^^^
app/controllers/submissions_controller.rb:234:7: C: Use snake_case for variable names.
      @displayFilename = @submission.filename
      ^^^^^^^^^^^^^^^^
app/controllers/submissions_controller.rb:240:101: C: Line is too long. [114/100]
      send_data(file, type: "application/pdf", disposition: "inline", filename: File.basename(filename)) && return
                                                                                                    ^^^^^^^^^^^^^^
app/controllers/submissions_controller.rb:246:101: C: Line is too long. [158/100]
      flash[:error] = "Sorry, we could not display your file because it contains non-ASCII characters. Please remove these characters and resubmit your work."
                                                                                                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
app/controllers/submissions_controller.rb:252:7: C: Prefer each over for.
      for i in 0...@data.length do
      ^^^
app/controllers/submissions_controller.rb:257:101: C: Line is too long. [142/100]
      flash[:error] = "Sorry, we could not parse your file because it contains non-ASCII characters. Please download file to view the source."
                                                                                                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
app/controllers/submissions_controller.rb:270:5: C: Use snake_case for variable names.
    @problemSummaries = {}
    ^^^^^^^^^^^^^^^^^
app/controllers/submissions_controller.rb:271:5: C: Use snake_case for variable names.
    @problemGrades = {}
    ^^^^^^^^^^^^^^
app/controllers/submissions_controller.rb:274:5: C: Prefer each over for.
    for annotation in @annotations do
    ^^^
app/controllers/submissions_controller.rb:281:101: C: Line is too long. [102/100]
      @problemSummaries[problem] << [description, value, line, annotation.submitted_by, annotation.id]
                                                                                                    ^^
app/controllers/submissions_controller.rb:299:101: C: Line is too long. [105/100]
        redirect_to([:history, @course, @assessment, cud_id: @submission.course_user_datum_id]) && return
                                                                                                    ^^^^^
app/controllers/submissions_controller.rb:307:7: C: Use snake_case for method names.
  def listArchive
      ^^^^^^^^^^^
app/controllers/submissions_controller.rb:323:7: C: Do not prefix reader method names with get_.
  def get_submission_file
      ^^^^^^^^^^^^^^^^^^^
app/controllers/submissions_controller.rb:326:42: C: Use && instead of and.
      redirect_to [@course, @assessment] and return false
                                         ^^^
app/controllers/submissions_controller.rb:334:42: C: Use && instead of and.
      redirect_to [@course, @assessment] and return false
                                         ^^^
app/controllers/submissions_controller.rb:342:7: C: Use snake_case for method names.
  def extractAndrewID(filename)
      ^^^^^^^^^^^^^^^
app/controllers/submissions_controller.rb:343:5: C: Use snake_case for variable names.
    underscoreInd = filename.index("_")
    ^^^^^^^^^^^^^
app/controllers/submissions_controller.rb:350:7: C: Use snake_case for method names.
  def extractVersion(filename)
      ^^^^^^^^^^^^^^
app/controllers/submissions_controller.rb:351:5: C: Use snake_case for variable names.
    firstUnderscoreInd = filename.index("_")
    ^^^^^^^^^^^^^^^^^^
app/controllers/submissions_controller.rb:354:5: C: Use snake_case for variable names.
    secondUnderscoreInd = filename.index("_", firstUnderscoreInd + 1)
    ^^^^^^^^^^^^^^^^^^^
app/controllers/users/omniauth_callbacks_controller.rb:1:1: C: Missing top-level class documentation comment.
class Users::OmniauthCallbacksController < Devise::OmniauthCallbacksController
^^^^^
app/controllers/users/omniauth_callbacks_controller.rb:1:7: C: Use nested module/class definitions instead of compact style.
class Users::OmniauthCallbacksController < Devise::OmniauthCallbacksController
      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
app/controllers/users/omniauth_callbacks_controller.rb:2:3: C: Assignment Branch Condition size for facebook is too high. [33.79/15]
  def facebook
  ^^^
app/controllers/users/omniauth_callbacks_controller.rb:2:3: C: Cyclomatic complexity for facebook is too high. [8/6]
  def facebook
  ^^^
app/controllers/users/omniauth_callbacks_controller.rb:2:3: C: Perceived complexity for facebook is too high. [10/7]
  def facebook
  ^^^
app/controllers/users/omniauth_callbacks_controller.rb:4:15: W: Assignment in condition - you probably meant to use ==.
      if data = request.env["omniauth.auth"]
              ^
app/controllers/users/omniauth_callbacks_controller.rb:17:101: C: Line is too long. [102/100]
        sign_in_and_redirect @user, event: :authentication # this will throw if @user is not activated
                                                                                                    ^^
app/controllers/users/omniauth_callbacks_controller.rb:28:3: C: Assignment Branch Condition size for google_oauth2 is too high. [33.44/15]
  def google_oauth2
  ^^^
app/controllers/users/omniauth_callbacks_controller.rb:28:3: C: Perceived complexity for google_oauth2 is too high. [8/7]
  def google_oauth2
  ^^^
app/controllers/users/omniauth_callbacks_controller.rb:30:15: W: Assignment in condition - you probably meant to use ==.
      if data = request.env["omniauth.auth"]
              ^
app/controllers/users/omniauth_callbacks_controller.rb:42:101: C: Line is too long. [102/100]
        sign_in_and_redirect @user, event: :authentication # this will throw if @user is not activated
                                                                                                    ^^
app/controllers/users/omniauth_callbacks_controller.rb:52:3: C: Assignment Branch Condition size for shibboleth is too high. [68.26/15]
  def shibboleth
  ^^^
app/controllers/users/omniauth_callbacks_controller.rb:52:3: C: Cyclomatic complexity for shibboleth is too high. [12/6]
  def shibboleth
  ^^^
app/controllers/users/omniauth_callbacks_controller.rb:52:3: C: Method has too many lines. [44/20]
  def shibboleth
  ^^^
app/controllers/users/omniauth_callbacks_controller.rb:52:3: C: Perceived complexity for shibboleth is too high. [14/7]
  def shibboleth
  ^^^
app/controllers/users/omniauth_callbacks_controller.rb:54:15: W: Assignment in condition - you probably meant to use ==.
      if data = request.env["omniauth.auth"]
              ^
app/controllers/users/omniauth_callbacks_controller.rb:66:101: C: Line is too long. [102/100]
        sign_in_and_redirect @user, event: :authentication # this will throw if @user is not activated
                                                                                                    ^^
app/controllers/users/omniauth_callbacks_controller.rb:79:11: C: Avoid more than 3 levels of block nesting.
          if @user.email.include? "@andrew.cmu.edu"
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
app/controllers/users/omniauth_callbacks_controller.rb:80:13: C: Use snake_case for variable names.
            ldapResult = User.ldap_lookup(@user.email.split("@")[0])
            ^^^^^^^^^^
app/controllers/users/omniauth_callbacks_controller.rb:91:11: C: Avoid more than 3 levels of block nesting.
          @user.first_name = "(blank)" if @user.first_name.nil?
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
app/controllers/users/omniauth_callbacks_controller.rb:92:11: C: Avoid more than 3 levels of block nesting.
          @user.last_name = "(blank)" if @user.last_name.nil?
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
app/controllers/users/omniauth_callbacks_controller.rb:104:101: C: Line is too long. [102/100]
        sign_in_and_redirect @user, event: :authentication # this will throw if @user is not activated
                                                                                                    ^^
app/controllers/users_controller.rb:1:1: C: Class definition is too long. [154/100]
class UsersController < ApplicationController
^^^^^
app/controllers/users_controller.rb:1:1: C: Missing top-level class documentation comment.
class UsersController < ApplicationController
^^^^^
app/controllers/users_controller.rb:28:3: C: Assignment Branch Condition size for show is too high. [34.47/15]
  def show
  ^^^
app/controllers/users_controller.rb:28:3: C: Cyclomatic complexity for show is too high. [9/6]
  def show
  ^^^
app/controllers/users_controller.rb:28:3: C: Method has too many lines. [29/20]
  def show
  ^^^
app/controllers/users_controller.rb:28:3: C: Perceived complexity for show is too high. [11/7]
  def show
  ^^^
app/controllers/users_controller.rb:44:12: C: Use next to skip iteration.
      cuds.each do |cud|
           ^^^^
app/controllers/users_controller.rb:84:3: C: Assignment Branch Condition size for create is too high. [26.65/15]
  def create
  ^^^
app/controllers/users_controller.rb:84:3: C: Perceived complexity for create is too high. [8/7]
  def create
  ^^^
app/controllers/users_controller.rb:112:3: C: Assignment Branch Condition size for edit is too high. [17.97/15]
  def edit
  ^^^
app/controllers/users_controller.rb:112:3: C: Perceived complexity for edit is too high. [8/7]
  def edit
  ^^^
app/controllers/users_controller.rb:134:3: C: Assignment Branch Condition size for update is too high. [33.66/15]
  def update
  ^^^
app/controllers/users_controller.rb:134:3: C: Cyclomatic complexity for update is too high. [11/6]
  def update
  ^^^
app/controllers/users_controller.rb:134:3: C: Method has too many lines. [24/20]
  def update
  ^^^
app/controllers/users_controller.rb:134:3: C: Perceived complexity for update is too high. [14/7]
  def update
  ^^^
app/controllers/users_controller.rb:154:20: C: Avoid multi-line ?: (the ternary operator); use if/unless instead.
    if user.update(current_user.administrator? ?
                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
app/controllers/users_controller.rb:166:3: C: Assignment Branch Condition size for destroy is too high. [19.67/15]
  def destroy
  ^^^
app/controllers/users_controller.rb:178:7: C: Annotation keywords should be all upper case, followed by a colon and a space, then a note describing the problem.
    # TODO Need to cleanup user resources here
      ^^^^
app/form_builders/form_builder_with_date_time_input.rb:84:42: C: Do not use to_time on Date objects,because it knows nothing about time zone in use.
      formatted_datetime = existing_time.to_time.strftime(strftime)
                                         ^^^^^^^
app/helpers/account_helper.rb:1:1: C: Missing top-level module documentation comment.
module AccountHelper
^^^^^^
app/helpers/admin_helper.rb:1:1: C: Missing top-level module documentation comment.
module AdminHelper
^^^^^^
app/helpers/annotations_helper.rb:1:1: C: Missing top-level module documentation comment.
module AnnotationsHelper
^^^^^^
app/helpers/application_helper.rb:19:7: C: Use snake_case for method names.
  def processStackDump(dump)
      ^^^^^^^^^^^^^^^^
app/helpers/application_helper.rb:25:7: C: Use snake_case for method names.
  def roundValue(val)
      ^^^^^^^^^^
app/helpers/application_helper.rb:79:5: C: Use a guard clause instead of wrapping the code inside a conditional expression.
    if submission.syntax?
    ^^
app/helpers/application_helper.rb:86:5: C: Use a guard clause instead of wrapping the code inside a conditional expression.
    if Archive.archive? submission.handin_file_path
    ^^
app/helpers/application_helper.rb:87:101: C: Line is too long. [102/100]
      link_to title, list_archive_course_assessment_submission_path(@course, @assessment, submission),
                                                                                                    ^^
app/helpers/application_helper.rb:103:63: C: Favor format over String#%.
    image_tag("/images/go.png", style: "vertical-align: %dpx" % vertical_align)
                                                              ^
app/helpers/application_helper.rb:111:3: C: Cyclomatic complexity for computed_score is too high. [7/6]
  def computed_score(link = nil, nil_to_dash = true)
  ^^^
app/helpers/application_helper.rb:115:39: W: Useless assignment to variable - e.
  rescue ScoreComputationException => e
                                      ^
app/helpers/application_helper.rb:152:3: C: Cyclomatic complexity for external_javascript_include_tag is too high. [11/6]
  def external_javascript_include_tag(library, version)
  ^^^
app/helpers/application_helper.rb:152:3: C: Method has too many lines. [24/20]
  def external_javascript_include_tag(library, version)
  ^^^
app/helpers/application_helper.rb:168:101: C: Line is too long. [102/100]
      javascript_include_tag "#{cloudflare}/backbone-relational/#{version}/backbone-relational.min.js"
                                                                                                    ^^
app/helpers/assessment_helper.rb:1:1: C: Missing top-level module documentation comment.
module AssessmentHelper
^^^^^^
app/helpers/assessment_helper.rb:2:3: C: Assignment Branch Condition size for stats_table is too high. [19.92/15]
  def stats_table(data)
  ^^^
app/helpers/assessment_helper.rb:2:3: C: Method has too many lines. [27/20]
  def stats_table(data)
  ^^^
app/helpers/assessment_helper.rb:62:3: C: Assignment Branch Condition size for csv_row_for is too high. [29.02/15]
  def csv_row_for(asmt, cud, as_seen_by)
  ^^^
app/helpers/assessment_helper.rb:62:3: C: Cyclomatic complexity for csv_row_for is too high. [8/6]
  def csv_row_for(asmt, cud, as_seen_by)
  ^^^
app/helpers/assessment_helper.rb:62:3: C: Method has too many lines. [24/20]
  def csv_row_for(asmt, cud, as_seen_by)
  ^^^
app/helpers/assessment_helper.rb:62:3: C: Perceived complexity for csv_row_for is too high. [8/7]
  def csv_row_for(asmt, cud, as_seen_by)
  ^^^
app/helpers/assessment_helper.rb:85:5: W: end at 85, 4 is not aligned with if at 73, 18
    end
    ^^^
app/helpers/assessment_helper.rb:93:7: C: Indent when as deep as case.
      when Symbol
      ^^^^
app/helpers/assessment_helper.rb:95:7: C: Indent when as deep as case.
      when Float
      ^^^^
app/helpers/assessment_helper.rb:102:7: C: Use snake_case for method names.
  def bulkGrade_cell(cell)
      ^^^^^^^^^^^^^^
app/helpers/assessment_helper.rb:113:7: C: Use snake_case for method names.
  def bulkGrade_cell_class(cell)
      ^^^^^^^^^^^^^^^^^^^^
app/helpers/assessment_user_datum_helper.rb:1:1: C: Missing top-level module documentation comment.
module AssessmentUserDatumHelper
^^^^^^
app/helpers/assessment_user_datum_helper.rb:13:101: C: Line is too long. [111/100]
  # Due At in string form. If it is nil (meaning there is infinite extension), display appropriate information.
                                                                                                    ^^^^^^^^^^^
app/helpers/assessment_user_datum_helper.rb:18:101: C: Line is too long. [108/100]
  # End At string form. If it is nil (meaning there is infinite extension), display appropriate information.
                                                                                                    ^^^^^^^^
app/helpers/course_helper.rb:1:1: C: Missing top-level module documentation comment.
module CourseHelper
^^^^^^
app/helpers/extension_helper.rb:1:1: C: Missing top-level module documentation comment.
module ExtensionHelper
^^^^^^
app/helpers/gradebook_helper.rb:3:1: C: Missing top-level module documentation comment.
module GradebookHelper
^^^^^^
app/helpers/gradebook_helper.rb:4:3: C: Assignment Branch Condition size for gradebook_columns is too high. [17.15/15]
  def gradebook_columns(matrix, course)
  ^^^
app/helpers/gradebook_helper.rb:4:3: C: Method has too many lines. [35/20]
  def gradebook_columns(matrix, course)
  ^^^
app/helpers/gradebook_helper.rb:53:3: C: Assignment Branch Condition size for gradebook_rows is too high. [57.65/15]
  def gradebook_rows(matrix, course, section = nil, lecture = nil)
  ^^^
app/helpers/gradebook_helper.rb:53:3: C: Cyclomatic complexity for gradebook_rows is too high. [8/6]
  def gradebook_rows(matrix, course, section = nil, lecture = nil)
  ^^^
app/helpers/gradebook_helper.rb:53:3: C: Method has too many lines. [30/20]
  def gradebook_rows(matrix, course, section = nil, lecture = nil)
  ^^^
app/helpers/gradebook_helper.rb:53:3: C: Perceived complexity for gradebook_rows is too high. [8/7]
  def gradebook_rows(matrix, course, section = nil, lecture = nil)
  ^^^
app/helpers/gradebook_helper.rb:122:7: C: Indent when as deep as case.
      when Float
      ^^^^
app/helpers/gradebook_helper.rb:124:7: C: Indent when as deep as case.
      when String
      ^^^^
app/helpers/gradebook_helper.rb:131:3: C: Assignment Branch Condition size for gradebook_csv is too high. [40.12/15]
  def gradebook_csv(matrix, course)
  ^^^
app/helpers/gradebook_helper.rb:139:101: C: Line is too long. [130/100]
        row = [cud.user.email, cud.user.first_name, cud.user.last_name, cud.lecture, cud.section, cud.school, cud.major, cud.year]
                                                                                                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
app/helpers/grading_helper.rb:1:1: C: Missing top-level module documentation comment.
module GradingHelper
^^^^^^
app/helpers/groups_helper.rb:1:1: C: Missing top-level module documentation comment.
module GroupsHelper
^^^^^^
app/helpers/home_helper.rb:1:1: C: Missing top-level module documentation comment.
module HomeHelper
^^^^^^
app/helpers/quick_grades_helper.rb:1:1: C: Missing top-level module documentation comment.
module QuickGradesHelper
^^^^^^
app/helpers/scheduler_helper.rb:1:1: C: Missing top-level module documentation comment.
module SchedulerHelper
^^^^^^
app/helpers/submission_helper.rb:1:1: C: Missing top-level module documentation comment.
module SubmissionHelper
^^^^^^
app/helpers/users_helper.rb:1:1: C: Missing top-level module documentation comment.
module UsersHelper
^^^^^^
app/mailers/course_mailer.rb:1:1: C: Missing top-level class documentation comment.
class CourseMailer < ActionMailer::Base
^^^^^
app/mailers/course_mailer.rb:9:21: C: Do not use Time.[:now] without zone. Use #Time.zone.[:now] instead.
      sent_on: Time.now
                    ^^^
app/mailers/course_mailer.rb:13:26: C: Avoid parameter lists longer than 5 parameters.
  def course_announcement(sender, to, subject, text, cud, course)
                         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
app/mailers/course_mailer.rb:22:21: C: Do not use Time.[:now] without zone. Use #Time.zone.[:now] instead.
      sent_on: Time.now
                    ^^^
app/mailers/course_mailer.rb:35:21: C: Do not use Time.[:now] without zone. Use #Time.zone.[:now] instead.
      sent_on: Time.now
                    ^^^
app/models/assessment.rb:5:1: C: Class definition is too long. [334/100]
class Assessment < ActiveRecord::Base
^^^^^
app/models/assessment.rb:5:1: C: Missing top-level class documentation comment.
class Assessment < ActiveRecord::Base
^^^^^
app/models/assessment.rb:23:3: C: Prefer the new style validations validates :column, uniqueness: value over validates_uniqueness_of.
  validates_uniqueness_of :name, scope: :course_id
  ^^^^^^^^^^^^^^^^^^^^^^^
app/models/assessment.rb:24:3: C: Prefer the new style validations validates :column, length: value over validates_length_of.
  validates_length_of :display_name, minimum: 1
  ^^^^^^^^^^^^^^^^^^^
app/models/assessment.rb:28:3: C: Prefer the new style validations validates :column, numericality: value over validates_numericality_of.
  validates_numericality_of :max_size, :max_submissions
  ^^^^^^^^^^^^^^^^^^^^^^^^^
app/models/assessment.rb:29:3: C: Prefer the new style validations validates :column, numericality: value over validates_numericality_of.
  validates_numericality_of :version_threshold, only_integer: true,
  ^^^^^^^^^^^^^^^^^^^^^^^^^
app/models/assessment.rb:31:3: C: Prefer the new style validations validates :column, numericality: value over validates_numericality_of.
  validates_numericality_of :max_grace_days, only_integer: true,
  ^^^^^^^^^^^^^^^^^^^^^^^^^
app/models/assessment.rb:33:3: C: Prefer the new style validations validates :column, numericality: value over validates_numericality_of.
  validates_numericality_of :group_size, only_integer: true, greater_than_or_equal_to: 1, allow_nil: true
  ^^^^^^^^^^^^^^^^^^^^^^^^^
app/models/assessment.rb:33:101: C: Line is too long. [105/100]
  validates_numericality_of :group_size, only_integer: true, greater_than_or_equal_to: 1, allow_nil: true
                                                                                                    ^^^^^
app/models/assessment.rb:34:3: C: Prefer the new style validations validates :column, presence: value over validates_presence_of.
  validates_presence_of :name, :display_name, :due_at, :end_at, :start_at,
  ^^^^^^^^^^^^^^^^^^^^^
app/models/assessment.rb:50:36: C: Do not use Time.[:now] without zone. Use #Time.zone.[:now] instead.
  scope :released, ->(as_of = Time.now) { where(RELEASED, as_of) }
                                   ^^^
app/models/assessment.rb:51:38: C: Do not use Time.[:now] without zone. Use #Time.zone.[:now] instead.
  scope :unreleased, ->(as_of = Time.now) { where.not(RELEASED, as_of) }
                                     ^^^
app/models/assessment.rb:59:7: C: Use snake_case for method names.
  def create_AUDs_modulo_callbacks
      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
app/models/assessment.rb:86:10: C: Do not use Time.[:now] without zone. Use #Time.zone.[:now] instead.
    Time.now <= grading_deadline
         ^^^
app/models/assessment.rb:105:30: C: Do not use Time.[:now] without zone. Use #Time.zone.[:now] instead.
  def released?(as_of = Time.now)
                             ^^^
app/models/assessment.rb:179:3: C: Assignment Branch Condition size for construct_default_config_file is too high. [16.31/15]
  def construct_default_config_file
  ^^^
app/models/assessment.rb:230:5: W: The use of eval is a serious security risk.
    eval config_module_name
    ^^^^
app/models/assessment.rb:288:7: C: Rename has_autograder? to autograder?.
  def has_autograder?
      ^^^^^^^^^^^^^^^
app/models/assessment.rb:292:7: C: Rename has_groups? to groups?.
  def has_groups?
      ^^^^^^^^^^^
app/models/assessment.rb:296:7: C: Rename has_scoreboard? to scoreboard?.
  def has_scoreboard?
      ^^^^^^^^^^^^^^^
app/models/assessment.rb:304:7: C: Use snake_case for method names.
  def grouplessCUDs
      ^^^^^^^^^^^^^
app/models/assessment.rb:305:101: C: Line is too long. [160/100]
    course.course_user_data.joins(:assessment_user_data).where(assessment_user_data: { assessment_id: id, membership_status: AssessmentUserDatum::UNCONFIRMED })
                                                                                                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
app/models/assessment.rb:326:3: C: Replace class var @@CONFIG_FILE_LAST_LOADED with a class instance var.
  @@CONFIG_FILE_LAST_LOADED = {}
  ^^^^^^^^^^^^^^^^^^^^^^^^^
app/models/assessment.rb:326:3: C: Use snake_case for variable names.
  @@CONFIG_FILE_LAST_LOADED = {}
  ^^^^^^^^^^^^^^^^^^^^^^^^^
app/models/assessment.rb:336:56: C: Do not use Time.[:now] without zone. Use #Time.zone.[:now] instead.
    @@CONFIG_FILE_LAST_LOADED[config_file_path] = Time.now
                                                       ^^^
app/models/assessment.rb:360:101: C: Line is too long. [134/100]
    Submission.select("submissions.*").where("(version, course_user_datum_id) IN (#{max_version_subquery}) AND assessment_id = ?", id)
                                                                                                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
app/models/assessment.rb:374:3: C: Assignment Branch Condition size for serialize is too high. [21.4/15]
  def serialize
  ^^^
app/models/assessment.rb:385:101: C: Line is too long. [211/100]
  GENERAL_SERIALIZABLE = Set.new %w(name display_name category_name description handin_filename handin_directory has_svn max_grace_days handout writeup max_submissions disable_handins max_size version_threshold)
                                                                                                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
app/models/assessment.rb:391:3: C: Assignment Branch Condition size for deserialize is too high. [31.65/15]
  def deserialize(s)
  ^^^
app/models/assessment.rb:392:96: C: Do not use Time.[:now] without zone. Use #Time.zone.[:now] instead.
    self.due_at = self.end_at = self.visible_at = self.start_at = self.grading_deadline = Time.now
                                                                                               ^^^
app/models/assessment.rb:407:5: C: Use a guard clause instead of wrapping the code inside a conditional expression.
    if s["version_penalty"]
    ^^
app/models/assessment.rb:428:7: C: Rename is_file? to file?.
  def is_file?(name)
      ^^^^^^^^
app/models/assessment.rb:477:10: C: Do not use Time.[:now] without zone. Use #Time.zone.[:now] instead.
    Time.now <= course.end_date
         ^^^
app/models/assessment.rb:491:3: C: Assignment Branch Condition size for backwards_compatibility is too high. [17.29/15]
  def backwards_compatibility(props)
  ^^^
app/models/assessment_user_datum.rb:9:1: C: Class definition is too long. [219/100]
class AssessmentUserDatum < ActiveRecord::Base
^^^^^
app/models/assessment_user_datum.rb:80:5: C: Redundant else-clause.
    else
    ^^^^
app/models/assessment_user_datum.rb:199:33: C: Do not use Time.[:now] without zone. Use #Time.zone.[:now] instead.
  def past_due_at?(as_of = Time.now)
                                ^^^
app/models/assessment_user_datum.rb:203:33: C: Do not use Time.[:now] without zone. Use #Time.zone.[:now] instead.
  def past_end_at?(as_of = Time.now)
                                ^^^
app/models/assessment_user_datum.rb:277:29: C: Do not use Time.[:now] without zone. Use #Time.zone.[:now] instead.
                    if Time.now <= assessment.grading_deadline
                            ^^^
app/models/assessment_user_datum.rb:332:5: C: Redundant else-clause.
    else
    ^^^^
app/models/attachment.rb:12:3: C: Assignment Branch Condition size for file= is too high. [15.68/15]
  def file=(upload)
  ^^^
app/models/course.rb:4:1: C: Class definition is too long. [138/100]
class Course < ActiveRecord::Base
^^^^^
app/models/course.rb:4:1: C: Missing top-level class documentation comment.
class Course < ActiveRecord::Base
^^^^^
app/models/course.rb:6:3: C: Prefer the new style validations validates :column, uniqueness: value over validates_uniqueness_of.
  validates_uniqueness_of :name
  ^^^^^^^^^^^^^^^^^^^^^^^
app/models/course.rb:7:3: C: Prefer the new style validations validates :column, presence: value over validates_presence_of.
  validates_presence_of :display_name, :start_date, :end_date
  ^^^^^^^^^^^^^^^^^^^^^
app/models/course.rb:8:3: C: Prefer the new style validations validates :column, presence: value over validates_presence_of.
  validates_presence_of :late_slack, :grace_days, :late_penalty, :version_penalty
  ^^^^^^^^^^^^^^^^^^^^^
app/models/course.rb:9:3: C: Prefer the new style validations validates :column, numericality: value over validates_numericality_of.
  validates_numericality_of :grace_days, greater_than_or_equal_to: 0
  ^^^^^^^^^^^^^^^^^^^^^^^^^
app/models/course.rb:10:3: C: Prefer the new style validations validates :column, numericality: value over validates_numericality_of.
  validates_numericality_of :version_threshold, only_integer: true, greater_than_or_equal_to: -1
  ^^^^^^^^^^^^^^^^^^^^^^^^^
app/models/course.rb:32:3: C: Assignment Branch Condition size for init_course_folder is too high. [19.24/15]
  def init_course_folder
  ^^^
app/models/course.rb:36:20: W: (...) interpreted as grouped expression.
    FileUtils.touch (File.join course_dir, "autolab.log")
                   ^
app/models/course.rb:42:22: W: (...) interpreted as grouped expression.
    FileUtils.mkdir_p (File.join Rails.root, "assessmentConfig")
                     ^
app/models/course.rb:43:22: W: (...) interpreted as grouped expression.
    FileUtils.mkdir_p (File.join Rails.root, "courseConfig")
                     ^
app/models/course.rb:44:22: W: (...) interpreted as grouped expression.
    FileUtils.mkdir_p (File.join Rails.root, "gradebooks")
                     ^
app/models/course.rb:51:38: C: Do not use DateTime.[:now] without zone. Use #Time.zone.[:now] instead.
  def temporal_status(now = DateTime.now)
                                     ^^^
app/models/course.rb:61:42: C: Do not use DateTime.[:now] without zone. Use #Time.zone.[:now] instead.
  def current_assessments(now = DateTime.now)
                                         ^^^
app/models/course.rb:73:3: C: Assignment Branch Condition size for reload_config_file is too high. [27.96/15]
  def reload_config_file
  ^^^
app/models/course.rb:73:3: C: Method has too many lines. [21/20]
  def reload_config_file
  ^^^
app/models/course.rb:85:5: C: Prefer each over for.
    for line in lines do
    ^^^
app/models/course.rb:96:5: W: The use of eval is a serious security risk.
    eval("Course#{course.camelize}")
    ^^^^
app/models/course.rb:106:5: W: Avoid rescuing the Exception class. Perhaps you meant to rescue StandardError?
    rescue Exception => @error
    ^^^^^^^^^^^^^^^^^^^^^^^^^^
app/models/course.rb:130:45: C: Do not use Time.[:now] without zone. Use #Time.zone.[:now] instead.
    assessments.update_all(updated_at: Time.now)
                                            ^^^
app/models/course.rb:168:47: C: Do not use Time.[:now] without zone. Use #Time.zone.[:now] instead.
    self.cgdub_dependencies_updated_at = Time.now
                                              ^^^
app/models/course.rb:175:24: W: The use of eval is a serious security risk.
      Class.new.extend eval(config_module_name)
                       ^^^^
app/models/course_logger.rb:1:1: C: Missing top-level class documentation comment.
class Logger
^^^^^
app/models/course_logger.rb:3:5: C: Use snake_case for variable names.
    strTime = time.strftime("%m/%d/%y %H:%M:%S")
    ^^^^^^^
app/models/course_logger.rb:8:1: C: Missing top-level class documentation comment.
class CourseLogger
^^^^^
app/models/course_logger.rb:13:7: C: Use snake_case for method names.
  def setCourse(course)
      ^^^^^^^^^
app/models/course_user_datum.rb:3:1: C: Class definition is too long. [203/100]
class CourseUserDatum < ActiveRecord::Base
^^^^^
app/models/course_user_datum.rb:3:1: C: Missing top-level class documentation comment.
class CourseUserDatum < ActiveRecord::Base
^^^^^
app/models/course_user_datum.rb:4:3: C: Missing top-level class documentation comment.
  class AuthenticationFailed < Exception
  ^^^^^
app/models/course_user_datum.rb:35:5: W: Useless assignment to variable - conditions.
    conditions = columns.map do|c|
    ^^^^^^^^^^
app/models/course_user_datum.rb:42:5: C: Prefer each over for.
    for column in CourseUserDatum.content_columns
    ^^^
app/models/course_user_datum.rb:42:5: C: Use next to skip iteration.
    for column in CourseUserDatum.content_columns
    ^^^
app/models/course_user_datum.rb:107:7: C: Use snake_case for method names.
  def CA_of?(student)
      ^^^^^^
app/models/course_user_datum.rb:121:7: C: Use snake_case for method names.
  def CA_only?
      ^^^^^^^^
app/models/course_user_datum.rb:140:3: C: Cyclomatic complexity for has_auth_level? is too high. [8/6]
  def has_auth_level?(level)
  ^^^
app/models/course_user_datum.rb:140:7: C: Rename has_auth_level? to auth_level?.
  def has_auth_level?(level)
      ^^^^^^^^^^^^^^^
app/models/course_user_datum.rb:177:5: W: Useless assignment to variable - cud.
    cud = user.course_user_data.find_by(course: course)
    ^^^
app/models/course_user_datum.rb:206:7: C: Use snake_case for method names.
  def create_AUDs_modulo_callbacks
      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
app/models/course_user_datum.rb:237:5: W: end at 237, 4 is not aligned with if at 230, 14
    end
    ^^^
app/models/course_user_datum.rb:246:5: C: Redundant else-clause.
    else
    ^^^^
app/models/course_user_datum.rb:251:3: C: Assignment Branch Condition size for category_average_input is too high. [16.58/15]
  def category_average_input(category, as_seen_by)
  ^^^
app/models/course_user_datum.rb:256:5: W: Useless assignment to variable - user_id.
    user_id = id
    ^^^^^^^
app/models/course_user_datum.rb:266:3: C: Assignment Branch Condition size for average_input is too high. [17.46/15]
  def average_input(as_seen_by)
  ^^^
app/models/course_user_datum.rb:270:5: W: Useless assignment to variable - user_id.
    user_id = id
    ^^^^^^^
app/models/extension.rb:1:1: C: Missing top-level class documentation comment.
class Extension < ActiveRecord::Base
^^^^^
app/models/extension.rb:4:3: C: Prefer the new style validations validates :column, presence: value over validates_presence_of.
  validates_presence_of :course_user_datum_id
  ^^^^^^^^^^^^^^^^^^^^^
app/models/extension.rb:6:3: C: Prefer the new style validations validates :column, uniqueness: value over validates_uniqueness_of.
  validates_uniqueness_of :course_user_datum_id, scope: :assessment_id,
  ^^^^^^^^^^^^^^^^^^^^^^^
app/models/extension.rb:13:5: C: Use a guard clause instead of wrapping the code inside a conditional expression.
    if days.blank? && !infinite?
    ^^
app/models/extension.rb:22:3: C: Assignment Branch Condition size for after_create is too high. [16.03/15]
  def after_create
  ^^^
app/models/grade_matrix.rb:1:1: C: Class definition is too long. [102/100]
class GradeMatrix
^^^^^
app/models/grade_matrix.rb:1:1: C: Missing top-level class documentation comment.
class GradeMatrix
^^^^^
app/models/grade_matrix.rb:12:41: W: Do not use prefix _ for a variable that is used.
      AssociationCache.new(@course) do |_|
                                        ^
app/models/grade_matrix.rb:59:7: C: Rename has_assessment? to assessment?.
  def has_assessment?(asmt_id)
      ^^^^^^^^^^^^^^^
app/models/grade_matrix.rb:64:7: C: Rename has_cud? to cud?.
  def has_cud?(cud_id)
      ^^^^^^^^
app/models/grade_matrix.rb:69:7: C: Rename has_category? to category?.
  def has_category?(cat)
      ^^^^^^^^^^^^^
app/models/grade_matrix.rb:83:3: C: Assignment Branch Condition size for matrix! is too high. [32.08/15]
  def matrix!
  ^^^
app/models/grade_matrix.rb:83:3: C: Method has too many lines. [29/20]
  def matrix!
  ^^^
app/models/grade_matrix.rb:117:30: C: Do not use Time.[:now] without zone. Use #Time.zone.[:now] instead.
      "last_updated" => Time.now
                             ^^^
app/models/grade_matrix.rb:121:3: C: Assignment Branch Condition size for summarize is too high. [21.12/15]
  def summarize(aud)
  ^^^
app/models/group.rb:1:1: C: Missing top-level class documentation comment.
class Group < ActiveRecord::Base
^^^^^
app/models/group.rb:7:7: C: Rename is_member to member?.
  def is_member(aud)
      ^^^^^^^^^
app/models/penalty.rb:1:1: C: Missing top-level class documentation comment.
class Penalty < ScoreAdjustment
^^^^^
app/models/penalty.rb:3:3: C: Prefer the new style validations validates :column, numericality: value over validates_numericality_of.
  validates_numericality_of :value, greater_than_or_equal_to: 0
  ^^^^^^^^^^^^^^^^^^^^^^^^^
app/models/scheduler.rb:1:1: C: Missing top-level class documentation comment.
class Scheduler < ActiveRecord::Base
^^^^^
app/models/scheduler.rb:6:3: C: Prefer the new style validations validates :column, numericality: value over validates_numericality_of.
  validates_numericality_of :interval
  ^^^^^^^^^^^^^^^^^^^^^^^^^
app/models/scheduler.rb:7:3: C: Prefer the new style validations validates :column, presence: value over validates_presence_of.
  validates_presence_of :action
  ^^^^^^^^^^^^^^^^^^^^^
app/models/score.rb:1:1: C: Missing top-level class documentation comment.
class Score < ActiveRecord::Base
^^^^^
app/models/score.rb:9:101: C: Line is too long. [119/100]
  scope :on_latest_submissions, -> { where(submissions: { ignored: false }).joins(submission: :assessment_user_datum) }
                                                                                                    ^^^^^^^^^^^^^^^^^^^
app/models/score.rb:19:3: C: Prefer the new style validations validates :column, uniqueness: value over validates_uniqueness_of.
  validates_uniqueness_of(:problem_id, scope: :submission_id)
  ^^^^^^^^^^^^^^^^^^^^^^^
app/models/score.rb:20:3: C: Prefer the new style validations validates :column, presence: value over validates_presence_of.
  validates_presence_of :grader_id
  ^^^^^^^^^^^^^^^^^^^^^
app/models/score.rb:38:3: C: Assignment Branch Condition size for log_entry is too high. [19.21/15]
  def log_entry
  ^^^
app/models/score.rb:45:5: C: Use a guard clause instead of wrapping the code inside a conditional expression.
    unless submission.nil?
    ^^^^^^
app/models/score_adjustment.rb:1:1: C: Missing top-level class documentation comment.
class ScoreAdjustment < ActiveRecord::Base
^^^^^
app/models/score_adjustment.rb:4:3: C: Prefer the new style validations validates :column, presence: value over validates_presence_of.
  validates_presence_of :value, :kind
  ^^^^^^^^^^^^^^^^^^^^^
app/models/score_adjustment.rb:5:3: C: Prefer the new style validations validates :column, numericality: value over validates_numericality_of.
  validates_numericality_of :value
  ^^^^^^^^^^^^^^^^^^^^^^^^^
app/models/score_adjustment.rb:18:7: C: Provide an exception class and message as arguments to fail.
      fail ArgumentError.new("ScoreAdjustment.applied_value: score was nil")
      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
app/models/score_adjustment.rb:69:5: C: Favor format over sprintf.
    sprintf("%+g", value) + " " + type_str
    ^^^^^^^
app/models/scoreboard.rb:21:3: C: Assignment Branch Condition size for colspec_is_well_formed is too high. [35.86/15]
  def colspec_is_well_formed
  ^^^
app/models/scoreboard.rb:21:3: C: Cyclomatic complexity for colspec_is_well_formed is too high. [12/6]
  def colspec_is_well_formed
  ^^^
app/models/scoreboard.rb:21:3: C: Method has too many lines. [35/20]
  def colspec_is_well_formed
  ^^^
app/models/scoreboard.rb:21:3: C: Perceived complexity for colspec_is_well_formed is too high. [12/7]
  def colspec_is_well_formed
  ^^^
app/models/scoreboard.rb:28:7: W: Useless assignment to variable - quoted.
      quoted = colspec.gsub(/([a-zA-Z0-9]+):/, '"\1":').gsub(/:([a-zA-Z0-9]+)/, ':"\1"')
      ^^^^^^
app/models/scoreboard.rb:50:9: W: Non-local exit from iterator, without return value. next, break, Array#find, Array#any?, etc. is preferred.
        return
        ^^^^^^
app/models/scoreboard.rb:55:9: W: Non-local exit from iterator, without return value. next, break, Array#find, Array#any?, etc. is preferred.
        return
        ^^^^^^
app/models/scoreboard.rb:61:11: W: Non-local exit from iterator, without return value. next, break, Array#find, Array#any?, etc. is preferred.
          return
          ^^^^^^
app/models/scoreboard.rb:65:101: C: Line is too long. [112/100]
          errors.add "colspec", "'asc' key in col #{i} ignored because only the first three columns are sorted."
                                                                                                    ^^^^^^^^^^^^
app/models/submission.rb:7:1: C: Class definition is too long. [337/100]
class Submission < ActiveRecord::Base
^^^^^
app/models/submission.rb:72:3: C: Assignment Branch Condition size for save_file is too high. [46.7/15]
  def save_file(upload)
  ^^^
app/models/submission.rb:72:3: C: Cyclomatic complexity for save_file is too high. [8/6]
  def save_file(upload)
  ^^^
app/models/submission.rb:72:3: C: Method has too many lines. [26/20]
  def save_file(upload)
  ^^^
app/models/submission.rb:72:3: C: Perceived complexity for save_file is too high. [10/7]
  def save_file(upload)
  ^^^
app/models/submission.rb:108:3: C: Assignment Branch Condition size for archive_handin is too high. [16.61/15]
  def archive_handin
  ^^^
app/models/submission.rb:268:43: C: Use %r around regular expression.
    self.detected_mime_type = file_output[/^(\w)+\/([\w-])+/]
                                          ^^^^^^^^^^^^^^^^^^
app/models/submission.rb:280:3: C: Assignment Branch Condition size for as_json is too high. [15.03/15]
  def as_json(options = {})
  ^^^
app/models/submission.rb:355:3: C: Assignment Branch Condition size for raw_score_input is too high. [15.43/15]
  def raw_score_input(include_unreleased)
  ^^^
app/models/submission.rb:424:3: C: Assignment Branch Condition size for allowed? is too high. [20.64/15]
  def allowed?
  ^^^
app/models/submission.rb:424:3: C: Cyclomatic complexity for allowed? is too high. [8/6]
  def allowed?
  ^^^
app/models/submission.rb:425:39: C: Do not use Time.[:now] without zone. Use #Time.zone.[:now] instead.
    submitted_at = created_at || Time.now
                                      ^^^
app/models/submission.rb:473:3: C: Assignment Branch Condition size for days_late! is too high. [23.87/15]
  def days_late!
  ^^^
app/models/tweak.rb:1:1: C: Missing top-level class documentation comment.
class Tweak < ScoreAdjustment
^^^^^
app/models/tweak.rb:12:7: C: Provide an exception class and message as arguments to fail.
      fail ArgumentError.new("ScoreAdjustment.apply_tweak: score was nil")
      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
app/models/user.rb:5:1: C: Class definition is too long. [198/100]
class User < ActiveRecord::Base
^^^^^
app/models/user.rb:63:3: C: Assignment Branch Condition size for ldap_reset is too high. [23.37/15]
  def ldap_reset
  ^^^
app/models/user.rb:100:3: C: Assignment Branch Condition size for new_with_session is too high. [35.26/15]
  def self.new_with_session(params, session)
  ^^^
app/models/user.rb:123:3: C: Assignment Branch Condition size for roster_create is too high. [19.26/15]
  def self.roster_create(email, first_name, last_name, school, major, year)
  ^^^
app/models/user.rb:123:3: C: Method has too many lines. [21/20]
  def self.roster_create(email, first_name, last_name, school, major, year)
  ^^^
app/models/user.rb:123:25: C: Avoid parameter lists longer than 5 parameters.
  def self.roster_create(email, first_name, last_name, school, major, year)
                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
app/models/user.rb:179:3: C: Assignment Branch Condition size for ldap_lookup is too high. [52.11/15]
  def self.ldap_lookup(andrewID)
  ^^^
app/models/user.rb:179:3: C: Cyclomatic complexity for ldap_lookup is too high. [44/6]
  def self.ldap_lookup(andrewID)
  ^^^
app/models/user.rb:179:3: C: Method has too many lines. [65/20]
  def self.ldap_lookup(andrewID)
  ^^^
app/models/user.rb:179:3: C: Perceived complexity for ldap_lookup is too high. [15/7]
  def self.ldap_lookup(andrewID)
  ^^^
lib/__defaultAssessment.rb:4:3: E: class or module name must be a constant literal
  include AssessmentBase
  ^^^^^^^
lib/__defaultAssessment.rb:11:1: E: unexpected token kEND
end
^^^
lib/__defaultCourse.rb:1:1: C: Use snake_case for source file names.
#
^
lib/__defaultCourse.rb:36:5: C: Use snake_case for method names.
def fooAverage(_user)
    ^^^^^^^^^^
lib/__defaultCourse.rb:52:5: C: Use snake_case for method names.
def courseAverage(_user)
    ^^^^^^^^^^^^^
lib/__defaultCourse.rb:59:5: C: Use snake_case for method names.
def gradebookMessage
    ^^^^^^^^^^^^^^^^
lib/AssessmentBase.rb:1:1: C: Missing top-level module documentation comment.
module AssessmentBase
^^^^^^
lib/AssessmentBase.rb:1:1: C: Use snake_case for source file names.
module AssessmentBase
^
lib/association_cache.rb:1:1: C: Missing top-level class documentation comment.
class AssociationCache
^^^^^
lib/association_cache.rb:53:101: C: Line is too long. [111/100]
      @latest_submission_scores[score.submission_id] = [] unless @latest_submission_scores[score.submission_id]
                                                                                                    ^^^^^^^^^^^
lib/association_cache.rb:93:1: C: Missing top-level module documentation comment.
module CUDAssociationCache
^^^^^^
lib/association_cache.rb:109:1: C: Missing top-level module documentation comment.
module CourseAssociationCache
^^^^^^
lib/association_cache.rb:131:1: C: Missing top-level module documentation comment.
module AssessmentAssociationCache
^^^^^^
lib/association_cache.rb:159:1: C: Missing top-level module documentation comment.
module AUDAssociationCache
^^^^^^
lib/association_cache.rb:183:101: C: Line is too long. [121/100]
    @latest_submission = cache.latest_submissions[au_key assessment_id, course_user_datum_id] if cache.latest_submissions
                                                                                                    ^^^^^^^^^^^^^^^^^^^^^
lib/association_cache.rb:187:1: C: Missing top-level module documentation comment.
module LatestSubmissionAssociationCache
^^^^^^
lib/association_cache.rb:211:3: C: Assignment Branch Condition size for association_cache= is too high. [19.1/15]
  def association_cache=(cache)
  ^^^
lib/CourseBase.rb:1:1: C: Use snake_case for source file names.
require "statistics.rb"
^
lib/CourseBase.rb:3:1: C: Missing top-level module documentation comment.
module CourseBase
^^^^^^
lib/CourseBase.rb:4:7: C: Use snake_case for method names.
  def courseAverage(user)
      ^^^^^^^^^^^^^
lib/CourseBase.rb:8:7: C: Use snake_case for method names.
  def gradebookMessage
      ^^^^^^^^^^^^^^^^
lib/module_base.rb:6:7: C: Use snake_case for method names.
  def updateModules
      ^^^^^^^^^^^^^
lib/module_base.rb:7:5: C: Use snake_case for variable names.
    @allModules = %w(Autograde Scoreboard Partners Svn)
    ^^^^^^^^^^^
lib/module_base.rb:8:5: C: Use snake_case for variable names.
    @modulesUsed = []
    ^^^^^^^^^^^^
lib/module_base.rb:10:5: C: Use snake_case for variable names.
    modName = (assign + (@course.name).gsub(/[^A-Za-z0-9]/, "")).camelize
    ^^^^^^^
lib/module_base.rb:12:5: C: Prefer each over for.
    for mod in @allModules do
    ^^^
lib/module_base.rb:14:9: C: Use snake_case for variable names.
        modUsed = eval("#{modName}.include?(#{mod})")
        ^^^^^^^
lib/module_base.rb:14:19: W: The use of eval is a serious security risk.
        modUsed = eval("#{modName}.include?(#{mod})")
                  ^^^^
lib/module_base.rb:16:7: W: Do not suppress exceptions.
      rescue Exception
      ^^^^^^^^^^^^^^^^
lib/module_base.rb:16:7: W: Avoid rescuing the Exception class. Perhaps you meant to rescue StandardError?
      rescue Exception
      ^^^^^^^^^^^^^^^^
lib/module_base.rb:23:7: C: Use snake_case for method names.
  def listAdmin
      ^^^^^^^^^
lib/module_base.rb:38:7: C: Use snake_case for method names.
  def loadHandinPage
      ^^^^^^^^^^^^^^
lib/module_base.rb:45:7: C: Use snake_case for method names.
  def validateHandin
      ^^^^^^^^^^^^^^
lib/module_base.rb:63:7: C: Use snake_case for method names.
  def saveHandin
      ^^^^^^^^^^
lib/modules/Autograde.rb:1:1: C: Use snake_case for source file names.
##
^
lib/modules/Partners.rb:1:1: C: Use snake_case for source file names.
##
^
lib/modules/Scoreboard.rb:1:1: C: Use snake_case for source file names.
##
^
lib/modules/Svn.rb:1:1: C: Use snake_case for source file names.
##
^
lib/modules/Svn.rb:7:7: C: Use snake_case for method names.
  def checkoutWork(_repo, targetDir)
      ^^^^^^^^^^^^
lib/modules/Svn.rb:15:7: C: Use snake_case for variable names.
      testFile = File.join(targetDir, "test.txt")
      ^^^^^^^^
lib/modules/Svn.rb:29:7: C: Use snake_case for method names.
  def subversionType
      ^^^^^^^^^^^^^^
lib/statistics.rb:1:1: C: Missing top-level class documentation comment.
class Statistics
^^^^^
lib/statistics.rb:6:31: C: Name inject block params |a, e|.
      result = (a.inject(0) { |sum, el| sum + el }.to_f) / a.to_a.length
                              ^^^^^^^^^
lib/statistics.rb:52:3: C: Assignment Branch Condition size for median is too high. [16.76/15]
  def median(a)
  ^^^
lib/utilities.rb:1:1: C: Missing top-level module documentation comment.
module Utilities
^^^^^^
lib/utilities.rb:8:12: C: Rename is_url? to url?.
  def self.is_url?(name)
           ^^^^^^^
lib/utilities.rb:19:3: W: Avoid rescuing the Exception class. Perhaps you meant to rescue StandardError?
  rescue Exception => e
  ^^^^^^^^^^^^^^^^^^^^^
lib/utilities.rb:20:5: C: Provide an exception class and message as arguments to raise.
    raise InstructorException.new("Error executing #{invoked_method_name}: #{e}")
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
lib/utilities.rb:23:3: C: Perceived complexity for validated_score_value is too high. [8/7]
  def self.validated_score_value(score, invoked_method_name, allow_nil = false)
  ^^^
lib/utilities.rb:27:19: C: Avoid using rescue in its modifier form.
      if (score = Float(score) rescue nil)
                  ^^^^^^^^^^^^^^^^^^^^^^^
lib/utilities.rb:28:9: C: Provide an exception class and message as arguments to fail.
        fail InvalidComputedScoreException.new("#{message}: returned infinite number") unless score.finite?
        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
lib/utilities.rb:28:101: C: Line is too long. [107/100]
        fail InvalidComputedScoreException.new("#{message}: returned infinite number") unless score.finite?
                                                                                                    ^^^^^^^
lib/utilities.rb:30:9: C: Provide an exception class and message as arguments to fail.
        fail InvalidComputedScoreException.new("#{message}: error converting to float")
        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
lib/utilities.rb:33:7: C: Provide an exception class and message as arguments to fail.
      fail InvalidComputedScoreException.new("#{message}: returned nil") unless allow_nil
      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
spec/controllers/schedulers_controller_spec.rb:142:7: W: Useless assignment to variable - cname.
      cname = Course.find(cid).name
      ^^^^^
spec/controllers/schedulers_controller_spec.rb:198:7: W: Useless assignment to variable - cname.
      cname = Course.find(cid).name
      ^^^^^
spec/factories/users.rb:4:13: W: (...) interpreted as grouped expression.
    sequence (:last_name) { |n| "User #{n}" }
            ^
spec/factories/users.rb:5:13: W: (...) interpreted as grouped expression.
    sequence (:email) { |n| "test#{n}@andrew.cmu.edu" }
            ^
spec/factories/users.rb:8:23: C: Do not use Time.[:now] without zone. Use #Time.zone.[:now] instead.
    confirmed_at Time.now
                      ^^^
spec/support/controller_macros.rb:3:1: C: Missing top-level module documentation comment.
module ControllerMacros
^^^^^^
spec/support/controller_macros.rb:4:7: C: Do not prefix reader method names with get_.
  def get_admin
      ^^^^^^^^^
spec/support/controller_macros.rb:9:7: C: Do not prefix reader method names with get_.
  def get_instructor
      ^^^^^^^^^^^^^^
spec/support/controller_macros.rb:10:5: C: Use snake_case for variable names.
    instructorCUDs = CourseUserDatum.where(instructor: true)
    ^^^^^^^^^^^^^^
spec/support/controller_macros.rb:14:7: C: Do not prefix reader method names with get_.
  def get_course_assistant
      ^^^^^^^^^^^^^^^^^^^^
spec/support/controller_macros.rb:15:5: C: Use snake_case for variable names.
    caCUDs = CourseUserDatum.where(course_assistant: true)
    ^^^^^^
spec/support/controller_macros.rb:19:7: C: Do not prefix reader method names with get_.
  def get_user
      ^^^^^^^^
spec/support/controller_macros.rb:60:69: C: Do not use Time.[:now] without zone. Use #Time.zone.[:now] instead.
                                       interval: 86_400, next: Time.now)
                                                                    ^^^
templates/AutoPopulated-labtemplate.rb:1:1: C: Use snake_case for source file names.
require "AssessmentBase.rb"
^
templates/AutoPopulated-labtemplate.rb:4:1: C: Missing top-level module documentation comment.
module LabtemplateAutoPopulated
^^^^^^
templates/AutoPopulated-labtemplate.rb:8:7: C: Use snake_case for method names.
  def assessmentInitialize(course)
      ^^^^^^^^^^^^^^^^^^^^
templates/labtemplate/labtemplate.rb:4:1: C: Missing top-level module documentation comment.
module Labtemplate
^^^^^^
templates/labtemplate/labtemplate.rb:8:7: C: Use snake_case for method names.
  def assessmentInitialize(course)
      ^^^^^^^^^^^^^^^^^^^^

151 files inspected, 932 offenses detected