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

Move tests to plugins #592

Merged
merged 42 commits into from
Sep 2, 2018
Merged

Conversation

hipstersmoothie
Copy link
Collaborator

@hipstersmoothie hipstersmoothie commented Sep 2, 2018

What's Changing and Why

  • move all tests to related plugins
  • factor out testing utils into own package
  • move browser testing to root of repo so it can run all the tests

closes #577

What else might be affected

Tasks

  • Add tests
  • Update Documentation
  • Update jimp.d.ts
  • Add SemVer Label

@hipstersmoothie hipstersmoothie added the internal Changes only affect the internal API label Sep 2, 2018
@hipstersmoothie hipstersmoothie merged commit 969e26f into jimp-dev:master Sep 2, 2018
turakvlad pushed a commit to turakvlad/jimp that referenced this pull request Sep 3, 2018
* create test utils, move print's test to print

* move browser testing to root

* move autocrop test

* these can just be test

* move blit test

* move color tests

* move and split cover/contain

* move mask tests

* move normalize test

* move resize test

* move rotate tests

* convert jimp to use test-utils

* move test utils

* now resides in test-utils

* must configure jimp with plugins for these functions

* test node no more

* move some tests to core

* these two things are tested alot elsewhere

* rename samples to images

* split up fileTypes test

* Revert "move some tests to core"

d4fa37b

* fix circular deps

* fix lint

* fix dirnmae

* correct image path

* move image

* correct path

* babel requires node 6.9

* correct name in workflow

* same dependancies are installed regardless of node version

* maybe just attach to old directory

* change working directory. seems like the install was needed

* try increasing timeout

* maybe sync will pass?

* increase timeout for CI

* add result image

* correct path

* get tests passing

* better test output

* see if you can run parallel
turakvlad added a commit to turakvlad/jimp that referenced this pull request Sep 3, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
internal Changes only affect the internal API
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Renovate testing
1 participant