search

AgnostiqHQ / covalent-slurm-plugin

Executor plugin interfacing Covalent with Slurm
https://covalent.xyz
Apache License 2.0
27 stars 6 forks source link

Add tests #14

Closed jkanem closed 2 years ago

jkanem commented 2 years ago

Environment

What is happening?

The Slurm executor plugin needs unit tests and a functional test.

How can we reproduce the issue?

Notice that the repo doesn't have any tests.

What should happen?

There should be a suite of unit tests and a functional test for the executor. See the tests in the custom executor template for guidance (https://github.com/AgnostiqHQ/covalent-executor-template)

Any suggestions?

No response

jkanem commented 2 years ago

Partially fixes https://github.com/AgnostiqHQ/covalent-staging/issues/413

jkanem commented 2 years ago

Fixed by https://github.com/AgnostiqHQ/covalent-slurm-plugin/pull/15