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 status-go tests #13376

Merged
merged 1 commit into from
May 25, 2022
Merged

Add status-go tests #13376

merged 1 commit into from
May 25, 2022

Conversation

cammellos
Copy link
Member

@cammellos cammellos commented May 19, 2022 •
edited

Fixes: #13335
status: ready

@status-github-bot status-github-bot bot added this to REVIEW in Pipeline for QA May 19, 2022
@status-im-auto
Copy link
Member

status-im-auto commented May 19, 2022 •
edited

Jenkins Builds

Click to see older builds (45)
❔ Commit #️⃣ Finished (UTC) Duration Platform Result
❌ 65be73f #1 2022-05-19 16:05:38 ~8 min android-e2e 📄log
❌ 65be73f #1 2022-05-19 16:06:09 ~8 min android 📄log
❌ 65be73f #1 2022-05-19 16:10:59 ~13 min ios 📄log
❌ 8e9c547 #2 2022-05-19 16:11:30 ~8 min android-e2e 📄log
❌ 8e9c547 #2 2022-05-19 16:11:39 ~9 min android 📄log
❌ 8e9c547 #2 2022-05-19 16:16:05 ~13 min ios 📄log
❌ 455d949 #3 2022-05-19 16:22:49 ~9 min android-e2e 📄log
❌ 455d949 #3 2022-05-19 16:25:46 ~11 min android 📄log
❌ 455d949 #3 2022-05-19 16:26:31 ~12 min ios 📄log
❌ 724a078 #4 2022-05-19 16:37:06 ~8 min android-e2e 📄log
❌ 724a078 #4 2022-05-19 16:37:21 ~8 min android 📄log
❌ 724a078 #4 2022-05-19 16:42:44 ~13 min ios 📄log
❌ 9c11c2e #5 2022-05-19 16:41:30 ~5 min ios 📄log
✔️ 9c11c2e #5 2022-05-19 16:46:09 ~9 min android 📦apk 📲
✔️ 9c11c2e #5 2022-05-19 16:46:21 ~9 min android-e2e 📦apk 📲
✔️ 0568c55 #6 2022-05-25 14:46:52 ~11 min android-e2e 📦apk 📲
✔️ 0568c55 #6 2022-05-25 14:47:04 ~11 min android 📦apk 📲
✔️ 0568c55 #6 2022-05-25 14:47:56 ~12 min ios 📦ipa 📲
❌ 91e8f49 #7 2022-05-25 15:00:30 ~4 min ios 📄log
✔️ 91e8f49 #7 2022-05-25 15:05:08 ~9 min android 📦apk 📲
✔️ 91e8f49 #7 2022-05-25 15:07:21 ~11 min android-e2e 📦apk 📲
✔️ f0401bc #8 2022-05-25 15:19:23 ~9 min android-e2e 📦apk 📲
✔️ f0401bc #8 2022-05-25 15:21:23 ~11 min android 📦apk 📲
✔️ f0401bc #8 2022-05-25 15:22:55 ~13 min ios 📦ipa 📲
❌ 20972ed #9 2022-05-25 15:19:56 ~5 min ios 📄log
❌ 20972ed #9 2022-05-25 15:25:17 ~10 min android-e2e 📄log
✔️ 20972ed #9 2022-05-25 15:26:17 ~11 min android 📦apk 📲
✔️ d74f7ff #10 2022-05-25 15:24:50 ~9 min android 📦apk 📲
❌ d74f7ff #10 2022-05-25 15:26:43 ~11 min android-e2e 📄log
❌ d74f7ff #10 2022-05-25 15:28:37 ~12 min ios 📄log
❌ 91ced1a #11 2022-05-25 15:26:44 ~4 min ios 📄log
✔️ 91ced1a #11 2022-05-25 15:31:00 ~9 min android-e2e 📦apk 📲
✔️ 91ced1a #11 2022-05-25 15:32:16 ~10 min android 📦apk 📲
✔️ 10123fa #12 2022-05-25 15:36:59 ~10 min ios 📦ipa 📲
✔️ 10123fa #12 2022-05-25 15:39:39 ~13 min android 📦apk 📲
✔️ 10123fa #12 2022-05-25 15:41:07 ~14 min android-e2e 📦apk 📲
✔️ 971dd06 #13 2022-05-25 15:49:22 ~10 min android 📦apk 📲
✔️ 971dd06 #13 2022-05-25 15:50:12 ~11 min android-e2e 📦apk 📲
✔️ 971dd06 #13 2022-05-25 15:52:20 ~13 min ios 📦ipa 📲
✔️ 28a9e3b #14 2022-05-25 15:53:39 ~8 min android-e2e 📦apk 📲
✔️ 28a9e3b #14 2022-05-25 15:54:17 ~9 min android 📦apk 📲
✔️ 28a9e3b #14 2022-05-25 15:56:03 ~11 min ios 📦ipa 📲
✔️ 3a8d704 #15 2022-05-25 16:09:46 ~11 min android 📦apk 📲
✔️ 3a8d704 #15 2022-05-25 16:09:47 ~11 min ios 📦ipa 📲
✔️ 3a8d704 #15 2022-05-25 16:09:57 ~11 min android-e2e 📦apk 📲
❔ Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ fbb3e27 #16 2022-05-25 16:49:09 ~10 min ios 📦ipa 📲
✔️ fbb3e27 #16 2022-05-25 16:49:28 ~11 min android-e2e 📦apk 📲
✔️ fbb3e27 #16 2022-05-25 16:49:33 ~11 min android 📦apk 📲
✔️ a5b0922 #17 2022-05-25 16:50:04 ~11 min android-e2e 📦apk 📲
✔️ a5b0922 #17 2022-05-25 16:50:23 ~11 min android 📦apk 📲
✔️ a5b0922 #17 2022-05-25 16:52:57 ~14 min ios 📦ipa 📲

@cammellos cammellos force-pushed the chore/add-status-go-tests branch 3 times, most recently from 724a078 to 9c11c2e Compare May 19, 2022 16:36
jakubgs
jakubgs requested changes May 20, 2022
View reviewed changes
Copy link
Member

@jakubgs jakubgs left a comment

Choose a reason for hiding this comment

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

Good stuff man.

Makefile Outdated Show resolved Hide resolved
nix/status-go/library/default.nix Outdated Show resolved Hide resolved
nix/status-go/library/default.nix Outdated Show resolved Hide resolved
nix/status-go/library/default.nix Outdated Show resolved Hide resolved
nix/status-go/library/default.nix Outdated Show resolved Hide resolved
nix/status-go/library/default.nix Outdated Show resolved Hide resolved
nix/status-go/library/default.nix Outdated Show resolved Hide resolved
nix/status-go/library/default.nix Outdated Show resolved Hide resolved
nix/status-go/library/default.nix Outdated Show resolved Hide resolved
nix/status-go/library/default.nix Outdated Show resolved Hide resolved
@cammellos cammellos marked this pull request as ready for review May 25, 2022 14:48
@cammellos cammellos force-pushed the chore/add-status-go-tests branch 7 times, most recently from 10123fa to 971dd06 Compare May 25, 2022 15:38
@jakubgs
Copy link
Member

jakubgs commented May 25, 2022

I verified on a CI host that make status-go-library builds fine on MacOS without nativeBuildInputs and preConfigure.

jakubgs
jakubgs approved these changes May 25, 2022
View reviewed changes
Copy link
Member

@jakubgs jakubgs left a comment

Choose a reason for hiding this comment

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

Cool stuff.

@cammellos cammellos force-pushed the chore/add-status-go-tests branch 3 times, most recently from fbb3e27 to a5b0922 Compare May 25, 2022 16:38
@cammellos
Add status-go tests
e4cba0f 
Signed-off-by: Andrea Maria Piana <andrea.maria.piana@gmail.com>
@cammellos cammellos merged commit e4cba0f into develop May 25, 2022
Pipeline for QA automation moved this from REVIEW to DONE May 25, 2022
@cammellos cammellos deleted the chore/add-status-go-tests branch May 25, 2022 16:59
@jakubgs jakubgs mentioned this pull request Jun 2, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jakubgs jakubgs jakubgs approved these changes

Assignees

@cammellos cammellos

Labels
None yet
Projects
Pipeline for QA
  
DONE
Pipeline for QA
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Create node bindings of status-go that can be run in tests
3 participants
@cammellos @status-im-auto @jakubgs