throw TypeError if str or bytes % operator gets too many arguments

Author: tomasstupka

graalpython/com.oracle.graal.python.test/src/tests/test_bytes.py

@@ -706,6 +706,15 @@ def test_bytes_init():
     bytes.__init__(ba, b'zzz')
     assert ba == bytes(b'abc')
 
+def test_bytes_mod():
+    assert b'%s' % (b'a') == b'a'
+    raised = False
+    try:
+        b'%s' % (b'a', b'b') 
+    except TypeError:
+        raised = True
+    assert raised
+    
 class BaseLikeBytes:
 
     def test_maketrans(self):

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/runtime/formatting/FormatProcessor.java

@@ -490,7 +490,7 @@ private T formatImpl(Object args1) {
          * of range; if a special value, it would be wrong if it were -1, indicating a single item
          * that has not yet been used.
          */
-        if (argIndex == -1 || (argIndex >= 0 && PythonObjectLibrary.getUncached().length(args1) > argIndex + 1)) {
+        if (argIndex == -1 || (argIndex >= 0 && PythonObjectLibrary.getUncached().length(args1) >= argIndex + 1)) {
             throw core.raise(TypeError, ErrorMessages.NOT_ALL_ARGS_CONVERTED_DURING_FORMATTING, getFormatType());
         }