Skip to content

Commit a0ac22e

Browse files
committed
Provide mem_per_node argument when testing job_generator
1 parent 74470a5 commit a0ac22e

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

smartdispatch/tests/test_job_generator.py

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ def setUp(self):
2222
self.mem_per_node = 32
2323
self.modules = ["cuda", "python"]
2424

25-
self.queue = Queue(self.name, self.cluster_name, self.walltime, self.cores, 0, self.modules)
25+
self.queue = Queue(self.name, self.cluster_name, self.walltime, self.cores, 0, self.mem_per_node, self.modules)
2626
self.queue_gpu = Queue(self.name, self.cluster_name, self.walltime, self.cores, self.gpus, self.mem_per_node, self.modules)
2727

2828
def tearDown(self):
@@ -57,6 +57,10 @@ def test_generate_pbs(self):
5757
# Since queue has no gpus it should not be specified in PBS resource `nodes`
5858
assert_true('gpus' not in pbs_list[0].resources['nodes'])
5959

60+
# Test modules to load
61+
# Check if needed modules for this queue are included in the PBS file
62+
assert_equal(pbs_list[0].modules, self.modules)
63+
6064
# Test nb_gpus_per_command argument
6165
# Should needs two PBS file
6266
command_params = {'nb_gpus_per_command': self.gpus // 2}

0 commit comments

Comments
 (0)