Skip to content

chore(storage): fix order dependency in get_object_contexts test#34844

Open
aandreassa wants to merge 1 commit into
mainfrom
fix-storage-order-test
Open

chore(storage): fix order dependency in get_object_contexts test#34844
aandreassa wants to merge 1 commit into
mainfrom
fix-storage-order-test

Conversation

@aandreassa

@aandreassa aandreassa commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Currently this fails if the order is incorrect, introducing flakiness:

1) Failure:
Files Snippets::get_object_contexts#test_0001_fetches all object contexts [acceptance/files_test.rb:341]:
In stdout.
--- expected
+++ actual
@@ -1,4 +1,4 @@
 "Custom Contexts for path/file_name_9d512d3ea21775ecf8d953f9f31138f6.txt are:
-Key: my-custom-key, Value: my-custom-value
 Key: my-custom-key-2, Value: my-custom-value-2
+Key: my-custom-key, Value: my-custom-value
 "

Noticed in #34751

Currently this fails if the order is incorrect:
```
-Key: my-custom-key, Value: my-custom-value
 Key: my-custom-key-2, Value: my-custom-value-2
+Key: my-custom-key, Value: my-custom-value
```

Noticed in #34751
@aandreassa
aandreassa force-pushed the fix-storage-order-test branch from c7d29af to 6789003 Compare July 16, 2026 22:04
@aandreassa
aandreassa marked this pull request as ready for review July 16, 2026 22:09
@aandreassa
aandreassa requested review from a team and yoshi-approver as code owners July 16, 2026 22:09
@aandreassa aandreassa closed this Jul 16, 2026
@aandreassa aandreassa reopened this Jul 16, 2026
@aandreassa

Copy link
Copy Markdown
Contributor Author

@bajajneha27 or @cpriti-os if I could get a LGTM on this flaky test. TY!

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.

4 participants