-
Notifications
You must be signed in to change notification settings - Fork 3.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
SAT: support yaml files in spec_path #12228
Conversation
def connector_spec_fixture(connector_spec_path) -> ConnectorSpecification: | ||
return ConnectorSpecification.parse_file(connector_spec_path) | ||
spec_obj = load_yaml_or_json_path(connector_spec_path) | ||
return ConnectorSpecification.parse_obj(spec_obj) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks like we can't test these fixtures directly, so moved the yaml/json logic into a utility method
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
awesome!
/publish connector=bases/source-acceptance-test
|
* support yaml files as spec_path * no unnecessary f-string * move to util for better testing * bump version, update changelog
What
We now support loading the spec from a yaml file in the CDK (#12104), but the SAT still expects a json file as
spec_path
for one of the tests.How
Check if
spec_path
is a.json
or.yaml
file and load accordingly.