Skip to content

Commit

Permalink
Don't multiply tests on LogForTesting providers.
Browse files Browse the repository at this point in the history
There are many of these, and this needlessly explodes the number
of tests that are created when adding debugging.
  • Loading branch information
robertwb committed Jan 16, 2025
1 parent 4611a81 commit 55f1639
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion sdks/python/apache_beam/yaml/integration_tests.py
Original file line number Diff line number Diff line change
Expand Up @@ -124,7 +124,10 @@ def provider_sets(spec, require_available=False):
for p in spec['pipelines']))

def filter_to_available(t, providers):
if require_available:
if t == 'LogForTesting':
# Don't fan out to all the (many) possibilities for this one...
return [p for p in providers if isinstance(p, yaml_provider.InlineProvider)]
elif require_available:
for p in providers:
if not p.available():
raise ValueError("Provider {p} required for {t} is not available.")
Expand Down

0 comments on commit 55f1639

Please sign in to comment.