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

Discussion of the spec #1

Open
dignifiedquire opened this issue Jan 4, 2013 · 2 comments
Open

Discussion of the spec #1

dignifiedquire opened this issue Jan 4, 2013 · 2 comments

Comments

@dignifiedquire
Copy link
Owner

I've created this repo to discuss and create the protocol we've talked about.

References

@airportyh
Copy link

Thanks @dignifiedquire. Here are my initial thoughts.

Extending TAP

To extend TAP we will need to get the TAP authors into the discussion too. But maybe we can simply use indentation to denote test suite hierarchy like Mocha's "spec" reporter already does. Something like

Config
    ok 1 can create
    ok 2 gives progOptions properties when got
    read yaml config file
        ok 3 gets properties from config file
        ok 4 falls back to config file value when progOptions is null
    read json config file
        ok 5 gets properties from config file
FileWatcher
    ok 6 should add
    ok 7 should ignore(not blow up) if watched file does not exist

Then for extra info like tags, we can just use yamlish which is already in the spec.

Extending TAP #2

Or we could extend TAP without changing the spec, but rather just by layering. One way is to use breadcrumbs to denote hierarchy

ok 1 Config > can create
ok 2 Config > gives progOptions properties when got
ok 3 Config > read yaml config file > gets properties from config file
ok 4 Config > read yaml config file > falls back to config file value when progOptions is null
ok 5 Config > read json config file > gets properties from config file
ok 6 FileWatcher > should add
ok 7 FileWatcher > should ignore(not blow up) if watched file does not exist

I am short on time so I'll save my thoughts for JSON stream for another post.

@Raynos
Copy link

Raynos commented Jan 5, 2013

cc @isaacs @substack

If tap were to be extended then tap consumers would need to be able to not puke on parsing the extension. Both @substack and @isaacs write tap consumers / parsers

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

No branches or pull requests

3 participants