From 0b85fe29f7e49c88cab88aa9303d5885fa9c9dd5 Mon Sep 17 00:00:00 2001
From: Andrei Zeliankou <zelenkov@nginx.com>
Date: Wed, 8 Nov 2023 18:37:02 +0000
Subject: Tests: 8XXX used as default port range.

After the launch of the project, the testing infrastructure was shared with
nginx project in some cases.  To avoid port overlap, a decision was made
to shift the port range for Unit tests.  This problem was resolved a long time
ago and is no longer relevant, so it is now safe to use port 8XXX range as the
default, as it is more appropriate for testing purposes.
---
 test/test_settings.py | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

(limited to 'test/test_settings.py')

diff --git a/test/test_settings.py b/test/test_settings.py
index 33180046..0cdc6c0d 100644
--- a/test/test_settings.py
+++ b/test/test_settings.py
@@ -460,7 +460,7 @@ def test_settings_log_route(findall, search_in_file, wait_for_record):
 
     assert 'success' in client.conf(
         {
-            "listeners": {"*:7080": {"pass": "routes"}},
+            "listeners": {"*:8080": {"pass": "routes"}},
             "routes": [
                 {
                     "match": {
@@ -487,7 +487,7 @@ def test_settings_log_route(findall, search_in_file, wait_for_record):
 
     assert 'success' in client.conf(
         {
-            "listeners": {"*:7080": {"pass": "routes/main"}},
+            "listeners": {"*:8080": {"pass": "routes/main"}},
             "routes": {
                 "main": [
                     {
@@ -516,7 +516,7 @@ def test_settings_log_route(findall, search_in_file, wait_for_record):
 
     assert 'success' in client.conf(
         {
-            "listeners": {"*:7080": {"pass": "routes/first"}},
+            "listeners": {"*:8080": {"pass": "routes/first"}},
             "routes": {
                 "first": [
                     {
@@ -541,7 +541,7 @@ def test_settings_log_route(findall, search_in_file, wait_for_record):
 
     assert 'success' in client.conf(
         {
-            "listeners": {"*:7080": {"pass": "routes/fall"}},
+            "listeners": {"*:8080": {"pass": "routes/fall"}},
             "routes": {
                 "fall": [
                     {
-- 
cgit 


From 5a8337933df1cf3aba967d86549e236dd9173386 Mon Sep 17 00:00:00 2001
From: Andrei Zeliankou <zelenkov@nginx.com>
Date: Mon, 15 Jan 2024 15:48:58 +0000
Subject: Tests: pathlib used where appropriate

Also fixed various pylint errors and style issues.
---
 test/test_settings.py | 1 +
 1 file changed, 1 insertion(+)

(limited to 'test/test_settings.py')

diff --git a/test/test_settings.py b/test/test_settings.py
index 0cdc6c0d..9d37d6ca 100644
--- a/test/test_settings.py
+++ b/test/test_settings.py
@@ -4,6 +4,7 @@ import subprocess
 import time
 
 import pytest
+
 from unit.applications.lang.python import ApplicationPython
 
 prerequisites = {'modules': {'python': 'any'}}
-- 
cgit