Revert "Attempt to use and setup a proxy for CI"

This reverts commit 48ae248.
koosai · Jan 16, 2023 · 394a973 · 394a973
1 parent 48ae248
commit 394a973
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 30 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -360,8 +360,6 @@ jobs:
         pip install -e .
 
     - name: Tests incl. ccxt compatibility tests
-      env:
-        CI_WEB_PROXY: http://152.67.78.211:13128
       run: |
         pytest --random-order --cov=freqtrade --cov-config=.coveragerc --longrun
 

diff --git a/tests/exchange/test_ccxt_compat.py b/tests/exchange/test_ccxt_compat.py
@@ -12,7 +12,6 @@
 
 import pytest
 
-from freqtrade.constants import Config
 from freqtrade.enums import CandleType
 from freqtrade.exchange import timeframe_to_minutes, timeframe_to_prev_date
 from freqtrade.exchange.exchange import Exchange, timeframe_to_msecs
@@ -32,17 +31,16 @@
         'leverage_tiers_public': False,
         'leverage_in_spot_market': False,
     },
-    'binance': {
-        'pair': 'BTC/USDT',
-        'stake_currency': 'USDT',
-        'use_ci_proxy': True,
-        'hasQuoteVolume': True,
-        'timeframe': '5m',
-        'futures': True,
-        'futures_pair': 'BTC/USDT:USDT',
-        'leverage_tiers_public': False,
-        'leverage_in_spot_market': False,
-    },
+    # 'binance': {
+    #     'pair': 'BTC/USDT',
+    #     'stake_currency': 'USDT',
+    #     'hasQuoteVolume': True,
+    #     'timeframe': '5m',
+    #     'futures': True,
+    #     'futures_pair': 'BTC/USDT:USDT',
+    #     'leverage_tiers_public': False,
+    #     'leverage_in_spot_market': False,
+    # },
     'kraken': {
         'pair': 'BTC/USDT',
         'stake_currency': 'USDT',
@@ -109,23 +107,8 @@ def exchange_conf():
     return config
 
 
-def set_test_proxy(config: Config, use_proxy: bool) -> Config:
-    # Set proxy to test in CI.
-    import os
-    if use_proxy and (proxy := os.environ.get('CI_WEB_PROXY')):
-        config['exchange']['ccxt_config'] = {
-            'proxies': {
-                'https': proxy,
-                'http': proxy,
-            }
-        }
-
-    return config
-
-
 @pytest.fixture(params=EXCHANGES, scope="class")
 def exchange(request, exchange_conf):
-    set_test_proxy(exchange_conf, EXCHANGES[request.param].get('use_ci_proxy', False))
     exchange_conf['exchange']['name'] = request.param
     exchange_conf['stake_currency'] = EXCHANGES[request.param]['stake_currency']
     exchange = ExchangeResolver.load_exchange(request.param, exchange_conf, validate=True)
@@ -138,7 +121,6 @@ def exchange_futures(request, exchange_conf, class_mocker):
     if not EXCHANGES[request.param].get('futures') is True:
         yield None, request.param
     else:
-        set_test_proxy(exchange_conf, EXCHANGES[request.param].get('use_ci_proxy', False))
         exchange_conf = deepcopy(exchange_conf)
         exchange_conf['exchange']['name'] = request.param
         exchange_conf['trading_mode'] = 'futures'