Skip to content

Commit 866c91c

Browse files
committed
remove unused isSubscriptionProduct and associated tests
1 parent b5e48c7 commit 866c91c

File tree

2 files changed

+0
-79
lines changed

2 files changed

+0
-79
lines changed

WooCommerce/Classes/ViewRelated/Products/ProductSelector/ProductSelectorViewModel.swift

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -373,13 +373,6 @@ final class ProductSelectorViewModel: ObservableObject {
373373
return variableProduct(for: productOrVariationID) != nil
374374
}
375375

376-
func isSubscriptionProduct(productOrVariationID: Int64) -> Bool {
377-
guard let product = products.first(where: { $0.productID == productOrVariationID }) else {
378-
return false
379-
}
380-
return product.productType == .subscription || product.productType == .variableSubscription
381-
}
382-
383376
func variationCheckboxTapped(for productOrVariationID: Int64) {
384377
if toggleAllVariationsOnSelection {
385378
toggleSelectionForAllVariations(of: productOrVariationID)

WooCommerce/WooCommerceTests/ViewRelated/Orders/Order Creation/ProductSelectorViewModelTests.swift

Lines changed: 0 additions & 72 deletions
Original file line numberDiff line numberDiff line change
@@ -1710,78 +1710,6 @@ final class ProductSelectorViewModelTests: XCTestCase {
17101710
// Then
17111711
XCTAssertEqual(viewModel.filterListViewModel.criteria.stockStatus, .inStock)
17121712
}
1713-
1714-
func test_isSubscriptionProduct_returns_true_for_subscription_product() {
1715-
// Given
1716-
let subscriptionProduct = Product.fake().copy(siteID: sampleSiteID,
1717-
productID: 1,
1718-
productTypeKey: ProductType.subscription.rawValue,
1719-
purchasable: true)
1720-
insert(subscriptionProduct)
1721-
let viewModel = ProductSelectorViewModel(siteID: sampleSiteID,
1722-
source: .orderForm(flow: .creation),
1723-
storageManager: storageManager)
1724-
1725-
// When
1726-
let result = viewModel.isSubscriptionProduct(productOrVariationID: subscriptionProduct.productID)
1727-
1728-
// Then
1729-
XCTAssertTrue(result)
1730-
}
1731-
1732-
func test_isSubscriptionProduct_returns_true_for_variable_subscription_product() {
1733-
// Given
1734-
let variableSubscriptionProduct = Product.fake().copy(siteID: sampleSiteID,
1735-
productID: 2,
1736-
productTypeKey: ProductType.variableSubscription.rawValue,
1737-
purchasable: true)
1738-
insert(variableSubscriptionProduct)
1739-
let viewModel = ProductSelectorViewModel(siteID: sampleSiteID,
1740-
source: .orderForm(flow: .creation),
1741-
storageManager: storageManager)
1742-
1743-
// When
1744-
let result = viewModel.isSubscriptionProduct(productOrVariationID: variableSubscriptionProduct.productID)
1745-
1746-
// Then
1747-
XCTAssertTrue(result)
1748-
}
1749-
1750-
func test_isSubscriptionProduct_returns_false_for_simple_product() {
1751-
// Given
1752-
let simpleProduct = Product.fake().copy(siteID: sampleSiteID,
1753-
productID: 3,
1754-
productTypeKey: ProductType.simple.rawValue,
1755-
purchasable: true)
1756-
insert(simpleProduct)
1757-
let viewModel = ProductSelectorViewModel(siteID: sampleSiteID,
1758-
source: .orderForm(flow: .creation),
1759-
storageManager: storageManager)
1760-
1761-
// When
1762-
let result = viewModel.isSubscriptionProduct(productOrVariationID: simpleProduct.productID)
1763-
1764-
// Then
1765-
XCTAssertFalse(result)
1766-
}
1767-
1768-
func test_isSubscriptionProduct_returns_false_for_variable_product() {
1769-
// Given
1770-
let variableProduct = Product.fake().copy(siteID: sampleSiteID,
1771-
productID: 4,
1772-
productTypeKey: ProductType.variable.rawValue,
1773-
purchasable: true)
1774-
insert(variableProduct)
1775-
let viewModel = ProductSelectorViewModel(siteID: sampleSiteID,
1776-
source: .orderForm(flow: .creation),
1777-
storageManager: storageManager)
1778-
1779-
// When
1780-
let result = viewModel.isSubscriptionProduct(productOrVariationID: variableProduct.productID)
1781-
1782-
// Then
1783-
XCTAssertFalse(result)
1784-
}
17851713
}
17861714

17871715
// MARK: - Utils

0 commit comments

Comments
 (0)