Skip to content

Commit dcc59d2

Browse files
committed
Delete excessive comments
1 parent 44a6e89 commit dcc59d2

File tree

1 file changed

+4
-15
lines changed

1 file changed

+4
-15
lines changed

test/api/test_invoice.py

Lines changed: 4 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -495,7 +495,7 @@ def test_retrieve_invoice_with_validation_type(self, mock_requests):
495495
json=retrieveInvoiceExample,
496496
)
497497

498-
config = Config("token") # is actually checked in mock
498+
config = Config("token")
499499
result = Invoice.retrieve(
500500
config,
501501
uuid="inv_22910fc6-c931-48e7-ac12-90d2cb5f0059",
@@ -507,10 +507,7 @@ def test_retrieve_invoice_with_validation_type(self, mock_requests):
507507
mock_requests.last_request.qs,
508508
{"validation_type": ["all"]},
509509
)
510-
511-
# Struct too complex to do 1:1 comparison
512510
self.assertTrue(isinstance(result, Invoice))
513-
514511
self.assertEqual(result.uuid, "inv_22910fc6-c931-48e7-ac12-90d2cb5f0059")
515512

516513
@requests_mock.mock()
@@ -525,7 +522,7 @@ def test_retrieve_invoice_with_all_params(self, mock_requests):
525522
json=retrieveInvoiceExample,
526523
)
527524

528-
config = Config("token") # is actually checked in mock
525+
config = Config("token")
529526
result = Invoice.retrieve(
530527
config,
531528
uuid="inv_22910fc6-c931-48e7-ac12-90d2cb5f0059",
@@ -539,13 +536,8 @@ def test_retrieve_invoice_with_all_params(self, mock_requests):
539536
self.assertEqual(qs["validation_type"], ["invalid"])
540537
self.assertEqual(qs["include_edit_histories"], ["true"])
541538
self.assertEqual(qs["with_disabled"], ["false"])
542-
543-
# Struct too complex to do 1:1 comparison
544539
self.assertTrue(isinstance(result, Invoice))
545-
546540
self.assertEqual(result.uuid, "inv_22910fc6-c931-48e7-ac12-90d2cb5f0059")
547-
548-
# Verify new fields are present
549541
self.assertTrue(result.disabled)
550542
self.assertEqual(result.disabled_at, datetime(2024, 1, 15, 10, 30, tzinfo=timezone.utc))
551543
self.assertEqual(result.disabled_by, "user@example.com")
@@ -590,7 +582,7 @@ def test_all_invoices_with_validation_type(self, mock_requests):
590582
json=invoiceListExample,
591583
)
592584

593-
config = Config("token") # is actually checked in mock
585+
config = Config("token")
594586
result = Invoice.all(config, validation_type="all").get()
595587

596588
self.assertEqual(mock_requests.call_count, 1, "expected call")
@@ -599,7 +591,6 @@ def test_all_invoices_with_validation_type(self, mock_requests):
599591
{"validation_type": ["all"]},
600592
)
601593

602-
# Struct too complex to do 1:1 comparison
603594
self.assertTrue(isinstance(result, Invoice._many))
604595
self.assertEqual(len(result.invoices), 1)
605596

@@ -615,7 +606,7 @@ def test_all_invoices_with_all_params(self, mock_requests):
615606
json=invoiceListExample,
616607
)
617608

618-
config = Config("token") # is actually checked in mock
609+
config = Config("token")
619610
result = Invoice.all(
620611
config,
621612
validation_type="valid",
@@ -628,7 +619,5 @@ def test_all_invoices_with_all_params(self, mock_requests):
628619
self.assertEqual(qs["validation_type"], ["valid"])
629620
self.assertEqual(qs["include_edit_histories"], ["true"])
630621
self.assertEqual(qs["with_disabled"], ["true"])
631-
632-
# Struct too complex to do 1:1 comparison
633622
self.assertTrue(isinstance(result, Invoice._many))
634623
self.assertEqual(len(result.invoices), 1)

0 commit comments

Comments
 (0)