Skip to content

Commit

Permalink
Fix exclusions for command dedup in compat tests (#10767)
Browse files Browse the repository at this point in the history
We need to differentiate between 3 cases:

1. Ledger API test tool versions that only support exclusion of whole
suites.
2. Ledger API test tool versions where the tests had different names.
3. and the current state

changelog_begin
changelog_end
  • Loading branch information
cocreature authored Sep 3, 2021
1 parent 50fecfb commit 7515871
Showing 1 changed file with 31 additions and 0 deletions.
31 changes: 31 additions & 0 deletions compatibility/bazel_tools/testing.bzl
Original file line number Diff line number Diff line change
Expand Up @@ -307,6 +307,37 @@ excluded_test_tool_tests = [
],
},
{
# Tests got renamed in
# https://github.com/digital-asset/daml/commit/f2707cc54f5b7da339bc565bc322be1e57db5edb
"end": last_nongranular_test_tool,
"platform_ranges": [
{
"start": "1.17.0-snapshot.20210831.7702.1.f058c2f1",
"exclusions": [
"CommandDeduplicationIT",
],
},
],
},
{
# Tests got renamed in
# https://github.com/digital-asset/daml/commit/f2707cc54f5b7da339bc565bc322be1e57db5edb
"start": first_granular_test_tool,
"end": "1.5.0-snapshot.20200907.5151.0.eb68e680",
"platform_ranges": [
{
"start": "1.17.0-snapshot.20210831.7702.1.f058c2f1",
"exclusions": [
"CommandDeduplicationIT:CDSimpleDeduplication",
"CommandDeduplicationIT:CDSimpleDeduplicationCommandClient",
],
},
],
},
{
# Tests got renamed in
# https://github.com/digital-asset/daml/commit/f2707cc54f5b7da339bc565bc322be1e57db5edb
"start": "1.5.0-snapshot.20200907.5151.0.eb68e680",
"end": "1.17.0-snapshot.20210831.7702.0.f058c2f1",
"platform_ranges": [
{
Expand Down

0 comments on commit 7515871

Please sign in to comment.