Skip to content

Commit 51d40a6

Browse files
author
octavia-squidington-iii
committed
Auto-fix lint and format issues
1 parent c932185 commit 51d40a6

File tree

3 files changed

+46
-62
lines changed

3 files changed

+46
-62
lines changed

airbyte_cdk/sources/declarative/models/declarative_component_schema.py

Lines changed: 39 additions & 59 deletions
Original file line numberDiff line numberDiff line change
@@ -618,9 +618,7 @@ class OAuthAuthenticator(BaseModel):
618618
scopes: Optional[List[str]] = Field(
619619
None,
620620
description="List of scopes that should be granted to the access token.",
621-
examples=[
622-
["crm.list.read", "crm.objects.contacts.read", "crm.schema.contacts.read"]
623-
],
621+
examples=[["crm.list.read", "crm.objects.contacts.read", "crm.schema.contacts.read"]],
624622
title="Scopes",
625623
)
626624
token_expiry_date: Optional[str] = Field(
@@ -1126,28 +1124,24 @@ class OAuthConfigSpecification(BaseModel):
11261124
class Config:
11271125
extra = Extra.allow
11281126

1129-
oauth_user_input_from_connector_config_specification: Optional[Dict[str, Any]] = (
1130-
Field(
1131-
None,
1132-
description="OAuth specific blob. This is a Json Schema used to validate Json configurations used as input to OAuth.\nMust be a valid non-nested JSON that refers to properties from ConnectorSpecification.connectionSpecification\nusing special annotation 'path_in_connector_config'.\nThese are input values the user is entering through the UI to authenticate to the connector, that might also shared\nas inputs for syncing data via the connector.\nExamples:\nif no connector values is shared during oauth flow, oauth_user_input_from_connector_config_specification=[]\nif connector values such as 'app_id' inside the top level are used to generate the API url for the oauth flow,\n oauth_user_input_from_connector_config_specification={\n app_id: {\n type: string\n path_in_connector_config: ['app_id']\n }\n }\nif connector values such as 'info.app_id' nested inside another object are used to generate the API url for the oauth flow,\n oauth_user_input_from_connector_config_specification={\n app_id: {\n type: string\n path_in_connector_config: ['info', 'app_id']\n }\n }",
1133-
examples=[
1134-
{"app_id": {"type": "string", "path_in_connector_config": ["app_id"]}},
1135-
{
1136-
"app_id": {
1137-
"type": "string",
1138-
"path_in_connector_config": ["info", "app_id"],
1139-
}
1140-
},
1141-
],
1142-
title="OAuth user input",
1143-
)
1127+
oauth_user_input_from_connector_config_specification: Optional[Dict[str, Any]] = Field(
1128+
None,
1129+
description="OAuth specific blob. This is a Json Schema used to validate Json configurations used as input to OAuth.\nMust be a valid non-nested JSON that refers to properties from ConnectorSpecification.connectionSpecification\nusing special annotation 'path_in_connector_config'.\nThese are input values the user is entering through the UI to authenticate to the connector, that might also shared\nas inputs for syncing data via the connector.\nExamples:\nif no connector values is shared during oauth flow, oauth_user_input_from_connector_config_specification=[]\nif connector values such as 'app_id' inside the top level are used to generate the API url for the oauth flow,\n oauth_user_input_from_connector_config_specification={\n app_id: {\n type: string\n path_in_connector_config: ['app_id']\n }\n }\nif connector values such as 'info.app_id' nested inside another object are used to generate the API url for the oauth flow,\n oauth_user_input_from_connector_config_specification={\n app_id: {\n type: string\n path_in_connector_config: ['info', 'app_id']\n }\n }",
1130+
examples=[
1131+
{"app_id": {"type": "string", "path_in_connector_config": ["app_id"]}},
1132+
{
1133+
"app_id": {
1134+
"type": "string",
1135+
"path_in_connector_config": ["info", "app_id"],
1136+
}
1137+
},
1138+
],
1139+
title="OAuth user input",
11441140
)
1145-
oauth_connector_input_specification: Optional[OauthConnectorInputSpecification] = (
1146-
Field(
1147-
None,
1148-
description='The DeclarativeOAuth specific blob.\nPertains to the fields defined by the connector relating to the OAuth flow.\n\nInterpolation capabilities:\n- The variables placeholders are declared as `{{my_var}}`.\n- The nested resolution variables like `{{ {{my_nested_var}} }}` is allowed as well.\n\n- The allowed interpolation context is:\n + base64Encoder - encode to `base64`, {{ {{my_var_a}}:{{my_var_b}} | base64Encoder }}\n + base64Decorer - decode from `base64` encoded string, {{ {{my_string_variable_or_string_value}} | base64Decoder }}\n + urlEncoder - encode the input string to URL-like format, {{ https://test.host.com/endpoint | urlEncoder}}\n + urlDecorer - decode the input url-encoded string into text format, {{ urlDecoder:https%3A%2F%2Fairbyte.io | urlDecoder}}\n + codeChallengeS256 - get the `codeChallenge` encoded value to provide additional data-provider specific authorisation values, {{ {{state_value}} | codeChallengeS256 }}\n\nExamples:\n - The TikTok Marketing DeclarativeOAuth spec:\n {\n "oauth_connector_input_specification": {\n "type": "object",\n "additionalProperties": false,\n "properties": {\n "consent_url": "https://ads.tiktok.com/marketing_api/auth?{{client_id_key}}={{client_id_value}}&{{redirect_uri_key}}={{ {{redirect_uri_value}} | urlEncoder}}&{{state_key}}={{state_value}}",\n "access_token_url": "https://business-api.tiktok.com/open_api/v1.3/oauth2/access_token/",\n "access_token_params": {\n "{{ auth_code_key }}": "{{ auth_code_value }}",\n "{{ client_id_key }}": "{{ client_id_value }}",\n "{{ client_secret_key }}": "{{ client_secret_value }}"\n },\n "access_token_headers": {\n "Content-Type": "application/json",\n "Accept": "application/json"\n },\n "extract_output": ["data.access_token"],\n "client_id_key": "app_id",\n "client_secret_key": "secret",\n "auth_code_key": "auth_code"\n }\n }\n }',
1149-
title="DeclarativeOAuth Connector Specification",
1150-
)
1141+
oauth_connector_input_specification: Optional[OauthConnectorInputSpecification] = Field(
1142+
None,
1143+
description='The DeclarativeOAuth specific blob.\nPertains to the fields defined by the connector relating to the OAuth flow.\n\nInterpolation capabilities:\n- The variables placeholders are declared as `{{my_var}}`.\n- The nested resolution variables like `{{ {{my_nested_var}} }}` is allowed as well.\n\n- The allowed interpolation context is:\n + base64Encoder - encode to `base64`, {{ {{my_var_a}}:{{my_var_b}} | base64Encoder }}\n + base64Decorer - decode from `base64` encoded string, {{ {{my_string_variable_or_string_value}} | base64Decoder }}\n + urlEncoder - encode the input string to URL-like format, {{ https://test.host.com/endpoint | urlEncoder}}\n + urlDecorer - decode the input url-encoded string into text format, {{ urlDecoder:https%3A%2F%2Fairbyte.io | urlDecoder}}\n + codeChallengeS256 - get the `codeChallenge` encoded value to provide additional data-provider specific authorisation values, {{ {{state_value}} | codeChallengeS256 }}\n\nExamples:\n - The TikTok Marketing DeclarativeOAuth spec:\n {\n "oauth_connector_input_specification": {\n "type": "object",\n "additionalProperties": false,\n "properties": {\n "consent_url": "https://ads.tiktok.com/marketing_api/auth?{{client_id_key}}={{client_id_value}}&{{redirect_uri_key}}={{ {{redirect_uri_value}} | urlEncoder}}&{{state_key}}={{state_value}}",\n "access_token_url": "https://business-api.tiktok.com/open_api/v1.3/oauth2/access_token/",\n "access_token_params": {\n "{{ auth_code_key }}": "{{ auth_code_value }}",\n "{{ client_id_key }}": "{{ client_id_value }}",\n "{{ client_secret_key }}": "{{ client_secret_value }}"\n },\n "access_token_headers": {\n "Content-Type": "application/json",\n "Accept": "application/json"\n },\n "extract_output": ["data.access_token"],\n "client_id_key": "app_id",\n "client_secret_key": "secret",\n "auth_code_key": "auth_code"\n }\n }\n }',
1144+
title="DeclarativeOAuth Connector Specification",
11511145
)
11521146
complete_oauth_output_specification: Optional[Dict[str, Any]] = Field(
11531147
None,
@@ -1165,9 +1159,7 @@ class Config:
11651159
complete_oauth_server_input_specification: Optional[Dict[str, Any]] = Field(
11661160
None,
11671161
description="OAuth specific blob. This is a Json Schema used to validate Json configurations persisted as Airbyte Server configurations.\nMust be a valid non-nested JSON describing additional fields configured by the Airbyte Instance or Workspace Admins to be used by the\nserver when completing an OAuth flow (typically exchanging an auth code for refresh token).\nExamples:\n complete_oauth_server_input_specification={\n client_id: {\n type: string\n },\n client_secret: {\n type: string\n }\n }",
1168-
examples=[
1169-
{"client_id": {"type": "string"}, "client_secret": {"type": "string"}}
1170-
],
1162+
examples=[{"client_id": {"type": "string"}, "client_secret": {"type": "string"}}],
11711163
title="OAuth input specification",
11721164
)
11731165
complete_oauth_server_output_specification: Optional[Dict[str, Any]] = Field(
@@ -1899,9 +1891,7 @@ class RecordSelector(BaseModel):
18991891
description="Responsible for filtering records to be emitted by the Source.",
19001892
title="Record Filter",
19011893
)
1902-
schema_normalization: Optional[
1903-
Union[SchemaNormalization, CustomSchemaNormalization]
1904-
] = Field(
1894+
schema_normalization: Optional[Union[SchemaNormalization, CustomSchemaNormalization]] = Field(
19051895
None,
19061896
description="Responsible for normalization according to the schema.",
19071897
title="Schema Normalization",
@@ -2147,9 +2137,7 @@ class Config:
21472137
extra = Extra.allow
21482138

21492139
type: Literal["DeclarativeStream"]
2150-
name: Optional[str] = Field(
2151-
"", description="The stream name.", example=["Users"], title="Name"
2152-
)
2140+
name: Optional[str] = Field("", description="The stream name.", example=["Users"], title="Name")
21532141
retriever: Union[SimpleRetriever, AsyncRetriever, CustomRetriever] = Field(
21542142
...,
21552143
description="Component used to coordinate how records are extracted across stream slices and request pages.",
@@ -2327,20 +2315,18 @@ class HttpRequester(BaseModelWithDeprecations):
23272315
description="Allows for retrieving a dynamic set of properties from an API endpoint which can be injected into outbound request using the stream_partition.extra_fields.",
23282316
title="Fetch Properties from Endpoint",
23292317
)
2330-
request_parameters: Optional[Union[Dict[str, Union[str, QueryProperties]], str]] = (
2331-
Field(
2332-
None,
2333-
description="Specifies the query parameters that should be set on an outgoing HTTP request given the inputs.",
2334-
examples=[
2335-
{"unit": "day"},
2336-
{
2337-
"query": 'last_event_time BETWEEN TIMESTAMP "{{ stream_interval.start_time }}" AND TIMESTAMP "{{ stream_interval.end_time }}"'
2338-
},
2339-
{"searchIn": "{{ ','.join(config.get('search_in', [])) }}"},
2340-
{"sort_by[asc]": "updated_at"},
2341-
],
2342-
title="Query Parameters",
2343-
)
2318+
request_parameters: Optional[Union[Dict[str, Union[str, QueryProperties]], str]] = Field(
2319+
None,
2320+
description="Specifies the query parameters that should be set on an outgoing HTTP request given the inputs.",
2321+
examples=[
2322+
{"unit": "day"},
2323+
{
2324+
"query": 'last_event_time BETWEEN TIMESTAMP "{{ stream_interval.start_time }}" AND TIMESTAMP "{{ stream_interval.end_time }}"'
2325+
},
2326+
{"searchIn": "{{ ','.join(config.get('search_in', [])) }}"},
2327+
{"sort_by[asc]": "updated_at"},
2328+
],
2329+
title="Query Parameters",
23442330
)
23452331
request_headers: Optional[Union[Dict[str, str], str]] = Field(
23462332
None,
@@ -2530,9 +2516,7 @@ class QueryProperties(BaseModel):
25302516

25312517
class StateDelegatingStream(BaseModel):
25322518
type: Literal["StateDelegatingStream"]
2533-
name: str = Field(
2534-
..., description="The stream name.", example=["Users"], title="Name"
2535-
)
2519+
name: str = Field(..., description="The stream name.", example=["Users"], title="Name")
25362520
full_refresh_stream: DeclarativeStream = Field(
25372521
...,
25382522
description="Component used to coordinate how records are extracted across stream slices and request pages when the state is empty or not provided.",
@@ -2621,9 +2605,7 @@ class AsyncRetriever(BaseModel):
26212605
)
26222606
download_extractor: Optional[
26232607
Union[DpathExtractor, CustomRecordExtractor, ResponseToFileExtractor]
2624-
] = Field(
2625-
None, description="Responsible for fetching the records from provided urls."
2626-
)
2608+
] = Field(None, description="Responsible for fetching the records from provided urls.")
26272609
creation_requester: Union[HttpRequester, CustomRequester] = Field(
26282610
...,
26292611
description="Requester component that describes how to prepare HTTP requests to send to the source API to create the async server-side job.",
@@ -2763,12 +2745,10 @@ class DynamicDeclarativeStream(BaseModel):
27632745
stream_template: DeclarativeStream = Field(
27642746
..., description="Reference to the stream template.", title="Stream Template"
27652747
)
2766-
components_resolver: Union[HttpComponentsResolver, ConfigComponentsResolver] = (
2767-
Field(
2768-
...,
2769-
description="Component resolve and populates stream templates with components values.",
2770-
title="Components Resolver",
2771-
)
2748+
components_resolver: Union[HttpComponentsResolver, ConfigComponentsResolver] = Field(
2749+
...,
2750+
description="Component resolve and populates stream templates with components values.",
2751+
title="Components Resolver",
27722752
)
27732753

27742754

airbyte_cdk/sources/declarative/resolvers/config_components_resolver.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -120,7 +120,9 @@ def prepare_streams() -> Iterable[List[Tuple[int, Any]]]:
120120
path = resolve_path(stream_config.configs_pointer)
121121
stream_configs_raw = dpath.get(dict(self.config), path, default=[])
122122
stream_configs = (
123-
list(stream_configs_raw) if isinstance(stream_configs_raw, list) else [stream_configs_raw]
123+
list(stream_configs_raw)
124+
if isinstance(stream_configs_raw, list)
125+
else [stream_configs_raw]
124126
)
125127

126128
if stream_config.default_values:

unit_tests/sources/declarative/resolvers/test_config_components_resolver.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,8 @@
33
#
44

55
import json
6-
from unittest.mock import MagicMock
76
from copy import deepcopy
7+
from unittest.mock import MagicMock
88

99
import pytest
1010

@@ -141,7 +141,9 @@ def to_configured_catalog(
141141

142142
# Manifest with a placeholder for custom stream config list override
143143
_MANIFEST_WITH_STREAM_CONFIGS_LIST = deepcopy(_MANIFEST)
144-
_MANIFEST_WITH_STREAM_CONFIGS_LIST["dynamic_streams"][0]["components_resolver"]["stream_config"] = [STREAM_CONFIG]
144+
_MANIFEST_WITH_STREAM_CONFIGS_LIST["dynamic_streams"][0]["components_resolver"]["stream_config"] = [
145+
STREAM_CONFIG
146+
]
145147

146148

147149
@pytest.mark.parametrize(

0 commit comments

Comments
 (0)