Merge pull request #6294 from MetRonnie/comm

Demote "timer stopped" warning to info & fix functional tests `contains_ok` function

cylc/flow/timer.py

@@ -52,7 +52,7 @@ def stop(self) -> None:
         if self.timeout is None:
             return
         self.timeout = None
-        LOG.warning(f"{self.name} stopped")
+        LOG.info(f"{self.name} stopped")
 
     def timed_out(self) -> bool:
         """Return whether timed out yet."""

tests/functional/lib/bash/test_header

@@ -386,7 +386,7 @@ contains_ok() {
     local FILE_CONTROL="${2:--}"
     local TEST_NAME
     TEST_NAME="$(basename "${FILE_TEST}")-contains-ok"
-    LANG=C comm -13 <(sort "${FILE_TEST}") <(sort "${FILE_CONTROL}") \
+    LANG=C comm -13 <(LANG=C sort "${FILE_TEST}") <(LANG=C sort "${FILE_CONTROL}") \
         1>"${TEST_NAME}.stdout" 2>"${TEST_NAME}.stderr"
     if [[ -s "${TEST_NAME}.stdout" ]]; then
         mkdir -p "${TEST_LOG_DIR}"

tests/unit/test_timer.py

@@ -14,6 +14,7 @@
 # You should have received a copy of the GNU General Public License
 # along with this program.  If not, see <http://www.gnu.org/licenses/>.
 
+import logging
 from time import sleep
 
 import pytest
@@ -23,7 +24,7 @@
 
 def test_Timer(caplog: pytest.LogCaptureFixture):
     """Test the Timer class."""
-    caplog.set_level('WARNING')
+    caplog.set_level(logging.INFO)
     timer = Timer("bob timeout", 1.0)
 
     # timer attributes