From 7224ca70a5cf8d1311cfe88c904e5bd769c25ffb Mon Sep 17 00:00:00 2001 From: Siddarth Kumar Date: Thu, 29 Jun 2023 09:06:41 +0530 Subject: [PATCH] remove component test stage & fix browser lock I added component tests stage when i cherry picked : https://github.com/status-im/status-mobile/commit/05929a16c7bb5cfa2ac6e1fec32cdf9935a92a7f commit from develop to release branch. Only to later realise that we did not have component tests at the time of this release. This commit removes component tests form CI. This commit also fixes a logic where the secure lock was not shown for all the websites except ".top". Those changes were introduced by me in https://github.com/status-im/status-mobile/commit/7958bf189caf4555971fbeb9affd6bd8fe7efe33 --- ci/Jenkinsfile.tests | 8 -------- src/status_im/browser/core.cljs | 2 +- 2 files changed, 1 insertion(+), 9 deletions(-) diff --git a/ci/Jenkinsfile.tests b/ci/Jenkinsfile.tests index fe5591c1d0..dbf372d6ab 100644 --- a/ci/Jenkinsfile.tests +++ b/ci/Jenkinsfile.tests @@ -61,14 +61,6 @@ pipeline { } } } - stage('Component Tests') { - steps { - sh """#!/bin/bash - set -eo pipefail - make component-test 2>&1 | tee -a ${LOG_FILE} - """ - } - } stage('Upload') { steps { script { diff --git a/src/status_im/browser/core.cljs b/src/status_im/browser/core.cljs index 090d9fe898..c34f6894d3 100644 --- a/src/status_im/browser/core.cljs +++ b/src/status_im/browser/core.cljs @@ -44,7 +44,7 @@ (or dapp? (and (not error?) (when url - (and (string/starts-with? url "https://") (string/includes? url ".top")))))) + (and (string/starts-with? url "https://") (not (string/includes? url ".top"))))))) (fx/defn remove-browser {:events [:browser.ui/remove-browser-pressed]}