status-im/status-mobile
2
0
Fork 0
mirror of https://github.com/status-im/status-mobile.git synced 2025-02-23 05:48:46 +00:00
Code Issues Projects Releases Wiki Activity

remove component test stage & fix browser lock

Browse Source 
I added component tests stage when i cherry picked : 05929a16c7 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 7958bf189c
This commit is contained in:
Siddarth Kumar 2023-06-29 09:06:41 +05:30
parent abdaa80a96
commit 7224ca70a5
No known key found for this signature in database
GPG Key ID: F84DB2CA5207F534
2 changed files with 1 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
ci/Jenkinsfile.tests
View File

@ -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') { stage('Upload') {
steps { steps {
script { script {

2
src/status_im/browser/core.cljs
View File

@ -44,7 +44,7 @@
(or dapp? (or dapp?
(and (not error?) (and (not error?)
(when url (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 (fx/defn remove-browser
{:events [:browser.ui/remove-browser-pressed]} {:events [:browser.ui/remove-browser-pressed]}