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

chore(test): set up integration testing #461

Merged
merged 1 commit into from
Jan 14, 2021
Merged

chore(test): set up integration testing #461

merged 1 commit into from
Jan 14, 2021

Conversation

codymikol
Copy link
Owner

This will ensure that karma-webpack
can run as expected within a live
karma server.

Fixes #460

This PR contains a:

  • bugfix
  • new feature
  • code refactor
  • test update
  • typo fix
  • metadata update

Motivation / Use-Case

This will make it easier to update dependencies as it will ensure that karma-webpack will still work in at least a basic setup. This also lays groundwork for adding additional integration tests as well for more niche scenarios.

Breaking Changes

N/A

Additional Info

This also introduces puppeteer as a dev dependency as I found it to be the most best headless option for doing an integration test.

@codymikol codymikol added this to the 5.0.0 milestone Jan 9, 2021
// Explicitly turn off reporters so the simulated test results are not confused with the actual results.
reporters: [],
port: 2389,
logLevel: 'OFF',

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can this be the reason, why you don't see logs?

Copy link
Owner Author

@codymikol codymikol Jan 14, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was added right at the end to keep the tests from being noisy, unrelated to the issue I was having. I'll try to get a repro up sometime today.

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking at this made me realize that I totally missed pushing one of my files though 🤦‍♂️

@codymikol codymikol changed the title chore(test): set up integration testing WIP: chore(test): set up integration testing Jan 14, 2021
This will ensure that karma-webpack
can run as expected within a live
karma server.

Fixes #460
@codymikol codymikol changed the title WIP: chore(test): set up integration testing chore(test): set up integration testing Jan 14, 2021
@codymikol codymikol requested a review from ryanclark January 14, 2021 16:51
Copy link
Collaborator

@ryanclark ryanclark left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome stuff

@codymikol codymikol merged commit e8ad372 into codymikol:next Jan 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants