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.

Sign up for GitHub

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

Tdl 14962 implement tap tester tests #53

Closed
wants to merge 25 commits into from
Closed

Tdl 14962 implement tap tester tests #53

Show file tree
Hide file tree
Changes from 5 commits
Commits
Show all changes
25 commits
Select commit Hold shift + click to select a range
0cc64d2
Added new tep-tester test cases
prijendev Sep 10, 2021
305dfdc
Updated all_fields test case for track_events stream
prijendev Sep 13, 2021
eb4cb85
updated dev and test dependency for circleci
prijendev Sep 13, 2021
9285815
Updated start_date and added lookback_window assertion in bookmark test.
prijendev Sep 14, 2021
7978e80
Updated all_fields and automatic_fields test cases as per review comm…
prijendev Sep 15, 2021
d08f2c0
Updated as per review comments
prijendev Sep 30, 2021
45b22cb
revert back replicatio key change
prijendev Sep 30, 2021
7db0b87
Added back config file
prijendev Oct 4, 2021
939cfc3
Commented some part of code
prijendev Oct 13, 2021
751b2ef
Updated circleci config file
prijendev Oct 13, 2021
8fa6bf9
Removed f-string
savan-chovatiya Oct 18, 2021
5a547ef
Tdl 15671 full table sync record count (#60)
prijendev Oct 27, 2021
d182d23
Bug fix for using new bookmark for child streams (#61)
prijendev Oct 27, 2021
fcd1a19
Best practices (#62)
prijendev Oct 27, 2021
a3434e9
Error backoff (#63)
prijendev Oct 27, 2021
be58545
Tdl 14795 rmoved endpoints dict (#64)
prijendev Oct 27, 2021
30dd453
Tdl 6877 add backoff for conn reset error (#65)
prijendev Oct 27, 2021
09c14c8
Fix incremental streams as full table (#66)
prijendev Oct 27, 2021
1beac7f
Tdl 14950 correctly discover the custom fields (#67)
prijendev Oct 27, 2021
782b7bd
Tdl 14964 fix none type date parsing (#68)
prijendev Oct 27, 2021
e099234
Tdl 14945 fix sync of visitor stream (#69)
prijendev Oct 27, 2021
9dab3f1
Merge remote-tracking branch 'origin/TDL-15987-all-non-pk-changes' in…
prijendev Oct 27, 2021
87a577d
Changes updated
prijendev Oct 27, 2021
5ee8cb2
updated base file and bookmark test
prijendev Oct 27, 2021
5b04c0f
Updated bookmark test
prijendev Oct 27, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .circleci/config.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ jobs:
python3 -mvenv /usr/local/share/virtualenvs/tap-pendo
source /usr/local/share/virtualenvs/tap-pendo/bin/activate
pip install -U pip setuptools
pip install .[dev]
pip install .[test]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@prijendev Does this indicate that we no more consider the modules added as part of the 'dev' list?

Copy link
Contributor Author

@prijendev prijendev Oct 11, 2021 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, because earlier circleci was failing due to ipdb dependency listed under 'dev' list. So, as per suggestion given in link split down 'dev' list dependency into 'test' and 'dev' list.

- run:
name: 'JSON Validator'
command: |
Expand Down
6 changes: 4 additions & 2 deletions setup.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,9 +17,11 @@
'ijson==3.1.4',
],
extras_require={
'test': [
'pylint==2.5.3'
],
'dev': [
'ipdb==0.11',
'pylint==2.5.3',
'ipdb==0.11'
]
},
entry_points="""
Expand Down
Loading