Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix projection.selectMany test. #27

Closed
wants to merge 1 commit into from
Closed

Fix projection.selectMany test. #27

wants to merge 1 commit into from

Conversation

M0ns1gn0r
Copy link
Collaborator

The test used to fail with the following error:

Module: Projection Test: selectMany
Died on test #1     at Object.<anonymous> (X:\linq\test\projection.js:49:1)
    at Module._compile (module.js:570:32)
    at Object.Module._extensions..js (module.js:579:10)
    at Module.load (module.js:487:32)
    at tryModuleLoad (module.js:446:12)
    at Function.Module._load (module.js:438:3)
    at Module.require (module.js:497:17)
    at require (internal/module.js:20:19): Enumerable is not defined
ReferenceError: Enumerable is not defined
    at eval (eval at createLambda (X:\linq\linq.min.js:1:552), <anonymous>:3:8)
    at D.<anonymous> (X:\linq\linq.min.js:17:149)
    at g.moveNext (X:\linq\linq.min.js:3:19)
    at d.forEach (X:\linq\linq.min.js:47:466)
    at d.toArray (X:\linq\linq.min.js:45:208)
    at Object.<anonymous> (X:\linq\test\projection.js:50:77)
    at Object.run (X:\linq\node_modules\qunit\support\qunit\qunit\qunit.js:136:18)
    at X:\linq\node_modules\qunit\support\qunit\qunit\qunit.js:279:10
    at process (X:\linq\node_modules\qunit\support\qunit\qunit\qunit.js:1235:24)
    at Timeout._onTimeout (X:\linq\node_modules\qunit\support\qunit\qunit\qunit.js:378:5)

@mihaifm
Copy link
Owner

mihaifm commented Feb 3, 2017

I'll get this in this evening, sorry for the delay

@mihaifm
Copy link
Owner

mihaifm commented Feb 3, 2017

The test can also be fixed by replacing "Enumerable" with "this" in the lambdas. I added more assertions to include your approach as well.

thanks

@mihaifm mihaifm closed this Feb 3, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants