test (loading): Ensure data is loaded in correct step order

related to GH 48, occurs test_mock_experiment on some systems.  Previous
passes depended upon how glob ordered files.  Current version adds an
explicit test, in addition to fixing test_mock_experiment

mitosis/__init__.py

@@ -73,7 +73,7 @@ def variant_types():
 
 def load_trial_data(hexstr: str, *, trials_folder: Optional[Path | str] = None):
     trial = locate_trial_folder(hexstr, trials_folder=trials_folder)
-    all_files = glob("results*.dill", root_dir=trial)
+    all_files = sorted(glob("results*.dill", root_dir=trial))
     results = []
     for file in all_files:
         with open(trial / file, "rb") as fh:

mitosis/_disk.py

@@ -30,7 +30,7 @@ def load_mitosis_steps(
         )
     if any(not isinstance(vals, list) or len(vals) != 2 for vals in result.values()):
         raise RuntimeError("tool.mitosis.steps table is malformed")
-    return {k: tuple(v) for k, v in result.items()}
+    return {k: tuple(v)[:2] for k, v in result.items()}
 
 
 @lru_cache

mitosis/tests/test_all.py

@@ -1,3 +1,4 @@
+import pickle
 import sys
 from pathlib import Path
 from types import ModuleType
@@ -139,11 +140,25 @@ def test_mock_experiment(mock_steps, tmp_path):
         trials_folder=tmp_path,
     )
     data = mitosis.load_trial_data(exp_key, trials_folder=tmp_path)
-    assert len(data[1]["data"]) == 5
+    assert len(data[0]["data"]) == 5
     metadata = mitosis._disk.locate_trial_folder(exp_key, trials_folder=tmp_path)
     assert (metadata / "experiment").resolve().exists()
 
 
+def test_load_results_order(tmp_path):
+    exp_key = "test_results"
+    (tmp_path / exp_key).mkdir()
+    filename1 = "results1.dill"
+    filename2 = "results0.dill"
+    with open(tmp_path / exp_key / filename1, "wb") as f1:
+        pickle.dump("second", f1)
+    with open(tmp_path / exp_key / filename2, "wb") as f2:
+        pickle.dump("first", f2)
+    result = mitosis.load_trial_data(exp_key, trials_folder=tmp_path)
+    expected = ["first", "second"]
+    assert result == expected
+
+
 def test_mod_metadata_debug(mock_steps, tmp_path):
     hexstr = mitosis.run(
         mock_steps,