ISSUE #1836: Address PR Feedback

* Make method name align with parameters/ui

jira/client.py

@@ -1293,7 +1293,7 @@ def copy_dashboard(
 
     @cloud
     @experimental
-    def update_dashboard_automatic_refresh_seconds(
+    def update_dashboard_automatic_refresh_minutes(
         self, id: str, minutes: int
     ) -> Response:
         # NOTE(jpavlav): The payload expects milliseconds, we are doing a conversion
@@ -1638,7 +1638,9 @@ def group_members(self, group: str) -> OrderedDict:
                 (
                     user["id"]
                     if hasId
-                    else user.get("name") if hasName else user.get("accountId")
+                    else user.get("name")
+                    if hasName
+                    else user.get("accountId")
                 )
             ] = {
                 "name": user.get("name"),

jira/resources.py

@@ -608,9 +608,9 @@ def update(  # type: ignore[override] # incompatible supertype ignored
           Resource
         """
         options = self._options.copy()
-        options["path"] = (
-            f"dashboard/{dashboard_id}/items/{item_id}/properties/{self.key}"
-        )
+        options[
+            "path"
+        ] = f"dashboard/{dashboard_id}/items/{item_id}/properties/{self.key}"
         self.raw["value"].update(value)
         self._session.put(self.JIRA_BASE_URL.format(**options), self.raw["value"])
 
@@ -628,9 +628,9 @@ def delete(self, dashboard_id: str, item_id: str) -> Response:  # type: ignore[o
           Response
         """
         options = self._options.copy()
-        options["path"] = (
-            f"dashboard/{dashboard_id}/items/{item_id}/properties/{self.key}"
-        )
+        options[
+            "path"
+        ] = f"dashboard/{dashboard_id}/items/{item_id}/properties/{self.key}"
 
         return self._session.delete(self.JIRA_BASE_URL.format(**options))
 

tests/resources/test_dashboard.py

@@ -196,16 +196,16 @@ def test_dashboard_gadgets(self):
         reason="Functionality only available on Jira Cloud",
     )
     @allow_on_cloud
-    def test_update_dashboard_automatic_refresh_seconds(self):
+    def test_update_dashboard_automatic_refresh_minutes(self):
         dashboard = self.jira.create_dashboard(
             name=rndstr(), share_permissions=[{"type": "authenticated"}]
         )
         self.dashboards_to_delete.append(dashboard)
-        response = self.jira.update_dashboard_automatic_refresh_seconds(
+        response = self.jira.update_dashboard_automatic_refresh_minutes(
             dashboard.id, 10
         )
         self.assertEqual(response.status_code, 204)
-        response = self.jira.update_dashboard_automatic_refresh_seconds(dashboard.id, 0)
+        response = self.jira.update_dashboard_automatic_refresh_minutes(dashboard.id, 0)
         self.assertEqual(response.status_code, 204)
 
     @pytest.mark.skipif(