Fix Worksheet ID Type Inconsistencies

gspread/spreadsheet.py

@@ -7,6 +7,7 @@
 """
 
 import warnings
+from typing import Union
 
 from .exceptions import WorksheetNotFound
 from .urls import (
@@ -270,11 +271,11 @@ def get_worksheet(self, index):
         except (KeyError, IndexError):
             raise WorksheetNotFound("index {} not found".format(index))
 
-    def get_worksheet_by_id(self, id):
+    def get_worksheet_by_id(self, id: Union[str, int]):
         """Returns a worksheet with specified `worksheet id`.
 
         :param id: The id of a worksheet. it can be seen in the url as the value of the parameter 'gid'.
-        :type id: int
+        :type id: str | int
 
         :returns: an instance of :class:`gspread.worksheet.Worksheet`.
         :raises:
@@ -287,14 +288,19 @@ def get_worksheet_by_id(self, id):
         """
         sheet_data = self.fetch_sheet_metadata()
 
+        try:
+            worksheet_id_int = int(id)
+        except ValueError as ex:
+            raise ValueError("id should be int") from ex
+
         try:
             item = finditem(
-                lambda x: x["properties"]["sheetId"] == id,
+                lambda x: x["properties"]["sheetId"] == worksheet_id_int,
                 sheet_data["sheets"],
             )
             return Worksheet(self, item["properties"])
         except (StopIteration, KeyError):
-            raise WorksheetNotFound("id {} not found".format(id))
+            raise WorksheetNotFound("id {} not found".format(worksheet_id_int))
 
     def worksheets(self, exclude_hidden: bool = False):
         """Returns a list of all :class:`worksheets <gspread.worksheet.Worksheet>`
@@ -439,12 +445,16 @@ def del_worksheet(self, worksheet):
 
         return self.batch_update(body)
 
-    def del_worksheet_by_id(self, worksheet_id: str):
+    def del_worksheet_by_id(self, worksheet_id: Union[str, int]):
         """
         Deletes a Worksheet by id
         """
+        try:
+            worksheet_id_int = int(worksheet_id)
+        except ValueError as ex:
+            raise ValueError("id should be int") from ex
 
-        body = {"requests": [{"deleteSheet": {"sheetId": worksheet_id}}]}
+        body = {"requests": [{"deleteSheet": {"sheetId": worksheet_id_int}}]}
 
         return self.batch_update(body)