Skip to content

Commit

Permalink
Re-scope mocked method
Browse files Browse the repository at this point in the history
Mocked a method at too large of scope and it affected other tests.

Signed-off-by: Anderson Chauphan <[email protected]>
  • Loading branch information
achauphan committed Dec 5, 2024
1 parent cd9ef97 commit 49e08eb
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -630,7 +630,6 @@ def create_package_enables_file(self, dryrun=False):
job_name = self.arg_pr_jenkins_job_name

enable_map_entry = self.get_multi_property_from_config("ENABLE_MAP", job_name, delimeter=" ")

# Generate files using ATDM/TriBiTS Scripts
if enable_map_entry is None:
cmd = [os.path.join( self.arg_workspace_dir,
Expand Down Expand Up @@ -741,6 +740,7 @@ def prepare_test(self):
self.message("--- arg_ctest_driver = {}".format(self.arg_ctest_driver))
self.message("--- arg_ctest_drop_site = {}".format(self.arg_ctest_drop_site))
self.message("--- arg_ccache_enable = {}".format(self.arg_ccache_enable))
self.message("--- arg_skip_create_packageenables = {}".format(self.arg_skip_create_packageenables))
self.message("")
self.message("--- concurrency_build = {}".format(self.concurrency_build))
self.message("--- concurrency_test = {}".format(self.concurrency_test))
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -707,7 +707,7 @@ def test_TrilinosPRConfigurationBase_prepare_test_skip_create_package_enables_fi
args.skip_create_packageenables = True
pr_config = trilinosprhelpers.TrilinosPRConfigurationBase(args)

trilinosprhelpers.TrilinosPRConfigurationBase.create_package_enables_file = Mock()
pr_config.create_package_enables_file = Mock()
pr_config.prepare_test()

pr_config.create_package_enables_file.assert_not_called()
Expand Down

0 comments on commit 49e08eb

Please sign in to comment.