Excercise 11: Official solution fails due to incorrect test cases

[Windstalker]

module.exports = (Bacon, nidelva, leirelva, buttonClicked) =>
        nidelva
            .combine(leirelva, (leirelvaSample, nidelvaSample) => leirelvaSample + nidelvaSample)
            .sampledBy(buttonClicked);

✓ Should return an event stream ✓ Should make an EventStream from samples with the sum of the two rivers on click ✗ Should emit values every time button is clicked and give the sum of the two rivers on click

It seems last test expects return of Property, but not EventStream.

module.exports = (Bacon, nidelva, leirelva, buttonClicked) =>
        nidelva
            .combine(leirelva, (leirelvaSample, nidelvaSample) => leirelvaSample + nidelvaSample)
            .sampledBy(buttonClicked) 
            .toProperty();

[mikaelbr]

Hi. Thanks for the issue. It doesn't really expect a property as far as I can see (or remember), so there might be something else happening here. Needs to be investigated some.

[adrianmcli]

I cannot reproduce this bug, my version runs fine:

module.exports = (Bacon, nidelva, leirelva, buttonClicked) => {
  const combinedLevels = nidelva.combine(leirelva, (a, b) => a + b);
  return combinedLevels.sampledBy(buttonClicked);
};