Fix linting issues and finalize test improvements

Co-authored-by: GrantBirki <[email protected]>

Author: Copilot

spec/unit/lib/hooks/app/endpoints/health_spec.rb

@@ -24,38 +24,38 @@ def app
 
     it "includes health status in response" do
       get "/"
-      
+
       response_data = JSON.parse(last_response.body)
       expect(response_data["status"]).to eq("healthy")
     end
 
     it "includes timestamp in ISO8601 format" do
       get "/"
-      
+
       response_data = JSON.parse(last_response.body)
       expect(response_data["timestamp"]).to eq(TIME_MOCK)
     end
 
     it "includes version information" do
       get "/"
-      
+
       response_data = JSON.parse(last_response.body)
       expect(response_data["version"]).to eq(Hooks::VERSION)
     end
 
     it "includes uptime in seconds" do
       get "/"
-      
+
       response_data = JSON.parse(last_response.body)
       expect(response_data["uptime_seconds"]).to be_a(Integer)
       expect(response_data["uptime_seconds"]).to eq(60) # 1 minute difference
     end
 
     it "returns valid JSON structure" do
       get "/"
-      
+
       expect { JSON.parse(last_response.body) }.not_to raise_error
-      
+
       response_data = JSON.parse(last_response.body)
       expect(response_data).to have_key("status")
       expect(response_data).to have_key("timestamp")
@@ -67,11 +67,11 @@ def app
       # Test with different start time
       different_start = Time.parse("2024-12-31T23:58:30Z")
       allow(Hooks::App::API).to receive(:start_time).and_return(different_start)
-      
+
       get "/"
-      
+
       response_data = JSON.parse(last_response.body)
       expect(response_data["uptime_seconds"]).to eq(90) # 1.5 minutes difference
     end
   end
-end
+end

spec/unit/lib/hooks/app/endpoints/version_spec.rb

@@ -19,33 +19,33 @@ def app
 
     it "includes version number in response" do
       get "/"
-      
+
       response_data = JSON.parse(last_response.body)
       expect(response_data["version"]).to eq(Hooks::VERSION)
     end
 
     it "includes timestamp in ISO8601 format" do
       get "/"
-      
+
       response_data = JSON.parse(last_response.body)
       expect(response_data["timestamp"]).to eq(TIME_MOCK)
     end
 
     it "returns valid JSON structure" do
       get "/"
-      
+
       expect { JSON.parse(last_response.body) }.not_to raise_error
-      
+
       response_data = JSON.parse(last_response.body)
       expect(response_data).to have_key("version")
       expect(response_data).to have_key("timestamp")
     end
 
     it "version matches expected format" do
       get "/"
-      
+
       response_data = JSON.parse(last_response.body)
       expect(response_data["version"]).to match(/^\d+\.\d+\.\d+$/)
     end
   end
-end
+end