test(integration_tests): add only_dir configuration to test_config.yaml#4629
test(integration_tests): add only_dir configuration to test_config.yaml#4629PranjalC100 wants to merge 1 commit intomasterfrom
Conversation
Summary of ChangesHello, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request updates the integration test configuration to explicitly enable the 'only_dir' mounting mode for several test suites. By standardizing this configuration, the project ensures consistent test execution behavior across all relevant test packages, aligning with recent framework updates. Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for GitHub and other Google products, sign up here. Footnotes
|
There was a problem hiding this comment.
Code Review
This pull request updates the integration test configuration in tools/integration_tests/test_config.yaml by adding the only_dir: "${ONLY_DIR}" parameter to several test suites, including inactive_stream_timeout, flag_optimizations, kernel_list_cache, negative_stat_cache, and managed_folders. I have no feedback to provide.
Description
This PR updates
tools/integration_tests/test_config.yamlto explicitly include theonly_dir: "${ONLY_DIR}"field for several integration test suites. As noted in recent test framework discussions, the test framework automatically enables theonly_direxecution mode if the test package contains this field.This change ensures the following test suites include the
only_dirmount configurations int test_config.yaml:inactive_stream_timeoutflag_optimizationskernel_list_cachenegative_stat_cachemanaged_foldersWith this configuration update, we will now be running the
only_dirtests for all 10 of the test packages that implementonly_dir_mountingincluding :local_file*
operations*
read_cache*
rename_dir_limit*
requester_pays_bucketLink to the issue in case of a bug fix.
Related to http://b/496966414
Testing details
Any backward incompatible change? If so, please explain.
No.