Skip to content

Commit

Permalink
Revert "Mute failing snapshot status tests (#15652)" (#15876)
Browse files Browse the repository at this point in the history
This reverts commit 48cf5f0.

Signed-off-by: Lakshya Taragi <[email protected]>
  • Loading branch information
ltaragi authored Sep 16, 2024
1 parent bd26056 commit c709400
Showing 1 changed file with 34 additions and 36 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -25,40 +25,38 @@ setup:
snapshot: test_snapshot
wait_for_completion: true

# TODO: fix and unmute tests
- do:
snapshot.status:
repository: test_repo_status_1
snapshot: test_snapshot

- is_true: snapshots
- match: { snapshots.0.snapshot: test_snapshot }
- match: { snapshots.0.state: SUCCESS }
- gt: { snapshots.0.stats.incremental.file_count: 0 }
- gt: { snapshots.0.stats.incremental.size_in_bytes: 0 }
- gt: { snapshots.0.stats.total.file_count: 0 }
- gt: { snapshots.0.stats.total.size_in_bytes: 0 }
- is_true: snapshots.0.stats.start_time_in_millis
## fast in memory snapshots can take less than one millisecond to complete.
- gte: { snapshots.0.stats.time_in_millis: 0 }

---
"Get missing snapshot status throws an exception":

- do:
catch: /snapshot_missing_exception.+ is missing/
snapshot.status:
repository: test_repo_status_1
snapshot: test_nonexistent_snapshot

---
"Get missing snapshot status succeeds when ignoreUnavailable is true":

- do:
snapshot.status:
repository: test_repo_status_1
snapshot: test_nonexistent_snapshot
ignore_unavailable: true

# - do:
# snapshot.status:
# repository: test_repo_status_1
# snapshot: test_snapshot
#
# - is_true: snapshots
# - match: { snapshots.0.snapshot: test_snapshot }
# - match: { snapshots.0.state: SUCCESS }
# - gt: { snapshots.0.stats.incremental.file_count: 0 }
# - gt: { snapshots.0.stats.incremental.size_in_bytes: 0 }
# - gt: { snapshots.0.stats.total.file_count: 0 }
# - gt: { snapshots.0.stats.total.size_in_bytes: 0 }
# - is_true: snapshots.0.stats.start_time_in_millis
### fast in memory snapshots can take less than one millisecond to complete.
# - gte: { snapshots.0.stats.time_in_millis: 0 }
#
#---
#"Get missing snapshot status throws an exception":
#
# - do:
# catch: /snapshot_missing_exception.+ is missing/
# snapshot.status:
# repository: test_repo_status_1
# snapshot: test_nonexistent_snapshot
#
#---
#"Get missing snapshot status succeeds when ignoreUnavailable is true":
#
# - do:
# snapshot.status:
# repository: test_repo_status_1
# snapshot: test_nonexistent_snapshot
# ignore_unavailable: true
#
# - is_true: snapshots
- is_true: snapshots

0 comments on commit c709400

Please sign in to comment.