mirror of
				https://github.com/encode/django-rest-framework.git
				synced 2025-11-04 09:57:55 +03:00 
			
		
		
		
	improve performance for noncallble attributes (#8502)
Co-authored-by: Dima Kryukov <dmitry.kryukov@pandadoc.com>
This commit is contained in:
		
							parent
							
								
									292ead1fe0
								
							
						
					
					
						commit
						281fc074ba
					
				| 
						 | 
					@ -62,6 +62,9 @@ def is_simple_callable(obj):
 | 
				
			||||||
    """
 | 
					    """
 | 
				
			||||||
    True if the object is a callable that takes no arguments.
 | 
					    True if the object is a callable that takes no arguments.
 | 
				
			||||||
    """
 | 
					    """
 | 
				
			||||||
 | 
					    if not callable(obj):
 | 
				
			||||||
 | 
					        return False
 | 
				
			||||||
 | 
					
 | 
				
			||||||
    # Bail early since we cannot inspect built-in function signatures.
 | 
					    # Bail early since we cannot inspect built-in function signatures.
 | 
				
			||||||
    if inspect.isbuiltin(obj):
 | 
					    if inspect.isbuiltin(obj):
 | 
				
			||||||
        raise BuiltinSignatureError(
 | 
					        raise BuiltinSignatureError(
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
| 
						 | 
					@ -75,6 +75,10 @@ class TestIsSimpleCallable:
 | 
				
			||||||
        assert is_simple_callable(valid_vargs_kwargs)
 | 
					        assert is_simple_callable(valid_vargs_kwargs)
 | 
				
			||||||
        assert not is_simple_callable(invalid)
 | 
					        assert not is_simple_callable(invalid)
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					    @pytest.mark.parametrize('obj', (True, None, "str", b'bytes', 123, 1.23))
 | 
				
			||||||
 | 
					    def test_not_callable(self, obj):
 | 
				
			||||||
 | 
					        assert not is_simple_callable(obj)
 | 
				
			||||||
 | 
					
 | 
				
			||||||
    def test_4602_regression(self):
 | 
					    def test_4602_regression(self):
 | 
				
			||||||
        from django.db import models
 | 
					        from django.db import models
 | 
				
			||||||
 | 
					
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
		Loading…
	
		Reference in New Issue
	
	Block a user