Allow to resolve salt in rust module tests
ClosedPublic
Actions

Authored by dereckson on Mar 7 2022, 19:35.

Details

Reviewers

dereckson

Commits

rOPS6fcc0848d68a: Allow to resolve __salt__ in rust module tests

Summary

Fixes the following issue:

======================================================================
ERROR: test_get_rustc_triplet (test_rust.Testinstance)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/opt/salt/nasqueron-operations/_tests/modules/test_rust.py", line 11, in test_get_rustc_triplet
    triplet = rust.get_rustc_triplet()
  File "../_modules/rust.py", line 35, in get_rustc_triplet
    return __salt__["cmd.shell"]("rustc -vV | sed -ne 's/^host: //p'")
NameError: name '__salt__' is not defined

Test Plan

make test

Diff Detail

Repository

rOPS Nasqueron Operations

Lint

Lint Passed

Unit

No Test Coverage

Branch

mock

Build Status

Buildable 4034
Build 4286: arc lint + arc unit

Event Timeline

dereckson requested review of this revision.Mar 7 2022, 19:35

dereckson created this revision.

Harbormaster completed remote builds in B4034: Diff 6504.Mar 7 2022, 19:35

dereckson accepted this revision.Mar 7 2022, 19:36

This revision is now accepted and ready to land.Mar 7 2022, 19:36

dereckson edited the test plan for this revision. (Show Details)Mar 7 2022, 19:42

$ cd _tests
$ ./modules/test_rust.py
<frozen importlib._bootstrap>:283: DeprecationWarning: the load_module() method is deprecated and slated for removal in Python 3.12; use exec_module() instead
.
----------------------------------------------------------------------
Ran 1 test in 0.108s

OK