Merge pull request #19 from python-paling:tests

Refactor test_examples.py and test_paling.py for Paling server integration tests

Author: savelkouls

tests/integration/test_examples.py

@@ -7,12 +7,12 @@
 from selenium.webdriver.support import expected_conditions
 from selenium.webdriver.support.wait import WebDriverWait
 
-from tests.utils import get_eel_server, get_console_logs
+from tests.utils import get_paling_server, get_console_logs
 
 
 def test_01_hello_world(driver):
-    with get_eel_server('examples/01 - hello_world/hello.py', 'hello.html') as eel_url:
-        driver.get(eel_url)
+    with get_paling_server('examples/01 - hello_world/hello.py', 'hello.html') as paling_url:
+        driver.get(paling_url)
         assert driver.title == "Hello, World!"
 
         console_logs = get_console_logs(driver, minimum_logs=2)
@@ -21,8 +21,8 @@ def test_01_hello_world(driver):
 
 
 def test_02_callbacks(driver):
-    with get_eel_server('examples/02 - callbacks/callbacks.py', 'callbacks.html') as eel_url:
-        driver.get(eel_url)
+    with get_paling_server('examples/02 - callbacks/callbacks.py', 'callbacks.html') as paling_url:
+        driver.get(paling_url)
         assert driver.title == "Callbacks Demo"
 
         console_logs = get_console_logs(driver, minimum_logs=1)
@@ -31,8 +31,8 @@ def test_02_callbacks(driver):
 
 
 def test_03_callbacks(driver):
-    with get_eel_server('examples/03 - sync_callbacks/sync_callbacks.py', 'sync_callbacks.html') as eel_url:
-        driver.get(eel_url)
+    with get_paling_server('examples/03 - sync_callbacks/sync_callbacks.py', 'sync_callbacks.html') as paling_url:
+        driver.get(paling_url)
         assert driver.title == "Synchronous callbacks"
 
         console_logs = get_console_logs(driver, minimum_logs=1)
@@ -41,9 +41,9 @@ def test_03_callbacks(driver):
 
 
 def test_04_file_access(driver: webdriver.Remote):
-    with get_eel_server('examples/04 - file_access/file_access.py', 'file_access.html') as eel_url:
-        driver.get(eel_url)
-        assert driver.title == "Eel Demo"
+    with get_paling_server('examples/04 - file_access/file_access.py', 'file_access.html') as paling_url:
+        driver.get(paling_url)
+        assert driver.title == "Paling Demo"
 
         with TemporaryDirectory() as temp_dir, NamedTemporaryFile(dir=temp_dir) as temp_file:
             driver.find_element_by_id('input-box').clear()
@@ -55,23 +55,23 @@ def test_04_file_access(driver: webdriver.Remote):
 
 
 def test_06_jinja_templates(driver: webdriver.Remote):
-    with get_eel_server('examples/06 - jinja_templates/hello.py', 'templates/hello.html') as eel_url:
-        driver.get(eel_url)
+    with get_paling_server('examples/06 - jinja_templates/hello.py', 'templates/hello.html') as paling_url:
+        driver.get(paling_url)
         assert driver.title == "Hello, World!"
 
         driver.find_element_by_css_selector('a').click()
         WebDriverWait(driver, 2.0).until(expected_conditions.presence_of_element_located((By.XPATH, '//h1[text()="This is page 2"]')))
 
 
 def test_10_custom_app(driver: webdriver.Remote):
-    # test default eel routes are working
-    with get_eel_server('examples/10 - custom_app_routes/custom_app.py', 'index.html') as eel_url:
-        driver.get(eel_url)
+    # test default paling routes are working
+    with get_paling_server('examples/10 - custom_app_routes/custom_app.py', 'index.html') as paling_url:
+        driver.get(paling_url)
         # we really need to test if the page 404s, but selenium has no support for status codes
         # so we just test if we can get our page title
         assert driver.title == 'Hello, World!'
 
     # test custom routes are working
-    with get_eel_server('examples/10 - custom_app_routes/custom_app.py', 'custom') as eel_url:
-        driver.get(eel_url)
+    with get_paling_server('examples/10 - custom_app_routes/custom_app.py', 'custom') as paling_url:
+        driver.get(paling_url)
         assert 'Hello, World!' in driver.page_source

tests/unit/test_eel.py tests/unit/test_paling.pytests/unit/test_eel.py renamed to tests/unit/test_paling.py

@@ -38,19 +38,19 @@ def get_process_listening_port(proc):
 
 
 @contextlib.contextmanager
-def get_eel_server(example_py, start_html):
-    """Run an Eel example with the mode/port overridden so that no browser is launched and a random port is assigned"""
+def get_paling_server(example_py, start_html):
+    """Run an Paling example with the mode/port overridden so that no browser is launched and a random port is assigned"""
     test = None
 
     try:
         with tempfile.NamedTemporaryFile(mode='w', dir=os.path.dirname(example_py), delete=False) as test:
             # We want to run the examples unmodified to keep the test as realistic as possible, but all of the examples
-            # want to launch browsers, which won't be supported in CI. The below script will configure eel to open on
+            # want to launch browsers, which won't be supported in CI. The below script will configure paling to open on
             # a random port and not open a browser, before importing the Python example file - which will then
-            # do the rest of the set up and start the eel server. This is definitely hacky, and means we can't
+            # do the rest of the set up and start the paling server. This is definitely hacky, and means we can't
             # test mode/port settings for examples ... but this is OK for now.
             test.write(f"""
-import eel
+import paling
 
 paling._start_args['mode'] = None
 paling._start_args['port'] = 0
@@ -61,9 +61,9 @@ def get_eel_server(example_py, start_html):
                 [sys.executable, test.name],
                 cwd=os.path.dirname(example_py),
             )
-        eel_port = get_process_listening_port(proc)
+        paling_port = get_process_listening_port(proc)
 
-        yield f"http://localhost:{eel_port}/{start_html}"
+        yield f"http://localhost:{paling_port}/{start_html}"
 
         proc.terminate()