don't run flaky CI in Windows (#4935)

* don't run flaky CI in Windows

* reduce stack size in `test_light_client`

* regen tests

* re-enable maybe non-problematic test module on Windows

---------

Co-authored-by: Etan Kissling <etan@status.im>

tests/all_tests.nim

@@ -56,7 +56,9 @@ when not defined(i386):
   import
     ./test_blockchain_dag,
     ./test_keystore,
-    ./test_keystore_management,
-    ./test_keymanager_api
+    ./test_keystore_management
+
+  when not defined(windows):
+    import ./test_keymanager_api
 
 summarizeLongTests("AllTests")

tests/test_light_client.nim

@@ -191,12 +191,15 @@ suite "Light client" & preset():
             store.migrateToDataFork(lcDataFork)
       withForkyStore(store):
         when lcDataFork > LightClientDataFork.None:
+          # Reduce stack size by making this a `proc`
+          proc syncToPeriod() =
             bootstrap.migrateToDataFork(lcDataFork)
             template forkyBootstrap: untyped = bootstrap.forky(lcDataFork)
             let upgradedUpdate = update.migratingToDataFork(lcDataFork)
             template forkyUpdate: untyped = upgradedUpdate.forky(lcDataFork)
             let res = process_light_client_update(
-            forkyStore, forkyUpdate, currentSlot, cfg, genesis_validators_root)
+              forkyStore, forkyUpdate, currentSlot, cfg,
+              genesis_validators_root)
             check:
               forkyUpdate.finalized_header.beacon.slot.sync_committee_period ==
                 period
@@ -206,6 +209,7 @@ suite "Light client" & preset():
                 forkyStore.finalized_header == forkyUpdate.finalized_header
               else:
                 forkyStore.finalized_header == forkyBootstrap.header
+          syncToPeriod()
       inc numIterations
       if numIterations > 20: doAssert false # Avoid endless loop on test failure