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

Jump to bottom

add e2e testing #2

Merged
merged 39 commits into from
Aug 30, 2023
Merged

add e2e testing #2

merged 39 commits into from
Aug 30, 2023

Conversation

mdtanrikulu
Copy link
Member

@mdtanrikulu mdtanrikulu commented Jul 3, 2023 •
edited
Loading

note: struggled compiling contracts under windows CI, gonna take it as a separate task later.

@mdtanrikulu
add e2e testing
db5ad59
@mdtanrikulu
update worker env
be79797
@mdtanrikulu
update server test
08d3a98
@mdtanrikulu
update readme
d9d9cf5
@mdtanrikulu
seperate workflows
4665702
@mdtanrikulu
update test workflow
bd548f8
@mdtanrikulu
update ci names
374ae00
@mdtanrikulu
update workflows
1dd418a
@mdtanrikulu
lint
e93c697
@mdtanrikulu
update workflow
61be9e2
@mdtanrikulu
temp remove nohup
715bc53
@mdtanrikulu
setup env vars
5f23531
@mdtanrikulu
update test workflow
ee1c809
@mdtanrikulu
update test workflow
4f1c345
@mdtanrikulu
hardcode batch gateway urls
02d2cc3
@mdtanrikulu
add nohup back
679e22e
@mdtanrikulu
increase wait-on timeout
f8f32b2
@mdtanrikulu
use http on wait-on
22e0ee7
@mdtanrikulu
add DoH gateway url as env
c972683
@mdtanrikulu
increase timeout for wait-on
2448e95
@mdtanrikulu
use node < v18.16 due to ada-url parser bug
93422d3
@mdtanrikulu
update node url
3f6ee62
@mdtanrikulu
bump setup-node action to v3
807d332
@mdtanrikulu
add different nohup setup for windows
531cfce
@mdtanrikulu
disable hh telemetry and fix windows bg task
3a8673a
@mdtanrikulu
move telemetry env into pwsh
a33d348
@mdtanrikulu
disable nohup on windows to debug
618f4f2
@mdtanrikulu
revert, incr. timeout
919b67f
@mdtanrikulu
another win try
f548f30
@mdtanrikulu
increase yarn network-timeout
e838b75
@mdtanrikulu mdtanrikulu requested a review from Arachnid July 5, 2023 15:30
Arachnid
Arachnid approved these changes Aug 7, 2023
View reviewed changes
Copy link
Member

@Arachnid Arachnid left a comment

Choose a reason for hiding this comment

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

It's a shame that you had to implement CCIP-read yourself here, but aside from one minor nit, LGTM.

tests/e2e.test.ts Outdated Show resolved Hide resolved
@mdtanrikulu mdtanrikulu merged commit 5049566 into main Aug 30, 2023
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Arachnid Arachnid Arachnid approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mdtanrikulu @Arachnid