fixed printing metasteps

Author: DavertMik

lib/container.js

@@ -274,6 +274,16 @@ function createSupportObjects(config) {
       {},
       {
         get(target, prop) {
+          // behavr like array or
+          if (prop === 'length') return Object.keys(config).length
+          if (prop === Symbol.iterator) {
+            return function* () {
+              for (let i = 0; i < Object.keys(config).length; i++) {
+                yield target[i]
+              }
+            }
+          }
+
           // load actual name from vocabulary
           if (container.translation.name) {
             name = container.translation.name
@@ -304,10 +314,10 @@ function createSupportObjects(config) {
           let currentValue = currentObject[prop]
 
           if (isFunction(currentValue) || isAsyncFunction(currentValue)) {
-            const ms = new MetaStep(name, currentValue)
+            const ms = new MetaStep(name, prop)
             ms.setContext(currentObject)
             if (isAsyncFunction(currentValue)) currentValue = asyncWrapper(currentValue)
-            currentObject[prop] = ms.run.bind(ms, currentValue)
+            return ms.run.bind(ms, currentValue)
           }
 
           return currentValue
@@ -317,8 +327,14 @@ function createSupportObjects(config) {
           return prop in container.support[name]
         },
         ownKeys() {
+          const keys = []
+          // Add numeric indices
+          for (let i = 0; i < Object.keys(config).length; i++) {
+            keys.push(String(i))
+          }
+          // Add other properties
           container.support[name] = container.support[name] || loadSupportObject(config[name])
-          return Reflect.ownKeys(container.support[name])
+          return [...keys, ...Reflect.ownKeys(container.support[name])]
         },
       },
     )
@@ -350,8 +366,6 @@ function createActor(actorPath) {
     container.support.I = actor()
   }
 
-  container.support.I = container.support.I
-
   return container.support.I
 }
 
@@ -420,12 +434,16 @@ function loadSupportObject(modulePath, supportObjectName) {
     if (typeof obj === 'function') {
       // If it's a class (constructor function)
       if (obj.prototype && obj.prototype.constructor === obj) {
-        const ClassName = obj;
-        return new ClassName();
+        const ClassName = obj
+        return new ClassName()
       }
       // If it's a regular function
       return obj()
     }
+    if (obj && Array.isArray(obj)) {
+      return obj
+    }
+
     // If it's a plain object
     if (obj && typeof obj === 'object' && !Array.isArray(obj)) {
       return obj

test/data/inject-fail-example/pages/page.js

@@ -1,15 +1,15 @@
-const { notpage, arraypage } = inject();
+const { notpage, arraypage } = inject()
 
 module.exports = {
   type: (s) => {
-    console.log('type => ', s);
-    console.log('strategy', arraypage);
-    notpage.domainIds.push('newdomain');
-    return notpage.domainIds;
+    console.log('type => ', s)
+    console.log('strategy', arraypage.toString())
+    notpage.domainIds.push('newdomain')
+    return notpage.domainIds
   },
 
   purgeDomains: (s) => {
-    console.log('purgeDomains');
-    console.log(s);
+    console.log('purgeDomains')
+    console.log(s)
   },
-};
+}

test/runner/pageobject_test.js

@@ -157,7 +157,7 @@ describe('CodeceptJS PageObject', () => {
     it('should inject page objects via proxy', (done) => {
       exec(`${config_run_config('../../../inject-fail-example')} --debug`, (err, stdout) => {
         expect(stdout).toContain('newdomain')
-        expect(stdout).toContain("[ 'veni', 'vedi', 'vici' ]", 'array objects work')
+        expect(stdout).toContain('veni,vedi,vici')
         expect(stdout).toContain('OK  | 1 passed')
         expect(err).toBeFalsy()
         done()