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

Combine the Node console reporters, update reporter config #1133

Open
jason0x43 opened this issue Apr 13, 2020 · 1 comment
Open

Combine the Node console reporters, update reporter config #1133

jason0x43 opened this issue Apr 13, 2020 · 1 comment
Labels
domain-reporters Related to Intern's built-in reporters effort-medium This may take a couple of days enhancement A new or improved feature priority-medium This should get done, but it's not a high priority

Comments

@jason0x43
Copy link
Member

jason0x43 commented Apr 13, 2020

Combine Runner, and Pretty into a single NodeConsole reporter, and drop Simple. The combined reporter should use Runner's output format by default (streaming text to the console), but should support Pretty's multi-session output format (erase screen, draw multiple progress bars) as an option. This reporter should always be loaded, and will not be dealt with by name; it is simply Intern's output.

Add a new "output" top-level config option to manage Intern's default output. It should have at least the following options:

{
  "output": {
    "disabled": true,      // false by default, disables the default reporter entirely
    "fullScreen": false,   // false by default, enables full screen mode for showing concurrent test sessions
    "hideSkipped": false,  // boolean or a string, like current Runner option
    "hidePassed": true,    // boolean or a string, like current Runner option
  }
}

See discussion in #1023.

@jason0x43 jason0x43 added enhancement A new or improved feature effort-medium This may take a couple of days domain-reporters Related to Intern's built-in reporters priority-medium This should get done, but it's not a high priority labels Apr 13, 2020
@jason0x43
Copy link
Member Author

The config updates are related to #1109.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
domain-reporters Related to Intern's built-in reporters effort-medium This may take a couple of days enhancement A new or improved feature priority-medium This should get done, but it's not a high priority
Projects
None yet
Development

No branches or pull requests

1 participant