Skip to content

Commit 77fb1fc

Browse files
committed
chore: fix minor cops
1 parent 03620db commit 77fb1fc

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

app/models/unit.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2254,7 +2254,7 @@ def update_task_status_from_csv(user, csv_str, success, _ignored, errors)
22542254

22552255
# read data from CSV
22562256
CSV.parse(csv_str, headers: true, return_headers: true,
2257-
header_converters: [->(body) { body&.encode('UTF-8', 'binary', invalid: :replace, undef: :replace, replace: '').downcase }],
2257+
header_converters: [->(body) { body&.encode('UTF-8', 'binary', invalid: :replace, undef: :replace, replace: '')&.downcase }],
22582258
converters: [->(body) { body&.encode('UTF-8', 'binary', invalid: :replace, undef: :replace, replace: '') }]).each do |task_entry|
22592259
group = nil
22602260

test/models/unit_model_test.rb

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -405,7 +405,7 @@ def check_task_completion_csv unit, col_count = nil
405405
CSV.parse(csv_str,
406406
headers: true,
407407
return_headers: false,
408-
header_converters: [->(body) { body&.encode('UTF-8', 'binary', invalid: :replace, undef: :replace, replace: '').downcase }],
408+
header_converters: [->(body) { body&.encode('UTF-8', 'binary', invalid: :replace, undef: :replace, replace: '')&.downcase }],
409409
converters: [->(body) { body&.encode('UTF-8', 'binary', invalid: :replace, undef: :replace, replace: '') }]).each do |entry|
410410

411411
assert_equal(col_count, entry.length, entry.inspect) unless col_count.nil?
@@ -493,7 +493,7 @@ def test_export_users
493493
rows = 0
494494
CSV.parse(csv_str,
495495
headers: true, return_headers: false,
496-
header_converters: [->(body) { body&.encode('UTF-8', 'binary', invalid: :replace, undef: :replace, replace: '').downcase }],
496+
header_converters: [->(body) { body&.encode('UTF-8', 'binary', invalid: :replace, undef: :replace, replace: '')&.downcase }],
497497
converters: [->(body) { body&.encode('UTF-8', 'binary', invalid: :replace, undef: :replace, replace: '') }]).each do |entry|
498498
assert_json_limit_keys_to_exactly %w[unit_code campus username student_id preferred_name first_name last_name email tutorial], entry.to_hash
499499
assert_equal 9, entry.count, entry

0 commit comments

Comments
 (0)