-
Notifications
You must be signed in to change notification settings - Fork 4.6k
✨ Source Paypal Transactions: Silver Certification #34510
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
Merged
Merged
Changes from 36 commits
Commits
Show all changes
40 commits
Select commit
Hold shift + click to select a range
d82799d
Start Silver certification. Add new streams
Jgerardopine d1b8976
Add Search invoices stream. Parametrize some variables.
Jgerardopine 8a0a6bb
Add new stream - Product catalog -. Add error handling for transactio…
Jgerardopine 57a62e9
New Stream - list disputes. Add data generator to test data.
Jgerardopine ca8e027
Add new stream and substream - show product details.
Jgerardopine 4d35714
Add new streams. Add seeding to some tests. Change Metadata version.
Jgerardopine b2a87c7
Add tests. Organize and clean some code. Commented out show_product_d…
Jgerardopine 70e30bc
Add coveragerc. Documentation md file. Change version and ql level
Jgerardopine 6d2a730
Update CDK version
Jgerardopine 9fb2466
Start Silver certification. Add new streams
Jgerardopine d66a2b9
Add Search invoices stream. Parametrize some variables.
Jgerardopine 9c430da
Add new stream - Product catalog -. Add error handling for transactio…
Jgerardopine 22c4980
New Stream - list disputes. Add data generator to test data.
Jgerardopine 85913ec
Add new stream and substream - show product details.
Jgerardopine 42f7138
Add new streams. Add seeding to some tests. Change Metadata version.
Jgerardopine 7c7ff74
Add tests. Organize and clean some code. Commented out show_product_d…
Jgerardopine 6b6b9e8
Add coveragerc. Documentation md file. Change version and ql level
Jgerardopine 2fa110b
Update CDK version
Jgerardopine 87442ed
Merge branch 'jgpines/paypal-trans-silver-cert' of https://github.com…
Jgerardopine f7f5028
Fix format (airbyte-ci)
Jgerardopine 710ad0f
Comment out Sandbox tests. Leave only configg.json for CI tests
Jgerardopine 106c760
Add empty streams to pass CI tests using GSM sandbox credentials
Jgerardopine 341abc3
Fix format JS. Validate records and streams
Jgerardopine 93e7687
Fix comments. Add test and test files
Jgerardopine 14ef96a
Update airbyte-integrations/connectors/source-paypal-transaction/CHAN…
Jgerardopine 3c944ed
Fix conflicts
Jgerardopine 6ad0d11
Fix merge conflicts Poetry. Update versions in poetry.
Jgerardopine 5ce5dbd
Merge branch 'master' into jgpines/paypal-trans-silver-cert
Jgerardopine dd2987c
regenerate poetry.lock with poetry 1.6.1
alafanechere 3a1ae80
Merge branch 'master' into jgpines/paypal-trans-silver-cert
Jgerardopine 7494eca
Update airbyte-integrations/connectors/source-paypal-transaction/pypr…
Jgerardopine f413325
Add end date to spec(optional). Change python to poetry in ReadME file
Jgerardopine c516725
Add pattern to end_date
Jgerardopine ee42253
Change congig[] to config.get and add dat-time
Jgerardopine a9bc636
Merge branch 'master' into jgpines/paypal-trans-silver-cert
Jgerardopine 3891edd
Update airbyte-integrations/connectors/source-paypal-transaction/READ…
Jgerardopine d6f252f
Update invoices.py
Jgerardopine c197b07
Update payments_generator.py
Jgerardopine 9b8276d
Update paypal_transaction_generator.py
Jgerardopine 85e5080
Fix Python format (airbyte-ci)
Jgerardopine File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
3 changes: 3 additions & 0 deletions
3
airbyte-integrations/connectors/source-paypal-transaction/.coveragerc
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,3 @@ | ||
[run] | ||
omit = | ||
source_paypal_transaction/run.py |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,54 +1,89 @@ | ||
# Paypal-Transaction source connector | ||
|
||
|
||
This is the repository for the Paypal-Transaction source connector, written in Python. | ||
For information about how to use this connector within Airbyte, see [the documentation](https://docs.airbyte.com/integrations/sources/paypal-transaction). | ||
|
||
## Local development | ||
|
||
### Prerequisites | ||
* Python (~=3.9) | ||
* Poetry (~=1.7) - installation instructions [here](https://python-poetry.org/docs/#installation) | ||
|
||
#### Prerequisites | ||
* Python (~=3.9) | ||
* Poetry (~=1.7) - installation instructions [here](https://python-poetry.org/docs/#installation) | ||
* Paypal Client ID and Client Secret | ||
* If you are going to use the data generator scripts you need to setup yourPaypal Sandbox and a Buyer user in your sandbox, to simulate the data. YOu cna get that information in the [Apps & Credentials page](https://developer.paypal.com/dashboard/applications/live). | ||
* Buyer Username | ||
* Buyer Password | ||
* Payer ID (Account ID) | ||
|
||
### Installing the connector | ||
|
||
From this connector directory, run: | ||
```bash | ||
poetry install --with dev | ||
``` | ||
|
||
|
||
### Create credentials | ||
|
||
**If you are a community contributor**, follow the instructions in the [documentation](https://docs.airbyte.com/integrations/sources/paypal-transaction) | ||
to generate the necessary credentials. Then create a file `secrets/config.json` conforming to the `source_paypal_transaction/spec.yaml` file. | ||
Note that any directory named `secrets` is gitignored across the entire Airbyte repo, so there is no danger of accidentally checking in sensitive information. | ||
See `sample_files/sample_config.json` for a sample config file. | ||
|
||
* You must have created your credentials under the `secrets/` folder | ||
* For the read command, you can create separate catalogs to test the streams individually. All catalogs are under the folder `integration_tests`. Select the one you want to test with the read command. | ||
|
||
|
||
### Locally running the connector | ||
|
||
``` | ||
poetry run source-paypal-transaction spec | ||
poetry run source-paypal-transaction check --config secrets/config.json | ||
poetry run source-paypal-transaction discover --config secrets/config.json | ||
poetry run source-paypal-transaction read --config secrets/config.json --catalog sample_files/configured_catalog.json | ||
#Example with list_payments catalog and the debug flag | ||
poetry run source-paypal-transaction read --config secrets/config.json --catalog integration_tests/configured_catalog_list_payments.json --debug | ||
``` | ||
|
||
### Running unit tests | ||
To run unit tests locally, from the connector directory run: | ||
|
||
``` | ||
poetry run pytest unit_tests | ||
``` | ||
|
||
### Building the docker image | ||
|
||
1. Install [`airbyte-ci`](https://github.com/airbytehq/airbyte/blob/master/airbyte-ci/connectors/pipelines/README.md) | ||
2. Run the following command to build the docker image: | ||
|
||
|
||
### Installing the connector | ||
From this connector directory, run: | ||
```bash | ||
airbyte-ci connectors --name=source-paypal-transaction build | ||
poetry install --with dev | ||
``` | ||
|
||
##### Customizing our build process | ||
When contributing on our connector you might need to customize the build process to add a system dependency or set an env var. | ||
You can customize our build process by adding a `build_customization.py` module to your connector. | ||
This module should contain a `pre_connector_install` and `post_connector_install` async function that will mutate the base image and the connector container respectively. | ||
It will be imported at runtime by our build process and the functions will be called if they exist. | ||
|
||
Here is an example of a `build_customization.py` module: | ||
```python | ||
from __future__ import annotations | ||
|
||
from typing import TYPE_CHECKING | ||
|
||
if TYPE_CHECKING: | ||
# Feel free to check the dagger documentation for more information on the Container object and its methods. | ||
# https://dagger-io.readthedocs.io/en/sdk-python-v0.6.4/ | ||
from dagger import Container | ||
``` | ||
|
||
An image will be available on your host with the tag `airbyte/source-paypal-transaction:dev`. | ||
|
||
|
||
|
||
### Running as a docker container | ||
Then run any of the connector commands as follows: | ||
``` | ||
|
@@ -58,23 +93,110 @@ docker run --rm -v $(pwd)/secrets:/secrets airbyte/source-paypal-transaction:dev | |
docker run --rm -v $(pwd)/secrets:/secrets -v $(pwd)/integration_tests:/integration_tests airbyte/source-paypal-transaction:dev read --config /secrets/config.json --catalog /integration_tests/configured_catalog.json | ||
``` | ||
|
||
|
||
### Running our CI test suite | ||
|
||
You can run our full test suite locally using [`airbyte-ci`](https://github.com/airbytehq/airbyte/blob/master/airbyte-ci/connectors/pipelines/README.md): | ||
|
||
```bash | ||
airbyte-ci connectors --name=source-paypal-transaction test | ||
``` | ||
|
||
If you are testing locally, you can use your local credentials (config.json file) by using `--use-local-secrets` | ||
|
||
```bash | ||
airbyte-ci connectors --name source-paypal-transaction --use-local-secrets test | ||
``` | ||
|
||
### Customizing acceptance Tests | ||
Customize `acceptance-test-config.yml` file to configure acceptance tests. See [Connector Acceptance Tests](https://docs.airbyte.com/connector-development/testing-connectors/connector-acceptance-tests-reference) for more information. | ||
If your connector requires to create or destroy resources for use during acceptance tests create fixtures for it and place them inside integration_tests/acceptance.py. | ||
|
||
### Dependency Management | ||
|
||
## Running Unit tests locally | ||
|
||
To run unit tests locally, form the root `source_paypal_transaction` directory run: | ||
|
||
```bash | ||
python -m pytest unit_test | ||
``` | ||
|
||
## Test changes in the sandbox | ||
|
||
If you have a [Paypal Sandbox](https://developer.paypal.com/tools/sandbox/accounts/) you will be able to use some APIs to create new data and test how data is being created in your destinaiton and choose the best syn strategy that suits better your use case. | ||
Some endpoints will require special permissions on the sandbox to update and change some values. | ||
|
||
In the `bin` folder you will find several data generator scripts: | ||
|
||
* **disputes_generator.py:** | ||
* Update dispute: Uses the _PATCH_ method of the `https://api-m.paypal.com/v1/customer/disputes/{dispute_id}` endpoint. You need the ID and create a payload to pass it as an argument. See more information [here](https://developer.paypal.com/docs/api/customer-disputes/v1/#disputes_patch). | ||
|
||
```bash | ||
python disputes_generator.py update DISPUTE_ID ''[{"op": "replace", "path": "/reason", "value": "The new reason"}]' | ||
``` | ||
|
||
* Update Evidence status: Uses the _POST_ method of the `https://api-m.paypal.com/v1/customer/disputes/{dispute_id}/require-evidence` endpoint. You need the ID and select an option to pass it as an argument. See more information [here](https://developer.paypal.com/docs/api/customer-disputes/v1/#disputes_require-evidence) | ||
```bash | ||
python update_dispute.py require-evidence DISPUTE_ID SELLER_EVIDENCE | ||
``` | ||
|
||
* **invoices.py:** | ||
* Create draft invoice: Uses the _POST_ method of the `https://api-m.sandbox.paypal.com/v2/invoicing/invoices` endpoint. It will automatically generate an invoice (no need to pass any parameters). See more information [here](https://developer.paypal.com/docs/api/invoicing/v2/#invoices_create). | ||
|
||
```bash | ||
python invoices.py create_draft | ||
``` | ||
|
||
* Send a Draft Invoice: Uses the _POST_ method of the `https://api-m.sandbox.paypal.com/v2/invoicing/invoices/{invoice_id}/send` endpoint. You need the Invoice ID, a subject and a note (just to have something to update) and an email as an argument. See more information [here](https://developer.paypal.com/docs/api/invoicing/v2/#invoices_send) | ||
```bash | ||
python invoices.py send_draft --invoice_id "INV2-XXXX-XXXX-XXXX-XXXX" --subject "Your Invoice Subject" --note "Your custom note" --additional_recipients [email protected] | ||
``` | ||
|
||
* **payments_generator.py:** | ||
* Partially update payment: Uses the _PATCH_ method of the `https://api-m.paypal.com/v1/payments/payment/{payment_id}` endpoint. You need the payment ID and a payload with new values. (no need to pass any parameters). See more information [here](https://developer.paypal.com/docs/api/invoicing/v2/#invoices_create). | ||
|
||
```bash | ||
python script_name.py update PAYMENT_ID '[{"op": "replace", "path": "/transactions/0/amount", "value": {"total": "50.00", "currency": "USD"}}]' | ||
``` | ||
|
||
* **paypal_transaction_generator.py:** | ||
Make sure you have the `buyer_username`, `buyer_password` and `payer_id` in your config file. You can get the sample configuratin in the `sample_config.json`. | ||
|
||
* Generate transactions: This uses Selenium, so you will be prompted to your account to simulate the complete transaction flow. You can add a number at the end of the command to do more than one transaction. By default the script runs 3 transactions. | ||
|
||
**NOTE: Be midnfu of the number of transactions, as it will be interacting with your machine, and you may not be able to use it while creating the transactions** | ||
|
||
```bash | ||
python paypal_transaction_generator.py [NUMBER_OF_DESIRED_TRANSACTIONS] | ||
``` | ||
|
||
* **product_catalog.py:** | ||
* Create a product: Uses the _POST_ method of the `https://api-m.sandbox.paypal.com/v1/catalogs/products` endpoint. You need to add the description and the category in the command line. For the proper category see more information [here](https://developer.paypal.com/docs/api/catalog-products/v1/#products_create). | ||
|
||
```bash | ||
python product_catalog.py --action create --description "YOUR DESCRIPTION" --category PAYPAL_CATEGORY | ||
``` | ||
|
||
* Update a product: Uses the _PATCH_ method of the `https://developer.paypal.com/docs/api/catalog-products/v1/#products_patch` endpoint. You need the product ID, a description and the Category as an argument. See more information [here](https://developer.paypal.com/docs/api/catalog-products/v1/#products_patch) | ||
```bash | ||
python product_catalog.py --action update --product_id PRODUCT_ID --update_payload '[{"op": "replace", "path": "/description", "value": "My Update. Does it changes it?"}]' | ||
``` | ||
|
||
## Dependency Management | ||
All of your dependencies should go in `setup.py`, NOT `requirements.txt`. The requirements file is only used to connect internal Airbyte dependencies in the monorepo for local development. | ||
We split dependencies between two groups, dependencies that are: | ||
* required for your connector to work need to go to `MAIN_REQUIREMENTS` list. | ||
* required for the testing need to go to `TEST_REQUIREMENTS` list | ||
|
||
|
||
All of your dependencies should be managed via Poetry. | ||
|
||
To add a new dependency, run: | ||
```bash | ||
poetry add <package-name> | ||
``` | ||
|
||
|
||
Please commit the changes to `pyproject.toml` and `poetry.lock` files. | ||
|
||
## Publishing a new version of the connector | ||
|
81 changes: 48 additions & 33 deletions
81
airbyte-integrations/connectors/source-paypal-transaction/acceptance-test-config.yml
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,69 +1,84 @@ | ||
# See [Connector Acceptance Tests](https://docs.airbyte.com/connector-development/testing-connectors/connector-acceptance-tests-reference) | ||
# for more information about how to configure these tests | ||
# Make sure the paths you have in each path matches with your data. | ||
#For multiple env testing, you can duplicate the tests and change the path to the proper credentials file | ||
Jgerardopine marked this conversation as resolved.
Show resolved
Hide resolved
|
||
connector_image: airbyte/source-paypal-transaction:dev | ||
test_strictness_level: high | ||
acceptance_tests: | ||
spec: | ||
tests: | ||
#Test with Prod credentials (Make sure you purt the right ones) | ||
- spec_path: "source_paypal_transaction/spec.yaml" | ||
config_path: secrets/config_oauth.json | ||
config_path: secrets/config.json | ||
backward_compatibility_tests_config: | ||
disable_for_version: "0.1.13" | ||
connection: | ||
tests: | ||
- config_path: secrets/config_oauth.json | ||
#Test With Prod Credentials | ||
- config_path: secrets/config.json | ||
status: succeed | ||
- config_path: secrets/config_oauth_sandbox.json | ||
status: succeed | ||
- config_path: integration_tests/invalid_config.json | ||
status: failed | ||
- config_path: integration_tests/invalid_config_oauth.json | ||
#Test with Invalid Credentials | ||
- config_path: integration_tests/sample_files/invalid_config.json | ||
status: failed | ||
#Test with Sandbox Credentials | ||
# - config_path: secrets/config_sandbox.json | ||
# status: succeed | ||
discovery: | ||
tests: | ||
- config_path: secrets/config_oauth.json | ||
- config_path: secrets/config.json | ||
# - config_path: secrets/config_sandbox.json | ||
backward_compatibility_tests_config: | ||
disable_for_version: "2.0.0" # Change in cursor field for transactions stream | ||
basic_read: | ||
tests: | ||
- config_path: secrets/config_oauth.json | ||
ignored_fields: | ||
balances: | ||
- name: last_refresh_time | ||
bypass_reason: "field changes during every read" | ||
#Test Prod Environment - Uncomment and change according to your prod setup | ||
#Change the expected records, remember to align them with the timeframe you have selected | ||
#Do not select streams that take more than 5 mins to load data as that can lead to timeouts | ||
#You can comment the lines if you are sure you have data for the below streams. | ||
- config_path: secrets/config.json | ||
# - config_path: secrets/config_sandbox.json | ||
empty_streams: | ||
- name: transactions | ||
bypass_reason: "can not populate" | ||
timeout_seconds: 1200 | ||
expect_records: | ||
path: "integration_tests/expected_records.jsonl" | ||
extra_fields: no | ||
exact_order: no | ||
extra_records: yes | ||
- config_path: secrets/config_oauth_sandbox.json | ||
- name: show_product_details | ||
bypass_reason: "Products may not exist" | ||
- name: list_products | ||
bypass_reason: "Product List may be too big causing timeout errors" | ||
- name: search_invoices | ||
bypass_reason: "Order makes the diff fail." | ||
#Have to add for testing PR CI. | ||
- name: list_disputes | ||
bypass_reason: "Disputes may not exist." | ||
ignored_fields: | ||
balances: | ||
- name: last_refresh_time | ||
bypass_reason: "field changes during every read" | ||
timeout_seconds: 1200 | ||
list_products: | ||
- name: description | ||
bypass_reason: "Sometimes it is not contained in the response" | ||
timeout_seconds: 3200 | ||
expect_records: | ||
path: "integration_tests/expected_records_sandbox.jsonl" | ||
extra_fields: no | ||
exact_order: no | ||
extra_records: yes | ||
fail_on_extra_columns: false | ||
path: "integration_tests/sample_files/expected_records_sandbox.jsonl" | ||
#path: "integration_tests/sample_files/expected_records.jsonl" | ||
extra_fields: yes | ||
exact_order: yes | ||
extra_records: no | ||
fail_on_extra_columns: False | ||
incremental: | ||
tests: | ||
- config_path: secrets/config_oauth.json | ||
configured_catalog_path: integration_tests/configured_catalog.json | ||
- config_path: secrets/config.json | ||
# - config_path: secrets/config_sandbox.json | ||
configured_catalog_path: integration_tests/incremental_catalog.json | ||
future_state: | ||
future_state_path: integration_tests/abnormal_state.json | ||
future_state_path: integration_tests/sample_files/abnormal_state.json | ||
skip_comprehensive_incremental_tests: true | ||
full_refresh: | ||
tests: | ||
- config_path: secrets/config_oauth.json | ||
- config_path: secrets/config.json | ||
# - config_path: secrets/config_sandbox.json | ||
configured_catalog_path: integration_tests/full_refresh_catalog.json | ||
ignored_fields: | ||
balances: | ||
- name: last_refresh_time | ||
bypass_reason: "field changes during every read" | ||
configured_catalog_path: integration_tests/configured_catalog.json | ||
list_products: | ||
- name: description | ||
bypass_reason: "Sometimes it is not contained in the response" |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.