Unit test adjustments (Post v0.4.2 merge) (#75)

* Initial v0.4.2 update - Cleanup and normalization

* Updated endpoint list. Deprecated invalid method names.

* Added local unit testing initialization script

* Linting helper script

* FalconGroupingTags via SDK (#72)

* added endpoint information for host tags. added tests

* missed comma and param name

* missed param, not caught in previous commit

* removed an unused test, no changes to uber test

* merging and removing left over conflicts

* Update _endpoint.py

* Update hosts.py

Moving FalconGroupingTags -> UpdateDeviceTags

* Update test_hosts.py

Moving FalconGroupignTags -> UpdateDeviceTags

Co-authored-by: Caleb Schwartz <[email protected]>
Co-authored-by: jshcodes <[email protected]>

* Linting script update, UpdateDeviceTags merge

* Minor fixes to UpdateDeviceTags

* Added unit tests to cover new code paths

* Unit test updates - Reducing AWS queries for Python <= 3.8

* Unit test adjustment - removing redundant service class check

* Removing uber class redundant test

Co-authored-by: rewgord <[email protected]>
Co-authored-by: Caleb Schwartz <[email protected]>

Author: jshcodes

tests/test_cloud_connect_aws.py

@@ -196,9 +196,10 @@ def test_GetAWSAccounts(self):
     def test_GetAWSAccountsUsingList(self):
         assert self.serviceCCAWS_GetAWSAccountsUsingList() == True
 
-    @pytest.mark.skipif(falcon.QueryAWSAccounts(parameters={"limit": 1})["status_code"] == 429, reason="API rate limit reached")
-    def test_VerifyAWSAccountAccess(self):
-        assert self.serviceCCAWS_VerifyAWSAccountAccess() == True
+    # @pytest.mark.skipif(falcon.QueryAWSAccounts(parameters={"limit": 1})["status_code"] == 429, reason="API rate limit reached")
+    # @pytest.mark.skipif(sys.version_info.minor < 9, reason="Frequency reduced due to potential race condition")
+    # def test_VerifyAWSAccountAccess(self):
+    #     assert self.serviceCCAWS_VerifyAWSAccountAccess() == True
 
     @pytest.mark.skipif(falcon.QueryAWSAccounts(parameters={"limit": 1})["status_code"] == 429, reason="API rate limit reached")
     @pytest.mark.skipif(sys.version_info.minor < 9, reason="Frequency reduced due to potential race condition")

tests/test_uber_api_complete.py

@@ -216,10 +216,11 @@ def test_QueryAWSAccounts(self):
     def test_GetAWSAccounts(self):
         assert self.uberCCAWS_GetAWSAccounts() == True
 
-    @pytest.mark.skipif(falcon.command("QueryAWSAccounts",
-                        parameters={"limit": 1})["status_code"] == 429, reason="API rate limit reached")
-    def test_VerifyAWSAccountAccess(self):
-        assert self.uberCCAWS_VerifyAWSAccountAccess() == True
+    # @pytest.mark.skipif(falcon.command("QueryAWSAccounts",
+    #                     parameters={"limit": 1})["status_code"] == 429, reason="API rate limit reached")
+    # @pytest.mark.skipif(sys.version_info.minor < 9, reason="Frequency reduced due to potential race condition")
+    # def test_VerifyAWSAccountAccess(self):
+    #     assert self.uberCCAWS_VerifyAWSAccountAccess() == True
 
     def test_QueryAWSAccountsForIDs(self):
         assert self.uberCCAWS_QueryAWSAccountsForIDs() == True