Skip to content

Commit

Permalink
Add cli args
Browse files Browse the repository at this point in the history
  • Loading branch information
erohmensing committed Mar 22, 2023
1 parent ae1c263 commit b538a8c
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 12 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -19,8 +19,8 @@
from yaml import Loader

parser = argparse.ArgumentParser(description="Migrate legacy acceptance-test-config.yml to the latest configuration format.")
parser.add_argument("config_path", type=str, help="Path to the acceptance-test-config.yml to migrate.")

parser.add_argument("--connectors", nargs='*')
parser.add_argument("--file")

def get_new_config_format(config_path: Path):

Expand Down Expand Up @@ -65,6 +65,6 @@ def migrate_configuration(config_path):
else:
connector_names = []

for connector in connector_names:
for connector in ["source-bing-ads"]:
config_path = utils.acceptance_test_config_path(connector)
migrate_configuration(config_path)
Original file line number Diff line number Diff line change
Expand Up @@ -19,23 +19,24 @@ acceptance_tests:
tests:
- config_path: secrets/config.json
expect_records:
path: "integration_tests/expected_records.txt"
extra_records: yes
path: integration_tests/expected_records.txt
extra_records: true
empty_streams:
- name: account_performance_report_hourly
bypass_reason: "Hourly reports are disabled, because sync is too long"
bypass_reason: Hourly reports are disabled, because sync is too long
- name: ad_group_performance_report_hourly
bypass_reason: "Hourly reports are disabled, because sync is too long"
bypass_reason: Hourly reports are disabled, because sync is too long
- name: ad_performance_report_hourly
bypass_reason: "Hourly reports are disabled, because sync is too long"
bypass_reason: Hourly reports are disabled, because sync is too long
- name: campaign_performance_report_hourly
bypass_reason: "Hourly reports are disabled, because sync is too long"
bypass_reason: Hourly reports are disabled, because sync is too long
- name: keyword_performance_report_hourly
bypass_reason: "Hourly reports are disabled, because sync is too long"
bypass_reason: Hourly reports are disabled, because sync is too long
full_refresh:
tests:
- config_path: secrets/config.json
configured_catalog_path: integration_tests/configured_catalog.json
incremental:
tests:
bypass_reason: "SAT doesn't support complex nested states used in incremental report streams"
tests: null
bypass_reason: SAT doesn't support complex nested states used in incremental report
streams

0 comments on commit b538a8c

Please sign in to comment.