Temporarily disable LedgerSigner tests (#1477)

* Skip `LedgerSigner` tests

* Update skip reason

* Remove unused import

* Update directory while building docker Starknet ledger app

Author: franciszekjob

.github/workflows/checks.yml

@@ -196,7 +196,7 @@ jobs:
           image: ghcr.io/ledgerhq/ledger-app-builder/ledger-app-dev-tools
           options: --rm -v ${{ github.workspace }}:/apps
           run: |
-            cd /apps/app-starknet
+            cd /apps/app-starknet/starknet
             cargo clean
             cargo ledger build nanox
 

starknet_py/tests/unit/signer/test_ledger_signer.py

@@ -1,5 +1,3 @@
-from sys import platform
-
 import pytest
 
 from starknet_py.common import create_sierra_compiled_contract
@@ -20,11 +18,8 @@
 from starknet_py.tests.e2e.fixtures.misc import load_contract
 
 
-# TODO (#1425): Currently Ledger tests are skipped on Windows due to different Speculos setup.
-@pytest.mark.skipif(
-    platform == "win32",
-    reason="Testing Ledger is skipped on Windows due to different Speculos setup.",
-)
+# TODO (#1476): Should be re-enabled once Ledger signer is updated with new APDU spec.
+@pytest.mark.skip
 def test_init_with_invalid_derivation_path():
     with pytest.raises(ValueError, match="Empty derivation path"):
         LedgerSigner(derivation_path_str="", chain_id=StarknetChainId.SEPOLIA)
@@ -81,10 +76,8 @@ def test_init_with_invalid_derivation_path():
         ),
     ],
 )
-@pytest.mark.skipif(
-    platform == "win32",
-    reason="Testing Ledger is skipped on Windows due to different Speculos setup.",
-)
+# TODO (#1476): Should be re-enabled once Ledger signer is updated with new APDU spec.
+@pytest.mark.skip
 def test_sign_transaction(transaction):
     # docs: start
 
@@ -104,10 +97,8 @@ def test_sign_transaction(transaction):
     assert all(i != 0 for i in signature)
 
 
-@pytest.mark.skipif(
-    platform == "win32",
-    reason="Testing Ledger is skipped on Windows due to different Speculos setup.",
-)
+# TODO (#1476): Should be re-enabled once Ledger signer is updated with new APDU spec.
+@pytest.mark.skip
 def test_create_account_with_ledger_signer():
     # pylint: disable=unused-variable
     signer = LedgerSigner(
@@ -141,10 +132,8 @@ async def _get_account_balance_strk(client: FullNodeClient, address: int):
 
 
 @pytest.mark.asyncio
-@pytest.mark.skipif(
-    platform == "win32",
-    reason="Testing Ledger is skipped on Windows due to different Speculos setup.",
-)
+# TODO (#1476): Should be re-enabled once Ledger signer is updated with new APDU spec.
+@pytest.mark.skip
 async def test_deploy_account_and_transfer(client):
     signer = LedgerSigner(
         derivation_path_str="m/2645'/1195502025'/1470455285'/0'/0'/0",