Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test_activate_operation_updates_active_users needs a review #2546

Open
ReimarBauer opened this issue Oct 4, 2024 · 3 comments
Open

test_activate_operation_updates_active_users needs a review #2546

ReimarBauer opened this issue Oct 4, 2024 · 3 comments
Labels

Comments

@ReimarBauer
Copy link
Member

I see this sometimes on my local tests

[gw5] [100%] PASSED tests/_test_msui/test_mscolab.py::Test_Mscolab::test_activate_operation_updates_active_users 

Exception ignored in: <function WMSControlWidget.__del__ at 0x718a67ac9120>
Traceback (most recent call last):
  File "/home/reimar/MAIN/MSS/mslib/msui/wms_control.py", line 550, in __del__
    self.service_cache()
  File "/home/reimar/MAIN/MSS/mslib/msui/wms_control.py", line 1455, in service_cache
    files = [os.path.join(self.wms_cache, f) for f in os.listdir(self.wms_cache)]
                                                      ^^^^^^^^^^^^^^^^^^^^^^^^^^
FileNotFoundError: [Errno 2] Datei oder Verzeichnis nicht gefunden: '/tmp/tmpnodr4uwv/msui/wms_cache/'
@ReimarBauer
Copy link
Member Author

@Pavanbhagi
Copy link

import os

class WMSControlWidget:
def init(self, wms_cache):
self.wms_cache = wms_cache
os.makedirs(self.wms_cache, exist_ok=True) # Ensure the cache directory exists

def service_cache(self):
    if os.path.exists(self.wms_cache):
        files = [os.path.join(self.wms_cache, f) for f in os.listdir(self.wms_cache)]
        # ... process files
    else:
        files = []  # Handle the case where the directory does not exist

def __del__(self):
    # Consider removing any critical logic here or checking for existence
    try:
        self.service_cache()
    except Exception as e:
        # Log the exception if necessary
        pass

@ReimarBauer
Copy link
Member Author

Not each test is failing. On production I don't have seen that problem. We need to investigate why in this order of tests the problem occures and likly improve a test.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants