Make write on exit default false

piker/clearing/_paper_engine.py

@@ -262,7 +262,7 @@ class PaperBoi(Struct):
 
         with (
                 open_trade_ledger(self.broker, 'paper') as ledger,
-                open_pps(self.broker, 'paper') as table
+                open_pps(self.broker, 'paper', True) as table
              ):
                 ledger.update({oid: t.to_dict()})
                 # Write to pps toml right now
@@ -530,7 +530,7 @@ async def trades_dialogue(
 
     ):
 
-        with open_pps(broker, 'paper', write_on_exit=False) as table:
+        with open_pps(broker, 'paper') as table:
             # save pps in local state
             _positions.update(table.pps)
 

piker/pp.py

@@ -880,7 +880,7 @@ class PpsEncoder(toml.TomlEncoder):
 def open_pps(
     brokername: str,
     acctid: str,
-    write_on_exit: bool = True,
+    write_on_exit: bool = False,
 
 ) -> Generator[PpTable, None, None]:
     '''

tests/conftest.py

@@ -171,7 +171,7 @@ def open_test_pikerd_and_ems(
     ) 
 
 
-@pytest.fixture(scope='session')
+@pytest.fixture(scope='module')
 def delete_testing_dir():
     '''
     This fixture removes the temp directory

tests/test_paper.py

@@ -114,7 +114,7 @@ def _assert(
     executions,
 ):
     with (
-        open_pps(broker, account) as table,
+        open_pps(broker, account, write_on_exit=False) as table,
     ):
         '''
         Assert multiple cases including pps,