Fix flaky storage tests (#47987)

* Fix flaky storage tests

* Fix tests

* Fix tests

* Retrigger CI/CD pipeline

* Remove extra guard

* Fix format

Author: rads-1996

sdk/monitor/azure-monitor-opentelemetry-exporter/tests/conftest.py

@@ -30,5 +30,3 @@
 collect_ignore_glob = []
 if sys.version_info < (3, 6):
     collect_ignore_glob.append("*")
-
-collect_ignore = ["test_storage.py"]

sdk/monitor/azure-monitor-opentelemetry-exporter/tests/test_storage.py

@@ -4,6 +4,7 @@
 import errno
 import os
 import shutil
+import tempfile
 import unittest
 from unittest import mock
 
@@ -39,7 +40,7 @@ def func(*_args, **_kwargs):
 
 
 def clean_folder(folder):
-    if os.path.isfile(folder):
+    if os.path.isdir(folder):
         for filename in os.listdir(folder):
             file_path = os.path.join(folder, filename)
             try:
@@ -53,16 +54,12 @@ def clean_folder(folder):
 
 # pylint: disable=unused-variable
 class TestLocalFileBlob(unittest.TestCase):
-    @classmethod
-    def setup_class(cls):
-        os.makedirs(TEST_FOLDER, exist_ok=True)
-
-    @classmethod
-    def tearDownClass(cls):
-        shutil.rmtree(TEST_FOLDER, True)
+    def setUp(self):
+        global TEST_FOLDER
+        TEST_FOLDER = tempfile.mkdtemp()
 
     def tearDown(self):
-        clean_folder(TEST_FOLDER)
+        shutil.rmtree(TEST_FOLDER, ignore_errors=True)
 
     def test_delete(self):
         blob = LocalFileBlob(os.path.join(TEST_FOLDER, "foobar"))
@@ -320,9 +317,12 @@ def test_lease_with_existing_lock(self):
 
 # pylint: disable=protected-access, too-many-public-methods
 class TestLocalFileStorage(unittest.TestCase):
-    @classmethod
-    def tearDownClass(cls):
-        shutil.rmtree(TEST_FOLDER, True)
+    def setUp(self):
+        global TEST_FOLDER
+        TEST_FOLDER = tempfile.mkdtemp()
+
+    def tearDown(self):
+        shutil.rmtree(TEST_FOLDER, ignore_errors=True)
 
     def test_get_nothing(self):
         with LocalFileStorage(os.path.join(TEST_FOLDER, "test", "a")) as stor:
@@ -374,7 +374,7 @@ def test_check_storage_size_full(self):
         os.makedirs(test_path, exist_ok=True)
         with mock.patch.object(LocalFileStorage, "_check_and_set_folder_permissions", return_value=True):
             with LocalFileStorage(test_path, 1) as stor:
-                stor.put(test_input)
+                stor.put(test_input, lease_period=0)
                 self.assertFalse(stor._check_storage_size())
 
     def test_check_storage_size_not_full(self):