Skip to content

Commit 673f32b

Browse files
authored
Merge pull request #5811 from woocommerce/feat/5744-revenue-stats-without-rounding
Show revenue stats without rounding
2 parents b17f04c + e7a666a commit 673f32b

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

WooCommerce/Classes/ViewRelated/Dashboard/Stats v4/StoreStatsPeriodViewModel.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -155,7 +155,7 @@ private extension StoreStatsPeriodViewModel {
155155

156156
func createRevenueStats(orderStatsData: OrderStatsData, selectedIntervalIndex: Int?) -> String {
157157
if let revenue = revenue(at: selectedIntervalIndex, orderStats: orderStatsData.stats, orderStatsIntervals: orderStatsData.intervals) {
158-
return currencyFormatter.formatHumanReadableAmount(String("\(revenue)"), with: currencyCode) ?? String()
158+
return currencyFormatter.formatAmount(revenue, with: currencyCode) ?? String()
159159
} else {
160160
return Constants.placeholderText
161161
}

WooCommerce/WooCommerceTests/ViewRelated/Dashboard/Stats V4/StoreStatsPeriodViewModelTests.swift

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -53,13 +53,13 @@ final class StoreStatsPeriodViewModelTests: XCTestCase {
5353
// When
5454
let orderStats = OrderStatsV4(siteID: siteID,
5555
granularity: timeRange.intervalGranularity,
56-
totals: .fake().copy(totalOrders: 3, grossRevenue: 62.7),
56+
totals: .fake().copy(totalOrders: 3, grossRevenue: 6220.7),
5757
intervals: [.fake()])
5858
insertOrderStats(orderStats, timeRange: timeRange)
5959

6060
// Then
6161
XCTAssertEqual(orderStatsTextValues, ["-", "3"])
62-
XCTAssertEqual(revenueStatsTextValues, ["-", "$62"])
62+
XCTAssertEqual(revenueStatsTextValues, ["-", "$6,220.70"])
6363
XCTAssertEqual(visitorStatsTextValues, ["-"])
6464
XCTAssertEqual(conversionStatsTextValues, ["-"])
6565
}
@@ -168,7 +168,7 @@ final class StoreStatsPeriodViewModelTests: XCTestCase {
168168

169169
// Then
170170
XCTAssertEqual(orderStatsTextValues, ["-", "3", "1"])
171-
XCTAssertEqual(revenueStatsTextValues, ["-", "$62", "$25"])
171+
XCTAssertEqual(revenueStatsTextValues, ["-", "$62.70", "$25.00"])
172172
XCTAssertEqual(visitorStatsTextValues, ["-"])
173173
XCTAssertEqual(conversionStatsTextValues, ["-"])
174174
}

0 commit comments

Comments
 (0)