Sep 23, 2021 · Dec 6, 2020
diff --git a/src/runtime/extensiontype.cs b/src/runtime/extensiontype.cs
            return -1;
        }


        /// <summary>
        /// Default __set__ implementation - this prevents descriptor instances
        /// being silently replaced in a type __dict__ by default __setattr__.
        /// </summary>
        public static int tp_descr_set(IntPtr ds, IntPtr ob, IntPtr val)
        {
            Exceptions.SetError(Exceptions.AttributeError, "attribute is read-only");
            return -1;
        }


        public static void tp_dealloc(IntPtr ob)
        {
            // Clean up a Python instance of this extension type. This
diff --git a/src/testing/methodtest.cs b/src/testing/methodtest.cs
        {
        }

        public string OverwritableMethod() => "overwritable";

        public string PublicMethod()
        {
            return "public";
diff --git a/tests/test_method.py b/tests/test_method.py
 import pytest
 from Python.Test import MethodTest

 def test_instance_method_overwritable():
    """Test instance method overwriting."""

 def test_instance_method_descriptor():
    """Test instance method descriptor behavior."""

    with pytest.raises(AttributeError):
        MethodTest().PublicMethod = 0

    with pytest.raises(AttributeError):
        MethodTest.PublicMethod = 0

    with pytest.raises(AttributeError):
        del MethodTest().PublicMethod

    with pytest.raises(AttributeError):
        del MethodTest.PublicMethod


 def test_static_method_descriptor():
    """Test static method descriptor behavior."""

    with pytest.raises(AttributeError):
        MethodTest().PublicStaticMethod = 0

    with pytest.raises(AttributeError):
        MethodTest.PublicStaticMethod = 0

    with pytest.raises(AttributeError):
        del MethodTest().PublicStaticMethod

    with pytest.raises(AttributeError):
        del MethodTest.PublicStaticMethod
    ob = MethodTest()
    ob.OverwritableMethod = lambda: "overwritten"
    assert ob.OverwritableMethod() == "overwritten"


 def test_public_instance_method():
Original file line number	Diff line number	Diff line change
Expand Up		@@ -87,18 +87,6 @@ public static int tp_setattro(IntPtr ob, IntPtr key, IntPtr val)
		return -1;
		}


		/// <summary>
		/// Default __set__ implementation - this prevents descriptor instances
		/// being silently replaced in a type __dict__ by default __setattr__.
		/// </summary>
		public static int tp_descr_set(IntPtr ds, IntPtr ob, IntPtr val)
		{
		Exceptions.SetError(Exceptions.AttributeError, "attribute is read-only");
		return -1;
		}


		public static void tp_dealloc(IntPtr ob)
		{
		// Clean up a Python instance of this extension type. This
Expand Down
Original file line number	Diff line number	Diff line change
Expand Up		@@ -14,6 +14,8 @@ public MethodTest()
		{
		}

		public string OverwritableMethod() => "overwritable";

		public string PublicMethod()
		{
		return "public";
Expand Down
Original file line number	Diff line number	Diff line change
Expand Up		@@ -6,37 +6,12 @@
		import pytest
		from Python.Test import MethodTest

		def test_instance_method_overwritable():
		"""Test instance method overwriting."""

		def test_instance_method_descriptor():
		"""Test instance method descriptor behavior."""

		with pytest.raises(AttributeError):
		MethodTest().PublicMethod = 0

		with pytest.raises(AttributeError):
		MethodTest.PublicMethod = 0

		with pytest.raises(AttributeError):
		del MethodTest().PublicMethod

		with pytest.raises(AttributeError):
		del MethodTest.PublicMethod


		def test_static_method_descriptor():
		"""Test static method descriptor behavior."""

		with pytest.raises(AttributeError):
		MethodTest().PublicStaticMethod = 0

		with pytest.raises(AttributeError):
		MethodTest.PublicStaticMethod = 0

		with pytest.raises(AttributeError):
		del MethodTest().PublicStaticMethod

		with pytest.raises(AttributeError):
		del MethodTest.PublicStaticMethod
		ob = MethodTest()
		ob.OverwritableMethod = lambda: "overwritten"
		assert ob.OverwritableMethod() == "overwritten"


		def test_public_instance_method():
Expand Down