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

🎉 New source: Timely #13292 #14335

Merged
merged 7 commits into from
Jul 15, 2022
Merged

Conversation

harshithmullapudi
Copy link
Contributor

Closing #13292.

Created this new PR as I didn't have permission to push to the other repo

@codecov
Copy link

codecov bot commented Jul 1, 2022

Codecov Report

❗ No coverage uploaded for pull request base (master@a97216f). Click here to learn what that means.
The diff coverage is n/a.

❗ Current head 8ce0468 differs from pull request most recent head f6fefbb. Consider uploading reports for the commit f6fefbb to get more accurate results

@@            Coverage Diff            @@
##             master   #14335   +/-   ##
=========================================
  Coverage          ?   79.03%           
=========================================
  Files             ?        2           
  Lines             ?       62           
  Branches          ?        0           
=========================================
  Hits              ?       49           
  Misses            ?       13           
  Partials          ?        0           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a97216f...f6fefbb. Read the comment docs.

@harshithmullapudi
Copy link
Contributor Author

harshithmullapudi commented Jul 13, 2022

/test connector=connectors/source-timely

🕑 connectors/source-timely https://github.com/airbytehq/airbyte/actions/runs/2662467053
❌ connectors/source-timely https://github.com/airbytehq/airbyte/actions/runs/2662467053
🐛 https://gradle.com/s/pz7wqvxjau65g

Build Failed

Test summary info:

=========================== short test summary info ============================
ERROR test_core.py::TestSpec::test_config_match_spec[inputs0] - FileNotFoundE...
ERROR test_core.py::TestConnection::test_check[inputs0] - FileNotFoundError: ...
ERROR test_core.py::TestDiscovery::test_discover[inputs0] - FileNotFoundError...
ERROR test_core.py::TestDiscovery::test_defined_cursors_exist_in_schema[inputs0]
ERROR test_core.py::TestDiscovery::test_defined_refs_exist_in_schema[inputs0]
ERROR test_core.py::TestDiscovery::test_defined_keyword_exist_in_schema[inputs0-allOf]
ERROR test_core.py::TestDiscovery::test_defined_keyword_exist_in_schema[inputs0-not]
ERROR test_core.py::TestDiscovery::test_primary_keys_exist_in_schema[inputs0]
ERROR test_core.py::TestBasicRead::test_read[inputs0] - FileNotFoundError: [E...
ERROR test_core.py::TestBasicRead::test_airbyte_trace_message_on_failure[inputs0]
ERROR test_full_refresh.py::TestFullRefresh::test_sequential_reads[inputs0]
SKIPPED [1] ../usr/local/lib/python3.9/site-packages/source_acceptance_test/plugin.py:56: Skipping TestIncremental.test_two_sequential_reads because not found in the config
=================== 10 passed, 1 skipped, 11 errors in 8.05s ===================

@harshithmullapudi
Copy link
Contributor Author

harshithmullapudi commented Jul 13, 2022

/test connector=connectors/source-timely

🕑 connectors/source-timely https://github.com/airbytehq/airbyte/actions/runs/2662546892
❌ connectors/source-timely https://github.com/airbytehq/airbyte/actions/runs/2662546892
🐛 https://gradle.com/s/f2h35cgm66gly

Build Failed

Test summary info:

=========================== short test summary info ============================
ERROR test_core.py::TestSpec::test_config_match_spec[inputs0] - FileNotFoundE...
ERROR test_core.py::TestConnection::test_check[inputs0] - FileNotFoundError: ...
ERROR test_core.py::TestDiscovery::test_discover[inputs0] - FileNotFoundError...
ERROR test_core.py::TestDiscovery::test_defined_cursors_exist_in_schema[inputs0]
ERROR test_core.py::TestDiscovery::test_defined_refs_exist_in_schema[inputs0]
ERROR test_core.py::TestDiscovery::test_defined_keyword_exist_in_schema[inputs0-allOf]
ERROR test_core.py::TestDiscovery::test_defined_keyword_exist_in_schema[inputs0-not]
ERROR test_core.py::TestDiscovery::test_primary_keys_exist_in_schema[inputs0]
ERROR test_core.py::TestBasicRead::test_read[inputs0] - FileNotFoundError: [E...
ERROR test_core.py::TestBasicRead::test_airbyte_trace_message_on_failure[inputs0]
ERROR test_full_refresh.py::TestFullRefresh::test_sequential_reads[inputs0]
SKIPPED [1] ../usr/local/lib/python3.9/site-packages/source_acceptance_test/plugin.py:56: Skipping TestIncremental.test_two_sequential_reads because not found in the config
=================== 10 passed, 1 skipped, 11 errors in 7.92s ===================

@harshithmullapudi
Copy link
Contributor Author

harshithmullapudi commented Jul 15, 2022

/test connector=connectors/source-timely

🕑 connectors/source-timely https://github.com/airbytehq/airbyte/actions/runs/2674909837

@harshithmullapudi
Copy link
Contributor Author

harshithmullapudi commented Jul 15, 2022

/test connector=connectors/source-timely

🕑 connectors/source-timely https://github.com/airbytehq/airbyte/actions/runs/2674935000

@harshithmullapudi
Copy link
Contributor Author

harshithmullapudi commented Jul 15, 2022

/test connector=connectors/source-timely

🕑 connectors/source-timely https://github.com/airbytehq/airbyte/actions/runs/2674961870
✅ connectors/source-timely https://github.com/airbytehq/airbyte/actions/runs/2674961870
Python tests coverage:

Name                                                 Stmts   Miss  Cover
------------------------------------------------------------------------
source_acceptance_test/utils/__init__.py                 6      0   100%
source_acceptance_test/tests/__init__.py                 4      0   100%
source_acceptance_test/__init__.py                       2      0   100%
source_acceptance_test/tests/test_full_refresh.py       52      2    96%
source_acceptance_test/utils/asserts.py                 37      2    95%
source_acceptance_test/config.py                        77      6    92%
source_acceptance_test/utils/json_schema_helper.py     105     13    88%
source_acceptance_test/tests/test_incremental.py       121     25    79%
source_acceptance_test/utils/common.py                  80     17    79%
source_acceptance_test/tests/test_core.py              294    106    64%
source_acceptance_test/utils/compare.py                 62     23    63%
source_acceptance_test/base.py                          10      4    60%
source_acceptance_test/utils/connector_runner.py       110     48    56%
------------------------------------------------------------------------
TOTAL                                                  960    246    74%
Name                        Stmts   Miss  Cover
-----------------------------------------------
source_timely/__init__.py       2      0   100%
source_timely/source.py        60     13    78%
-----------------------------------------------
TOTAL                          62     13    79%

Build Passed

Test summary info:

=========================== short test summary info ============================
SKIPPED [1] ../usr/local/lib/python3.9/site-packages/source_acceptance_test/plugin.py:56: Skipping TestIncremental.test_two_sequential_reads because not found in the config
======================== 21 passed, 1 skipped in 18.75s ========================

@github-actions github-actions bot added the area/documentation Improvements or additions to documentation label Jul 15, 2022
@harshithmullapudi harshithmullapudi temporarily deployed to more-secrets July 15, 2022 06:28 Inactive
@harshithmullapudi
Copy link
Contributor Author

harshithmullapudi commented Jul 15, 2022

/publish connector=connectors/source-timely

🕑 Publishing the following connectors:
connectors/source-timely
https://github.com/airbytehq/airbyte/actions/runs/2675076535


Connector Did it publish? Were definitions generated?
connectors/source-timely

if you have connectors that successfully published but failed definition generation, follow step 4 here ▶️

@harshithmullapudi harshithmullapudi temporarily deployed to more-secrets July 15, 2022 06:34 Inactive
@octavia-squidington-iii octavia-squidington-iii temporarily deployed to more-secrets July 15, 2022 06:50 Inactive
@harshithmullapudi harshithmullapudi merged commit bf9bfd1 into master Jul 15, 2022
@harshithmullapudi harshithmullapudi deleted the harshith/test-pr-13292 branch July 15, 2022 07:35
jsrcodes added a commit to jsrcodes/airbyte that referenced this pull request Jul 15, 2022
…rbytehq-master

* 'master' of https://github.com/airbytehq/airbyte: (1141 commits)
  pass USE_STREAM_CAPABLE_STATE env var to containers/deployments (airbytehq#14737)
  Bump mqtt connector (airbytehq#14648)
  Add error code to ManualOperationResult (airbytehq#14657)
  Bump elasticsearch version (airbytehq#14640)
  Ryan/sync oracle version number (airbytehq#14736)
  Fixed linter issue with add_fields.py comments (airbytehq#14742)
  🎉Redshift, Databricks, Snowflake, S3 Destinations: Make S3 output filename configurable (airbytehq#14494)
  🐛Source-mssql: aligned regular and cdc syncs and its datatype tests (airbytehq#14379)
  🎉 Source Amazon Seller Partner: Add new streams (airbytehq#13604)
  bump source-file-secure (airbytehq#14704)
  🎉 New source: Timely airbytehq#13292 (airbytehq#14335)
  🪟🔧 Refactor feature service (airbytehq#14559)
  [low code cdk] add a transformation for adding fields into an outgoing record (airbytehq#14638)
  Bump destination-postgres to 0.3.21 (airbytehq#14479)
  Remove `additionalProperties: false` from JDBC destination connectors (airbytehq#14618)
  🎉 Source Notion: add OAuth authorization for source-notion connector (airbytehq#14706)
  Use the configuration diff calculation in the update endpoint (airbytehq#14626)
  🪟 🐛 Fix input validation on blur and cleanup signup error handling (airbytehq#14724)
  lower sleep after wait for successful job (airbytehq#14725)
  Add configuration diff (airbytehq#14603)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/connectors Connector related issues area/documentation Improvements or additions to documentation connectors/source/timely waiting-for-cred Waiting for CI credentials
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants