Skip to content

Fix swapped Bucket Id and Version Id columns in get_object_versions.#232

Merged
RachelTucker merged 2 commits into
SpectraLogic:masterfrom
RachelTucker:fix-display-get-object-versions
Jul 10, 2026
Merged

Fix swapped Bucket Id and Version Id columns in get_object_versions.#232
RachelTucker merged 2 commits into
SpectraLogic:masterfrom
RachelTucker:fix-display-get-object-versions

Conversation

@RachelTucker

Copy link
Copy Markdown
Contributor

Fixed a bug where the "Bucket Id" and "Version Id" columns were swapped in the get_object_versions command output.

Changes

  • Modified GetObjectVersionsView.java to correctly map bucketId to the first column and versionId to the fifth column.
  • Added a unit test GetObjectVersionsView_Test.java and an integration test in FeatureIntegration_Test.java to verify column alignment.

scribe
scribe previously approved these changes Jul 10, 2026

@scribe scribe left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Modified PosixFileMetadata_Test to account for modern Linux environments where file birth time (creation time) is readable via statx but cannot be modified. Changing the assertion to greaterThanOrEqualTo(0) prevents failures on systems where birth time restoration is a no-op.

Additionally refactored PosixFileMetadata to use OffsetDateTime for more direct and robust time parsing, avoiding unnecessary local timezone conversions.
@RachelTucker
RachelTucker merged commit 74f226a into SpectraLogic:master Jul 10, 2026
4 checks passed
@RachelTucker
RachelTucker deleted the fix-display-get-object-versions branch July 10, 2026 21:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants