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 Sendgrid: fixed expected records #19819

Merged
merged 2 commits into from
Nov 29, 2022

Conversation

darynaishchenko
Copy link
Collaborator

@darynaishchenko darynaishchenko commented Nov 28, 2022

fixed field 'created' for global_suppressions and bounces streams

@darynaishchenko
Copy link
Collaborator Author

darynaishchenko commented Nov 28, 2022

/test connector=connectors/source-sendgrid

🕑 connectors/source-sendgrid https://github.com/airbytehq/airbyte/actions/runs/3563466798
❌ connectors/source-sendgrid https://github.com/airbytehq/airbyte/actions/runs/3563466798
🐛 https://gradle.com/s/fugs2jjak5nja

Build Failed

Test summary info:

=========================== short test summary info ============================
FAILED test_core.py::TestBasicRead::test_read[inputs0] - Failed: Stream bounc...
================== 1 failed, 32 passed, 31 warnings in 45.75s ==================

@darynaishchenko
Copy link
Collaborator Author

darynaishchenko commented Nov 28, 2022

/test connector=connectors/source-sendgrid

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

Name                          Stmts   Miss  Cover
-------------------------------------------------
source_sendgrid/__init__.py       2      0   100%
source_sendgrid/source.py        26      3    88%
source_sendgrid/streams.py      171     22    87%
-------------------------------------------------
TOTAL                           199     25    87%
	 Name                                                 Stmts   Miss  Cover   Missing
	 ----------------------------------------------------------------------------------
	 source_acceptance_test/base.py                          12      4    67%   16-19
	 source_acceptance_test/config.py                       139      5    96%   87, 93, 235, 239-240
	 source_acceptance_test/conftest.py                     196     92    53%   35, 41-43, 48, 54, 60, 66, 72-74, 93, 98-100, 106-108, 114-115, 120-121, 126, 132, 141-150, 156-161, 176, 200, 231, 237, 243-248, 256-261, 269-282, 287-293, 300-311, 318-334
	 source_acceptance_test/plugin.py                        69     25    64%   22-23, 31, 36, 120-140, 144-148
	 source_acceptance_test/tests/test_core.py              398    111    72%   53, 58, 87-95, 100-107, 111-112, 116-117, 299, 337-354, 363-371, 375-380, 386, 419-424, 462-469, 512-514, 517, 582-590, 602-605, 610, 666-667, 673, 676, 712-722, 735-760
	 source_acceptance_test/tests/test_incremental.py       158     14    91%   52-59, 64-77, 240
	 source_acceptance_test/utils/asserts.py                 37      2    95%   57-58
	 source_acceptance_test/utils/common.py                  94     10    89%   16-17, 32-38, 72, 75
	 source_acceptance_test/utils/compare.py                 62     23    63%   21-51, 68, 97-99
	 source_acceptance_test/utils/connector_runner.py       112     50    55%   23-26, 32, 36, 39-68, 71-73, 76-78, 81-83, 86-88, 91-93, 96-114, 148-150
	 source_acceptance_test/utils/json_schema_helper.py     107     13    88%   30-31, 38, 41, 65-68, 96, 120, 192-194
	 ----------------------------------------------------------------------------------
	 TOTAL                                                 1563    349    78%

Build Passed

Test summary info:

All Passed

@darynaishchenko darynaishchenko changed the title Source Sendgrid: fixed expected records for global_suppressions stream Source Sendgrid: fixed expected records Nov 28, 2022
@darynaishchenko darynaishchenko merged commit 5b98584 into master Nov 29, 2022
@darynaishchenko darynaishchenko deleted the daryna/sendgrid/fix-expected-records branch November 29, 2022 10:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants