Version 1.7.0

Author: MarcCote

tests/test_textworld.py

@@ -2,7 +2,6 @@
 # Licensed under the MIT license.
 
 
-import glob
 import shutil
 import tempfile
 import unittest
@@ -34,14 +33,6 @@ def setUpClass(cls):
     def tearDownClass(cls):
         shutil.rmtree(cls.tmpdir)
 
-    def setUp(self):
-        self.before_tw = glob.glob(pjoin(tempfile.gettempdir(), "tw_*"))
-
-    def tearDown(self):
-        # Check for file leaks.
-        after_tw = glob.glob(pjoin(tempfile.gettempdir(), "tw_*"))
-        assert set(after_tw) == set(self.before_tw)
-
     def test_100_sequential_runs(self):
         for gamefile in [self.game_file, self.game_file_z8]:
             for _ in range(100):

textworld/challenges/tests/test_coin_collector.py

@@ -15,7 +15,7 @@ def test_making_coin_collector():
         101: {"quest_length": 1, "nb_rooms": 2},
         200: {"quest_length": 100, "nb_rooms": 200},
         201: {"quest_length": 1, "nb_rooms": 3},
-        300: {"quest_length": 100, "nb_rooms": 300},
+        # 300: {"quest_length": 100, "nb_rooms": 300},  # Too large for .z8 game file.
     }
     for level in [1, 100, 101, 200, 201, 300]:
         options = textworld.GameOptions()

textworld/challenges/tests/test_cooking.py

@@ -23,7 +23,6 @@ def test_making_cooking_games():
         "recipe": nb_ingredients,
         "take": 3,
         "open": True,
-        "open": True,
         "cook": True,
         "cut": False,
         "drop": False,

textworld/challenges/tw_cooking/cooking.py

@@ -1013,7 +1013,7 @@ def _place_one_distractor(candidates, ingredient):
             _place_one_distractor(same_suffix_list, ingredient)
 
     # Add distractors foods. The amount is drawn from N(nb_ingredients, 3).
-    nb_distractors = abs(int(rng_objects.randn(1) * 3 + nb_ingredients))
+    nb_distractors = abs(int(rng_objects.randn(1).item() * 3 + nb_ingredients))
     distractors = place_random_foods(M, nb_distractors, rng_objects, allowed_foods)
 
     # If recipe_seed is positive, a new recipe is sampled.

textworld/envs/wrappers/tests/test_tw_inform7.py

@@ -31,7 +31,7 @@ def setUpClass(cls):
         cls.tmpdir = tempfile.mkdtemp()
         cls.options = textworld.GameOptions()
         cls.options.path = pjoin(cls.tmpdir, "tw-game.z8")
-        cls.gamefile_z8 = textworld.generator.compile_game(cls.game, cls.options)
+        cls.game, cls.gamefile_z8 = testing.build_and_compile_game(cls.options)
         cls.request_infos = EnvInfos(
             inventory=True,
             description=True,
@@ -185,7 +185,7 @@ def setUpClass(cls):
         cls.tmpdir = tempfile.mkdtemp()
         cls.options = textworld.GameOptions()
         cls.options.path = pjoin(cls.tmpdir, "tw-game.z8")
-        cls.gamefile_z8 = textworld.generator.compile_game(cls.game, cls.options)
+        cls.game, cls.gamefile_z8 = testing.build_and_compile_game(cls.options)
         cls.request_infos = EnvInfos(
             inventory=True,
             description=True,
@@ -270,7 +270,7 @@ def setUpClass(cls):
         cls.tmpdir = tempfile.mkdtemp()
         cls.options = textworld.GameOptions()
         cls.options.path = pjoin(cls.tmpdir, "tw-game.z8")
-        cls.gamefile_z8 = textworld.generator.compile_game(cls.game, cls.options)
+        cls.game, cls.gamefile_z8 = testing.build_and_compile_game(cls.options)
         cls.request_infos = EnvInfos(
             max_score=True,
             objective=True,
@@ -356,7 +356,7 @@ def setUpClass(cls):
         cls.tmpdir = tempfile.mkdtemp()
         cls.options = textworld.GameOptions()
         cls.options.path = pjoin(cls.tmpdir, "tw-game.z8")
-        cls.gamefile_z8 = textworld.generator.compile_game(cls.game, cls.options)
+        cls.game, cls.gamefile_z8 = testing.build_and_compile_game(cls.options)
         cls.request_infos = EnvInfos(
             facts=True,
             policy_commands=True,

textworld/gym/tests/test_textworld_gym.py

@@ -1,5 +1,4 @@
 import os
-import glob
 import shutil
 import tempfile
 import unittest
@@ -34,14 +33,6 @@ def setUpClass(cls):
     def tearDownClass(cls):
         shutil.rmtree(cls.tmpdir)
 
-    def setUp(self):
-        self.before_tw = glob.glob(pjoin(tempfile.gettempdir(), "tw_*"))
-
-    def tearDown(self):
-        # Check for file leaks.
-        after_tw = glob.glob(pjoin(tempfile.gettempdir(), "tw_*"))
-        assert set(after_tw) == set(self.before_tw)
-
     def test_register_game(self):
         env_options = EnvInfos(inventory=True, description=True,
                                admissible_commands=True,

textworld/version.py

@@ -1 +1 @@
-__version__ = '1.6.2'
+__version__ = '1.7.0rc1'