Style updates

Author: salbertson

app/models/user.rb

@@ -39,11 +39,11 @@ def has_active_repos?
 
   def token=(value)
     encrypted_token = crypt.encrypt_and_sign(value)
-    write_attribute(:token, encrypted_token)
+    self[:token] = encrypted_token
   end
 
   def token
-    encrypted_token = read_attribute(:token)
+    encrypted_token = self[:token]
     unless encrypted_token.nil?
       crypt.decrypt_and_verify(encrypted_token)
     end

app/models/violation.rb

@@ -17,11 +17,11 @@ def messages
 
   def source=(value)
     encrypted_source = crypt.encrypt_and_sign(value)
-    write_attribute(:source, encrypted_source)
+    self[:source] = encrypted_source
   end
 
   def source
-    encrypted_source = read_attribute(:source)
+    encrypted_source = self[:source]
     unless encrypted_source.nil?
       crypt.decrypt_and_verify(encrypted_source)
     end

app/services/complete_file_review.rb

@@ -31,7 +31,7 @@ def build_file_review_violations
       file_review.build_violation(
         line,
         violation.fetch(:message),
-        violation.fetch(:source)
+        violation.fetch(:source),
       )
     end
   end

spec/models/file_review_spec.rb

@@ -41,8 +41,16 @@
         line = build_line
         file_review = FileReview.new
 
-        file_review.build_violation(line, first_violation_message, "foo = bar(1, 2)")
-        file_review.build_violation(line, other_violation_message, "foo = bar(1, 2)")
+        file_review.build_violation(
+          line,
+          first_violation_message,
+          "foo = bar(1, 2)"
+        )
+        file_review.build_violation(
+          line,
+          other_violation_message,
+          "foo = bar(1, 2)"
+        )
         violation = file_review.violations.first
 
         expect(file_review.violations.size).to eq 1

spec/services/suggest_changes_spec.rb

@@ -9,9 +9,9 @@
         violation = instance_double(
           "Violation",
           messages: [
-            "Put a comma after the last parameter of a multiline method call."
+            "Put a comma after the last parameter of a multiline method call.",
           ],
-          source: "    violation.fetch(:source)"
+          source: "    violation.fetch(:source)",
         )
         suggest_changes = described_class.new(violation)
 
@@ -31,7 +31,7 @@
             "Missing semicolon semi",
             "Something else",
           ],
-          source: "  console.log('wat')"
+          source: "  console.log('wat')",
         )
         suggest_changes = described_class.new(violation)
 
@@ -72,7 +72,7 @@
             "Layout/TrailingWhitespace: Trailing whitespace detected.",
             "Put a comma after the last parameter of a multiline method call.",
           ],
-          source: "    violation.fetch(:source) "
+          source: "    violation.fetch(:source) ",
         )
         suggest_changes = described_class.new(violation)