Bug in Collections/Compound tasks?

ttoomm318 commented 6 years ago

There seems to be a bug with the test(s) for fun Shop.getNumberOfTimesProductWasOrdered(product: Product): Int

I've tried the the following two solutions:

Provided solution:

fun Shop.getNumberOfTimesProductWasOrdered(product: Product): Int {
    return customers.flatMap { it.getOrderedProductsList() }.count { it == product }
}

fun Customer.getOrderedProductsList(): List<Product> {
    return orders.flatMap { it.products }
}

My original solution:

fun Shop.getNumberOfTimesProductWasOrdered(product: Product): Int =
    customers.flatMap { it.orders }.flatMap { it.products }.count { it == product }

at the following two sites:

and I get

Assigning single elements to varargs in named form is forbidden

Just to add, I tried the same solution in the Playground at https://play.kotlinlang.org/ (using the same code from Shop.kt and simplified code from TestShop.kt) and it runs fine, giving the correct result.

I've attached the file with code copied from Playground. Kotlin Playground.kt.zip

antohaby commented 6 years ago

Got yesterday same issue. Because of that cant make 100% progress :(

It seems that it was fixed before https://github.com/Kotlin/kotlin-koans/commit/888c9a17a9130f69aa96a3f85014305591c22127 when it was just a deprecation.

EhsanKia commented 6 years ago

I just hit this now. The easy temporary solution is to change the runtime to something other than 1.3 in the bottom right corner.

ivannarino commented 5 years ago

The bug is still there

davidgiessing commented 5 years ago

Just hit the bug. It is just the testcase that is (now) wrong since it tries to assign a single value in named form to a collection. However @EhsanKia s temporary workaround does indeed work.