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

Test plan #581

Closed
17 tasks done
SpaceK33z opened this issue Sep 1, 2016 · 1 comment
Closed
17 tasks done

Test plan #581

SpaceK33z opened this issue Sep 1, 2016 · 1 comment

Comments

@SpaceK33z
Copy link
Member

SpaceK33z commented Sep 1, 2016

At the moment there are no tests. This makes it difficult to release without unintended breaking changes, and makes PR's very difficult and hard to review.

To start, I think we need to have examples for all features. Right now there are two basic examples, but they don't touch only a part of the code base.

With examples, it's easy to verify if something still works as intended. It's also something people can learn from. Later, we could automate these tests. Automating tests is not something I want to do initially, because this will be fairly difficult with so much moving components.

I think we need to have examples of at least the following features:

  • HMR
  • https
  • proxy - basic config
  • proxy - advanced config
  • proxy - bypass feature
  • lazy mode
  • inline modus disabled
  • iframe mode
  • gzip compression
  • history api fallback
  • array of webpack configs
  • custom host and port
  • Node API
  • Node - add middleware with setup()
  • CLI - --open
  • CLI - --stdin
  • CLI - --public

An example should consist of:

  • The code that is necessary
  • Instructions on how to run
  • A description of what should happen

Feedback welcome!

@SpaceK33z SpaceK33z mentioned this issue Sep 1, 2016
SpaceK33z added a commit that referenced this issue Sep 3, 2016
SpaceK33z added a commit that referenced this issue Sep 3, 2016
SpaceK33z added a commit that referenced this issue Sep 3, 2016
SpaceK33z added a commit that referenced this issue Sep 3, 2016
@SpaceK33z
Copy link
Member Author

Pfff, finally done :).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant