Add init for array.rs (RustPython#4667)

Author: minhrongcon2000

Lib/test/test_array.py

@@ -1114,8 +1114,6 @@ def test_bug_782369(self):
             b = array.array('B', range(64))
         self.assertEqual(rc, sys.getrefcount(10))
 
-    # TODO: RUSTPYTHON
-    @unittest.expectedFailure
     def test_subclass_with_kwargs(self):
         # SF bug #1486663 -- this used to erroneously raise a TypeError
         ArraySubclassWithKwargs('b', newarg=1)

stdlib/src/array.rs

@@ -46,7 +46,9 @@ mod array {
             },
             class_or_notimplemented,
             convert::{ToPyObject, ToPyResult, TryFromObject},
-            function::{ArgBytesLike, ArgIntoFloat, ArgIterable, OptionalArg, PyComparisonValue},
+            function::{
+                ArgBytesLike, ArgIntoFloat, ArgIterable, KwArgs, OptionalArg, PyComparisonValue,
+            },
             protocol::{
                 BufferDescriptor, BufferMethods, BufferResizeGuard, PyBuffer, PyIterReturn,
                 PyMappingMethods, PySequenceMethods,
@@ -644,18 +646,25 @@ mod array {
     }
 
     impl Constructor for PyArray {
-        type Args = ArrayNewArgs;
+        type Args = (ArrayNewArgs, KwArgs);
 
         fn py_new(
             cls: PyTypeRef,
-            Self::Args { spec, init }: Self::Args,
+            (ArrayNewArgs { spec, init }, kwargs): Self::Args,
             vm: &VirtualMachine,
         ) -> PyResult {
             let spec = spec.as_str().chars().exactly_one().map_err(|_| {
                 vm.new_type_error(
                     "array() argument 1 must be a unicode character, not str".to_owned(),
                 )
             })?;
+
+            if cls.is(PyArray::class(vm)) && !kwargs.is_empty() {
+                return Err(
+                    vm.new_type_error("array.array() takes no keyword arguments".to_owned())
+                );
+            }
+
             let mut array =
                 ArrayContentType::from_char(spec).map_err(|err| vm.new_value_error(err))?;
 

vm/src/function/argument.rs

@@ -328,6 +328,10 @@ impl<T> KwArgs<T> {
     pub fn pop_kwarg(&mut self, name: &str) -> Option<T> {
         self.0.remove(name)
     }
+
+    pub fn is_empty(self) -> bool {
+        self.0.is_empty()
+    }
 }
 impl<T> FromIterator<(String, T)> for KwArgs<T> {
     fn from_iter<I: IntoIterator<Item = (String, T)>>(iter: I) -> Self {