Skip to content
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

🐛 Source Chargebee, Delighted: disabled Incremental tests #28905

Merged
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
Expand Up @@ -34,5 +34,5 @@ COPY source_chargebee ./source_chargebee
ENV AIRBYTE_ENTRYPOINT "python /airbyte/integration_code/main.py"
ENTRYPOINT ["python", "/airbyte/integration_code/main.py"]

LABEL io.airbyte.version=0.2.3
LABEL io.airbyte.version=0.2.4
LABEL io.airbyte.name=airbyte/source-chargebee
2 changes: 1 addition & 1 deletion airbyte-integrations/connectors/source-chargebee/README.md
Expand Up @@ -80,4 +80,4 @@ You've checked out the repo, implemented a million dollar feature, and you're re
1. Bump the connector version in `Dockerfile` -- just increment the value of the `LABEL io.airbyte.version` appropriately (we use [SemVer](https://semver.org/)).
1. Create a Pull Request.
1. Pat yourself on the back for being an awesome contributor.
1. Someone from Airbyte will take a look at your PR and iterate with you to merge it into master.
1. Someone from Airbyte will take a look at your PR and iterate with you to merge it into master.
Expand Up @@ -49,19 +49,22 @@ acceptance_tests:
extra_records: yes
fail_on_extra_columns: false
incremental:
tests:
- config_path: "secrets/config.json"
timeout_seconds: 2400
configured_catalog_path: "integration_tests/configured_catalog.json"
future_state:
future_state_path: "integration_tests/future_state.json"
missing_streams:
- name: attached_item
bypass_reason: "This stream is Full-Refresh only"
- name: contact
bypass_reason: "This stream is Full-Refresh only"
- name: quote_line_group
bypass_reason: "This stream is Full-Refresh only"
# tests:
# - config_path: "secrets/config.json"
# timeout_seconds: 2400
# configured_catalog_path: "integration_tests/configured_catalog.json"
# future_state:
# future_state_path: "integration_tests/future_state.json"
# missing_streams:
# - name: attached_item
# bypass_reason: "This stream is Full-Refresh only"
# - name: contact
# bypass_reason: "This stream is Full-Refresh only"
# - name: quote_line_group
# bypass_reason: "This stream is Full-Refresh only"
bypass_reason: >
"Incrremental tests are disabled until CAT works with cursor data-types directly,
relatated slack thread: https://airbyte-globallogic.slack.com/archives/C02U9R3AF37/p1690810513681859"
full_refresh:
tests:
- config_path: "secrets/config.json"
Expand Down
Expand Up @@ -5,7 +5,7 @@ data:
connectorSubtype: api
connectorType: source
definitionId: 686473f1-76d9-4994-9cc7-9b13da46147c
dockerImageTag: 0.2.3
dockerImageTag: 0.2.4
dockerRepository: airbyte/source-chargebee
githubIssueLabel: source-chargebee
icon: chargebee.svg
Expand Down
2 changes: 1 addition & 1 deletion airbyte-integrations/connectors/source-chargebee/setup.py
Expand Up @@ -6,7 +6,7 @@
from setuptools import find_packages, setup

MAIN_REQUIREMENTS = [
"airbyte-cdk~=0.29",
"airbyte-cdk",
]

TEST_REQUIREMENTS = [
Expand Down
Expand Up @@ -23,11 +23,14 @@ acceptance_tests:
expect_records:
path: "integration_tests/expected_records.jsonl"
incremental:
tests:
- config_path: "secrets/config.json"
configured_catalog_path: "integration_tests/configured_catalog.json"
future_state:
future_state_path: "integration_tests/abnormal_state.json"
# tests:
# - config_path: "secrets/config.json"
# configured_catalog_path: "integration_tests/configured_catalog.json"
# future_state:
# future_state_path: "integration_tests/abnormal_state.json"
bypass_reason: >
"Incrremental tests are disabled until CAT works with cursor data-types directly,
relatated slack thread: https://airbyte-globallogic.slack.com/archives/C02U9R3AF37/p1690810513681859"
full_refresh:
tests:
- config_path: "secrets/config.json"
Expand Down
1 change: 1 addition & 0 deletions docs/integrations/sources/chargebee.md
Expand Up @@ -74,6 +74,7 @@ The Chargebee connector should not run into [Chargebee API](https://apidocs.char

| Version | Date | Pull Request | Subject |
|:--------|:-----------|:---------------------------------------------------------|:----------------------------------------------------------------------------------------------------|
| 0.2.4 | 2023-08-01 | [28905](https://github.com/airbytehq/airbyte/pull/28905) | Updated the connector to use latest CDK version |
| 0.2.3 | 2023-03-22 | [24370](https://github.com/airbytehq/airbyte/pull/24370) | Ignore 404 errors for `Contact` stream |
| 0.2.2 | 2023-02-17 | [21688](https://github.com/airbytehq/airbyte/pull/21688) | Migrate to CDK beta 0.29; fix schemas |
| 0.2.1 | 2023-02-17 | [23207](https://github.com/airbytehq/airbyte/pull/23207) | Edited stream schemas to get rid of unnecessary `enum` |
Expand Down