fix: Handle np.bool_ in insert and fix download_path tests

- Add np.bool_ to isinstance checks in table.py for proper boolean
  handling during insert (line 536 validate, line 1172 placeholder)
- Fix test_attach.py and test_update1.py to use dj.config.override()
  instead of deprecated download_path argument

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <[email protected]>

Author: dimitri-yatsenko

src/datajoint/table.py

@@ -533,7 +533,7 @@ def validate(self, rows, *, ignore_extra_fields=False) -> ValidationResult:
                         continue
 
                 # Numeric NaN check
-                if attr.numeric and value != "" and not isinstance(value, bool):
+                if attr.numeric and value != "" and not isinstance(value, (bool, np.bool_)):
                     try:
                         if np.isnan(float(value)):
                             # NaN is allowed - will be converted to NULL
@@ -1169,7 +1169,7 @@ def __make_placeholder(self, name, value, ignore_extra_fields=False, row=None):
                 value = json.dumps(value)
             # Numeric - convert to string
             elif attr.numeric:
-                value = str(int(value) if isinstance(value, bool) else value)
+                value = str(int(value) if isinstance(value, (bool, np.bool_)) else value)
             # Blob - pass through as bytes (use <blob> for automatic serialization)
 
         return name, placeholder, value

tests/integration/test_attach.py

@@ -7,6 +7,8 @@
 
 def test_attach_attributes(schema_ext, minio_client, tmpdir_factory):
     """Test saving files in attachments"""
+    import datajoint as dj
+
     # create a mock file
     table = Attach()
     source_folder = tmpdir_factory.mktemp("source")
@@ -23,27 +25,31 @@ def test_attach_attributes(schema_ext, minio_client, tmpdir_factory):
 
     download_folder = Path(tmpdir_factory.mktemp("download"))
     keys = table.keys(order_by="KEY")
-    path1, path2 = table.to_arrays("img", "txt", download_path=download_folder, order_by="KEY")
 
-    # verify that different attachment are renamed if their filenames collide
-    assert path1[0] != path2[0]
-    assert path1[0] != path1[1]
-    assert Path(path1[0]).parent == download_folder
-    with Path(path1[-1]).open("rb") as f:
-        check1 = f.read()
-    with Path(path2[-1]).open("rb") as f:
-        check2 = f.read()
-    assert data1 == check1
-    assert data2 == check2
+    with dj.config.override(download_path=str(download_folder)):
+        path1, path2 = table.to_arrays("img", "txt", order_by="KEY")
+
+        # verify that different attachment are renamed if their filenames collide
+        assert path1[0] != path2[0]
+        assert path1[0] != path1[1]
+        assert Path(path1[0]).parent == download_folder
+        with Path(path1[-1]).open("rb") as f:
+            check1 = f.read()
+        with Path(path2[-1]).open("rb") as f:
+            check2 = f.read()
+        assert data1 == check1
+        assert data2 == check2
 
-    # verify that existing files are not duplicated if their filename matches issue #592
-    p1, p2 = (Attach & keys[0]).fetch1("img", "txt", download_path=download_folder)
-    assert p1 == path1[0]
-    assert p2 == path2[0]
+        # verify that existing files are not duplicated if their filename matches issue #592
+        p1, p2 = (Attach & keys[0]).fetch1("img", "txt")
+        assert p1 == path1[0]
+        assert p2 == path2[0]
 
 
 def test_return_string(schema_ext, minio_client, tmpdir_factory):
     """Test returning string on fetch"""
+    import datajoint as dj
+
     # create a mock file
     table = Attach()
     source_folder = tmpdir_factory.mktemp("source")
@@ -59,6 +65,7 @@ def test_return_string(schema_ext, minio_client, tmpdir_factory):
     table.insert1(dict(attach=2, img=attach1, txt=attach2))
 
     download_folder = Path(tmpdir_factory.mktemp("download"))
-    path1, path2 = table.to_arrays("img", "txt", download_path=download_folder, order_by="KEY")
+    with dj.config.override(download_path=str(download_folder)):
+        path1, path2 = table.to_arrays("img", "txt", order_by="KEY")
 
     assert isinstance(path1[0], str)

tests/integration/test_update1.py

@@ -86,7 +86,8 @@ def test_update1(tmpdir, schema_update1, mock_stores_update):
     # Insert the relative path within the store
     Thing.update1(dict(key, img_file=f"{relpath}/{filename}"))
 
-    check2 = Thing.fetch1(download_path=tmpdir)
+    with dj.config.override(download_path=str(tmpdir)):
+        check2 = Thing.fetch1()
     buffer2 = Path(check2["picture"]).read_bytes()  # read attachment
     # For filepath, fetch returns ObjectRef - read the file through it
     filepath_ref = check2["img_file"]