Skip to content

Commit 956d6b6

Browse files
Merge branch 'master' into ECWID-150578-New-IS-Category-name-for-product
2 parents b0e1322 + 6a4b9c9 commit 956d6b6

5 files changed

Lines changed: 35 additions & 3 deletions

File tree

.github/workflows/pull-request.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@ jobs:
4848
${{ github.workspace }}/build/reports/detekt/detekt.html
4949
5050
- name: Upload static analysis results
51-
uses: github/codeql-action/upload-sarif@v2
51+
uses: github/codeql-action/upload-sarif@v3
5252
if: always()
5353
with:
5454
sarif_file: ${{ github.workspace }}/build/reports/detekt/detekt.sarif

.github/workflows/push-to-master.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ jobs:
3737
GRGIT_PASS: ${{ secrets.GRGIT_PASS }}
3838

3939
- name: Upload static analysis results
40-
uses: github/codeql-action/upload-sarif@v2
40+
uses: github/codeql-action/upload-sarif@v3
4141
if: always()
4242
with:
4343
sarif_file: ${{ github.workspace }}/build/reports/detekt/detekt.sarif

src/main/kotlin/com/ecwid/apiclient/v3/dto/order/enums/DiscountBase.kt

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,5 +4,8 @@ enum class DiscountBase {
44
ON_TOTAL,
55
ON_MEMBERSHIP,
66
ON_TOTAL_AND_MEMBERSHIP,
7+
SHIPPING,
8+
ITEM,
9+
SUBTOTAL,
710
CUSTOM
811
}

src/main/kotlin/com/ecwid/apiclient/v3/util/SecurePatterns.kt

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,9 @@ private const val PARAM_VALUE_PATTERN = "([^;,)]+)"
44

55
private val GLOBAL_SECURE_PATTERNS = listOf(
66
createKeyValueSecurePattern("email"),
7+
createKeyValueSecurePattern("token"),
78
createJsonSecurePattern("email"),
9+
createJsonSecurePattern("token"),
810
createJsonSecurePattern("name"),
911
createJsonSecurePattern("firstName"),
1012
createJsonSecurePattern("lastName"),

src/test/kotlin/com/ecwid/apiclient/v3/util/OrderUtils.kt

Lines changed: 28 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,10 @@ fun generateTestOrder(): UpdatedOrder {
7373
generateTestOnTotalDiscountInfo(),
7474
generateTestOnMembershipDiscountInfo(),
7575
generateTestOnTotalAndMembershipDiscountInfo(),
76-
generateTestCustomDiscountInfo()
76+
generateTestCustomDiscountInfo(),
77+
generateTestItemDiscountInfo(),
78+
generateTestShippingDiscountInfo(),
79+
generateTestSubtotalDiscountInfo()
7780
),
7881

7982
// TODO Pass real discount coupon code when API client will support this
@@ -190,6 +193,30 @@ private fun generateTestCustomDiscountInfo() = UpdatedOrder.DiscountInfo(
190193
description = "Custom discount " + randomAlphanumeric(8)
191194
)
192195

196+
private fun generateTestItemDiscountInfo() = UpdatedOrder.DiscountInfo(
197+
value = randomPrice(),
198+
type = randomEnumValue<DiscountType>(),
199+
base = DiscountBase.ITEM,
200+
orderTotal = randomPrice(),
201+
description = "On item discount " + randomAlphanumeric(8)
202+
)
203+
204+
private fun generateTestShippingDiscountInfo() = UpdatedOrder.DiscountInfo(
205+
value = randomPrice(),
206+
type = randomEnumValue<DiscountType>(),
207+
base = DiscountBase.SHIPPING,
208+
orderTotal = randomPrice(),
209+
description = "On shipping discount " + randomAlphanumeric(8)
210+
)
211+
212+
private fun generateTestSubtotalDiscountInfo() = UpdatedOrder.DiscountInfo(
213+
value = randomPrice(),
214+
type = randomEnumValue<DiscountType>(),
215+
base = DiscountBase.SUBTOTAL,
216+
orderTotal = randomPrice(),
217+
description = "On subtotal discount " + randomAlphanumeric(8)
218+
)
219+
193220
private fun generateTestOrderItem() = UpdatedOrder.OrderItem(
194221
// var id: Int? = null, // TODO readonly
195222
// productId = com.ecwid.api.v3.randomId(), // TODO need to pass existing product id

0 commit comments

Comments
 (0)