Fix CI test failures introduced by resources branch

- salt/states/saltutil.py: add sync_resources() state function to match
  the sync_resources() module function added in saltutil.py; the existing
  test enforces a 1:1 correspondence between all sync_* module and state
  functions
- tests/pytests/unit/client/test_netapi.py: update test_run_log to use
  AsyncMock for process_manager.run(), which our netapi.py change wrapped
  in asyncio.run() and therefore requires a coroutine, not a plain Mock

Author: dwoz

salt/states/saltutil.py

@@ -323,6 +323,20 @@ def sync_tops(name, **kwargs):
     return _sync_single(name, "tops", **kwargs)
 
 
+def sync_resources(name, **kwargs):
+    """
+    Performs the same task as saltutil.sync_resources module
+    See :mod:`saltutil module for full list of options <salt.modules.saltutil>`
+
+    .. code-block:: yaml
+
+        sync_everything:
+          saltutil.sync_resources:
+            - refresh: True
+    """
+    return _sync_single(name, "resources", **kwargs)
+
+
 def sync_thorium(name, **kwargs):
     """
     Performs the same task as saltutil.sync_thorium module

tests/pytests/unit/client/test_netapi.py

@@ -1,7 +1,7 @@
 import logging
 
 import salt.client.netapi
-from tests.support.mock import Mock, patch
+from tests.support.mock import AsyncMock, Mock, patch
 
 
 def test_run_log(caplog, master_opts):
@@ -11,6 +11,7 @@ def test_run_log(caplog, master_opts):
     master_opts["rest_cherrypy"] = {"port": 8000}
     mock_process = Mock()
     mock_process.add_process.return_value = True
+    mock_process.return_value.run = AsyncMock()
     patch_process = patch.object(salt.utils.process, "ProcessManager", mock_process)
     with caplog.at_level(logging.INFO):
         with patch_process: