🤖

app/controllers/adam/articles_controller.rb

@@ -30,7 +30,7 @@ def publish
   end
 
   private
-    
+
     def set_article
       @article = Article.find(params[:id])
     end

app/controllers/adam/notes_controller.rb

@@ -31,7 +31,7 @@ def publish
   end
 
   private
-    
+
     def set_note
       @note = Note.find(params[:id])
     end

app/controllers/sessions_controller.rb

@@ -3,13 +3,19 @@ def new
   end
 
   def create
-    if params[:password] === Rails.application.credentials.adam[:password]
-      session[:logged_in] = Rails.application.credentials.adam[:username]
-      redirect_path = session[:pre_auth_path] || root_url
-      redirect_to redirect_path
+    login
+    if logged_in?
+      redirect_to session[:pre_auth_path] || root_url
     else
       render :new, notice: "Wrong!"
     end
-
   end
+
+  private
+
+    def login
+      if params[:password] === Rails.application.credentials.adam[:password]
+        session[:logged_in] = Rails.application.credentials.adam[:username]
+      end
+    end
 end

app/helpers/sessions_helper.rb

@@ -1,5 +1,5 @@
 module SessionsHelper
   def logged_in?
-      session[:logged_in] == Rails.application.credentials.adam[:username]
+    session[:logged_in] == Rails.application.credentials.adam[:username]
   end
 end

app/models/post.rb

@@ -11,7 +11,7 @@ def validate(record)
 class Post < ApplicationRecord
   validates_presence_of :content
   validates_with SlugValidator
-  
+
   has_many :syndicates
 
   before_validation :generate_slug

app/services/twitter_service.rb

@@ -6,7 +6,7 @@
 end
 
 class TwitterService
-  def self.post(post)  
+  def self.post(post)
     tweet = post.content
     tweet.gsub!('*', '＊')
     response = TwitterClient.update(tweet)

spec/factories/posts.rb

@@ -2,8 +2,8 @@
   factory :post do
     content { "Text" }
     type { Note }
-     trait :published do
-       published_at { Time.now }
+    trait :published do
+      published_at { Time.now }
     end
   end
 end

spec/models/post_spec.rb

@@ -5,7 +5,7 @@
     it { is_expected.to validate_presence_of :content }
     # We're getting round this by actually looping through the stack of candidates
     # describe "slug validation" do
-    #   before do 
+    #   before do
     #     create(:post, published_at: 1.day.ago, slug: "the-same-slug")
     #   end
     #   it "requires slugs to be unique for posts on the same day" do
@@ -56,12 +56,12 @@
         expect(post.slug).to eq "slug-2"
       end
 
-       it "cleans the slug" do
-         post = create(:post, slug: "slug")
-         post.update(slug: "slug 2")
+      it "cleans the slug" do
+        post = create(:post, slug: "slug")
+        post.update(slug: "slug 2")
 
-         expect(post.slug).to eq "slug-2"
-       end
+        expect(post.slug).to eq "slug-2"
+      end
     end
 
     context "when the slug hasn't changed, but other attributes have" do