style: Lint code
This commit is contained in:
@@ -69,7 +69,7 @@ class CleanupExpiredDraftsJobTest < ActiveJob::TestCase
|
|||||||
assert_nothing_raised do
|
assert_nothing_raised do
|
||||||
CleanupExpiredDraftsJob.perform_now
|
CleanupExpiredDraftsJob.perform_now
|
||||||
end
|
end
|
||||||
|
|
||||||
# Basic functional verification
|
# Basic functional verification
|
||||||
assert_not_nil Ticket.find(expired_ticket.id)
|
assert_not_nil Ticket.find(expired_ticket.id)
|
||||||
end
|
end
|
||||||
@@ -77,7 +77,7 @@ class CleanupExpiredDraftsJobTest < ActiveJob::TestCase
|
|||||||
test "should handle multiple expired tickets" do
|
test "should handle multiple expired tickets" do
|
||||||
# Create multiple orders with multiple expired tickets
|
# Create multiple orders with multiple expired tickets
|
||||||
@order.update!(expires_at: 1.hour.ago)
|
@order.update!(expires_at: 1.hour.ago)
|
||||||
|
|
||||||
ticket1 = Ticket.create!(
|
ticket1 = Ticket.create!(
|
||||||
order: @order,
|
order: @order,
|
||||||
ticket_type: @ticket_type,
|
ticket_type: @ticket_type,
|
||||||
@@ -85,20 +85,20 @@ class CleanupExpiredDraftsJobTest < ActiveJob::TestCase
|
|||||||
first_name: "John",
|
first_name: "John",
|
||||||
last_name: "Doe"
|
last_name: "Doe"
|
||||||
)
|
)
|
||||||
|
|
||||||
ticket2 = Ticket.create!(
|
ticket2 = Ticket.create!(
|
||||||
order: @order,
|
order: @order,
|
||||||
ticket_type: @ticket_type,
|
ticket_type: @ticket_type,
|
||||||
status: "draft",
|
status: "draft",
|
||||||
first_name: "Jane",
|
first_name: "Jane",
|
||||||
last_name: "Doe"
|
last_name: "Doe"
|
||||||
)
|
)
|
||||||
|
|
||||||
# Job should run without errors
|
# Job should run without errors
|
||||||
assert_nothing_raised do
|
assert_nothing_raised do
|
||||||
CleanupExpiredDraftsJob.perform_now
|
CleanupExpiredDraftsJob.perform_now
|
||||||
end
|
end
|
||||||
|
|
||||||
# Verify both tickets still exist (functional test)
|
# Verify both tickets still exist (functional test)
|
||||||
assert_not_nil Ticket.find(ticket1.id)
|
assert_not_nil Ticket.find(ticket1.id)
|
||||||
assert_not_nil Ticket.find(ticket2.id)
|
assert_not_nil Ticket.find(ticket2.id)
|
||||||
@@ -114,12 +114,12 @@ class CleanupExpiredDraftsJobTest < ActiveJob::TestCase
|
|||||||
first_name: "John",
|
first_name: "John",
|
||||||
last_name: "Doe"
|
last_name: "Doe"
|
||||||
)
|
)
|
||||||
|
|
||||||
# Job should run without errors
|
# Job should run without errors
|
||||||
assert_nothing_raised do
|
assert_nothing_raised do
|
||||||
CleanupExpiredDraftsJob.perform_now
|
CleanupExpiredDraftsJob.perform_now
|
||||||
end
|
end
|
||||||
|
|
||||||
# Ticket should remain unchanged
|
# Ticket should remain unchanged
|
||||||
assert_equal "draft", ticket.reload.status
|
assert_equal "draft", ticket.reload.status
|
||||||
end
|
end
|
||||||
@@ -127,10 +127,10 @@ class CleanupExpiredDraftsJobTest < ActiveJob::TestCase
|
|||||||
test "should handle empty expired tickets list" do
|
test "should handle empty expired tickets list" do
|
||||||
# Ensure no tickets are expired
|
# Ensure no tickets are expired
|
||||||
@order.update!(expires_at: 1.hour.from_now)
|
@order.update!(expires_at: 1.hour.from_now)
|
||||||
|
|
||||||
# Job should run without errors
|
# Job should run without errors
|
||||||
assert_nothing_raised do
|
assert_nothing_raised do
|
||||||
CleanupExpiredDraftsJob.perform_now
|
CleanupExpiredDraftsJob.perform_now
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|||||||
@@ -50,7 +50,7 @@ class ExpiredOrdersCleanupJobTest < ActiveJob::TestCase
|
|||||||
assert_nothing_raised do
|
assert_nothing_raised do
|
||||||
ExpiredOrdersCleanupJob.perform_now
|
ExpiredOrdersCleanupJob.perform_now
|
||||||
end
|
end
|
||||||
|
|
||||||
# Order should still exist (functional test)
|
# Order should still exist (functional test)
|
||||||
assert_not_nil Order.find(expired_order.id)
|
assert_not_nil Order.find(expired_order.id)
|
||||||
end
|
end
|
||||||
@@ -64,7 +64,7 @@ class ExpiredOrdersCleanupJobTest < ActiveJob::TestCase
|
|||||||
total_amount_cents: 2500,
|
total_amount_cents: 2500,
|
||||||
expires_at: 2.hours.ago
|
expires_at: 2.hours.ago
|
||||||
)
|
)
|
||||||
|
|
||||||
order2 = Order.create!(
|
order2 = Order.create!(
|
||||||
user: @user,
|
user: @user,
|
||||||
event: @event,
|
event: @event,
|
||||||
@@ -77,7 +77,7 @@ class ExpiredOrdersCleanupJobTest < ActiveJob::TestCase
|
|||||||
assert_nothing_raised do
|
assert_nothing_raised do
|
||||||
ExpiredOrdersCleanupJob.perform_now
|
ExpiredOrdersCleanupJob.perform_now
|
||||||
end
|
end
|
||||||
|
|
||||||
# Both orders should still exist (functional test)
|
# Both orders should still exist (functional test)
|
||||||
assert_not_nil Order.find(order1.id)
|
assert_not_nil Order.find(order1.id)
|
||||||
assert_not_nil Order.find(order2.id)
|
assert_not_nil Order.find(order2.id)
|
||||||
@@ -97,7 +97,7 @@ class ExpiredOrdersCleanupJobTest < ActiveJob::TestCase
|
|||||||
assert_nothing_raised do
|
assert_nothing_raised do
|
||||||
ExpiredOrdersCleanupJob.perform_now
|
ExpiredOrdersCleanupJob.perform_now
|
||||||
end
|
end
|
||||||
|
|
||||||
# Order should remain unchanged
|
# Order should remain unchanged
|
||||||
assert_equal "draft", active_order.reload.status
|
assert_equal "draft", active_order.reload.status
|
||||||
end
|
end
|
||||||
@@ -116,7 +116,7 @@ class ExpiredOrdersCleanupJobTest < ActiveJob::TestCase
|
|||||||
assert_nothing_raised do
|
assert_nothing_raised do
|
||||||
ExpiredOrdersCleanupJob.perform_now
|
ExpiredOrdersCleanupJob.perform_now
|
||||||
end
|
end
|
||||||
|
|
||||||
# Order should remain paid
|
# Order should remain paid
|
||||||
assert_equal "paid", paid_order.reload.status
|
assert_equal "paid", paid_order.reload.status
|
||||||
end
|
end
|
||||||
@@ -126,7 +126,7 @@ class ExpiredOrdersCleanupJobTest < ActiveJob::TestCase
|
|||||||
Order.create!(
|
Order.create!(
|
||||||
user: @user,
|
user: @user,
|
||||||
event: @event,
|
event: @event,
|
||||||
status: "draft",
|
status: "draft",
|
||||||
total_amount_cents: 2500,
|
total_amount_cents: 2500,
|
||||||
expires_at: 1.hour.from_now
|
expires_at: 1.hour.from_now
|
||||||
)
|
)
|
||||||
@@ -148,4 +148,4 @@ class ExpiredOrdersCleanupJobTest < ActiveJob::TestCase
|
|||||||
ExpiredOrdersCleanupJob.perform_now
|
ExpiredOrdersCleanupJob.perform_now
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|||||||
@@ -2,11 +2,11 @@ require "test_helper"
|
|||||||
|
|
||||||
class TicketPdfGeneratorTest < ActiveSupport::TestCase
|
class TicketPdfGeneratorTest < ActiveSupport::TestCase
|
||||||
def setup
|
def setup
|
||||||
# Stub QR code generation to avoid dependency issues
|
# Stub QR code generation to avoid dependency issues
|
||||||
mock_qrcode = mock("qrcode")
|
mock_qrcode = mock("qrcode")
|
||||||
mock_qrcode.stubs(:modules).returns([])
|
mock_qrcode.stubs(:modules).returns([])
|
||||||
RQRCode::QRCode.stubs(:new).returns(mock_qrcode)
|
RQRCode::QRCode.stubs(:new).returns(mock_qrcode)
|
||||||
|
|
||||||
@user = User.create!(
|
@user = User.create!(
|
||||||
email: "test@example.com",
|
email: "test@example.com",
|
||||||
password: "password123",
|
password: "password123",
|
||||||
|
|||||||
Reference in New Issue
Block a user