ciscoheat / buddy

Your friendly BDD testing library for Haxe!
MIT License
96 stars 24 forks source link

Multi-argument traces don't work in tests #69

Closed Gama11 closed 6 years ago

Gama11 commented 6 years ago
class Main extends buddy.SingleSuite {
    function new() {
        trace(1, 2); // Main.hx:3: 1, 2

        describe("foo", {
            it("bar", {
                trace(1, 2); // Main.hx:7: 1
            });
        });
    }
}

Buddy's custom trace implementation just seems to throw additional arguments away, instead of printing them comma-separated as expected.

ciscoheat commented 6 years ago

Thank you! Fixed, and haxelib updated.

Gama11 commented 6 years ago

The formatting is slightly different still - Haxe puts a space between the arguments. ;)

1, 2 vs 1,2.

ciscoheat commented 6 years ago

No it doesn't (check the Haxe source and you'll see, also try.haxe.org), what version and target are you using?

Den mån 4 dec. 2017 11:32Jens Fischer notifications@github.com skrev:

The formatting is slightly different still - Haxe puts a space between the arguments. ;)

1, 2 vs 1,2.

— You are receiving this because you modified the open/close state.

Reply to this email directly, view it on GitHub https://github.com/ciscoheat/buddy/issues/69#issuecomment-348922497, or mute the thread https://github.com/notifications/unsubscribe-auth/AAH9kl9J0bib4FK71ISee9SNJrveoGn2ks5s88pBgaJpZM4QzRwc .

Gama11 commented 6 years ago

Huh, interesting.. I'm using the latest development branch build, seems the behavior has changed. See http://try-haxe.mrcdk.com/#A3E12, and try selecting one of the development releases in the Haxe version dropdown.