From 8544802b7fdaa461ae7f7acbe4eed5f4d9924f4f Mon Sep 17 00:00:00 2001 From: kbe Date: Fri, 5 Sep 2025 14:38:14 +0200 Subject: [PATCH] style: Lint code --- test/jobs/cleanup_expired_drafts_job_test.rb | 20 ++++++++++---------- test/jobs/expired_orders_cleanup_job_test.rb | 14 +++++++------- test/services/ticket_pdf_generator_test.rb | 4 ++-- 3 files changed, 19 insertions(+), 19 deletions(-) diff --git a/test/jobs/cleanup_expired_drafts_job_test.rb b/test/jobs/cleanup_expired_drafts_job_test.rb index 027e0b7..098b910 100644 --- a/test/jobs/cleanup_expired_drafts_job_test.rb +++ b/test/jobs/cleanup_expired_drafts_job_test.rb @@ -69,7 +69,7 @@ class CleanupExpiredDraftsJobTest < ActiveJob::TestCase assert_nothing_raised do CleanupExpiredDraftsJob.perform_now end - + # Basic functional verification assert_not_nil Ticket.find(expired_ticket.id) end @@ -77,7 +77,7 @@ class CleanupExpiredDraftsJobTest < ActiveJob::TestCase test "should handle multiple expired tickets" do # Create multiple orders with multiple expired tickets @order.update!(expires_at: 1.hour.ago) - + ticket1 = Ticket.create!( order: @order, ticket_type: @ticket_type, @@ -85,20 +85,20 @@ class CleanupExpiredDraftsJobTest < ActiveJob::TestCase first_name: "John", last_name: "Doe" ) - + ticket2 = Ticket.create!( order: @order, ticket_type: @ticket_type, - status: "draft", + status: "draft", first_name: "Jane", last_name: "Doe" ) - + # Job should run without errors assert_nothing_raised do CleanupExpiredDraftsJob.perform_now end - + # Verify both tickets still exist (functional test) assert_not_nil Ticket.find(ticket1.id) assert_not_nil Ticket.find(ticket2.id) @@ -114,12 +114,12 @@ class CleanupExpiredDraftsJobTest < ActiveJob::TestCase first_name: "John", last_name: "Doe" ) - + # Job should run without errors assert_nothing_raised do CleanupExpiredDraftsJob.perform_now end - + # Ticket should remain unchanged assert_equal "draft", ticket.reload.status end @@ -127,10 +127,10 @@ class CleanupExpiredDraftsJobTest < ActiveJob::TestCase test "should handle empty expired tickets list" do # Ensure no tickets are expired @order.update!(expires_at: 1.hour.from_now) - + # Job should run without errors assert_nothing_raised do CleanupExpiredDraftsJob.perform_now end end -end \ No newline at end of file +end diff --git a/test/jobs/expired_orders_cleanup_job_test.rb b/test/jobs/expired_orders_cleanup_job_test.rb index 104f2d1..840589a 100644 --- a/test/jobs/expired_orders_cleanup_job_test.rb +++ b/test/jobs/expired_orders_cleanup_job_test.rb @@ -50,7 +50,7 @@ class ExpiredOrdersCleanupJobTest < ActiveJob::TestCase assert_nothing_raised do ExpiredOrdersCleanupJob.perform_now end - + # Order should still exist (functional test) assert_not_nil Order.find(expired_order.id) end @@ -64,7 +64,7 @@ class ExpiredOrdersCleanupJobTest < ActiveJob::TestCase total_amount_cents: 2500, expires_at: 2.hours.ago ) - + order2 = Order.create!( user: @user, event: @event, @@ -77,7 +77,7 @@ class ExpiredOrdersCleanupJobTest < ActiveJob::TestCase assert_nothing_raised do ExpiredOrdersCleanupJob.perform_now end - + # Both orders should still exist (functional test) assert_not_nil Order.find(order1.id) assert_not_nil Order.find(order2.id) @@ -97,7 +97,7 @@ class ExpiredOrdersCleanupJobTest < ActiveJob::TestCase assert_nothing_raised do ExpiredOrdersCleanupJob.perform_now end - + # Order should remain unchanged assert_equal "draft", active_order.reload.status end @@ -116,7 +116,7 @@ class ExpiredOrdersCleanupJobTest < ActiveJob::TestCase assert_nothing_raised do ExpiredOrdersCleanupJob.perform_now end - + # Order should remain paid assert_equal "paid", paid_order.reload.status end @@ -126,7 +126,7 @@ class ExpiredOrdersCleanupJobTest < ActiveJob::TestCase Order.create!( user: @user, event: @event, - status: "draft", + status: "draft", total_amount_cents: 2500, expires_at: 1.hour.from_now ) @@ -148,4 +148,4 @@ class ExpiredOrdersCleanupJobTest < ActiveJob::TestCase ExpiredOrdersCleanupJob.perform_now end end -end \ No newline at end of file +end diff --git a/test/services/ticket_pdf_generator_test.rb b/test/services/ticket_pdf_generator_test.rb index 50ca0e2..808a0a8 100644 --- a/test/services/ticket_pdf_generator_test.rb +++ b/test/services/ticket_pdf_generator_test.rb @@ -2,11 +2,11 @@ require "test_helper" class TicketPdfGeneratorTest < ActiveSupport::TestCase def setup - # Stub QR code generation to avoid dependency issues + # Stub QR code generation to avoid dependency issues mock_qrcode = mock("qrcode") mock_qrcode.stubs(:modules).returns([]) RQRCode::QRCode.stubs(:new).returns(mock_qrcode) - + @user = User.create!( email: "test@example.com", password: "password123",