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/issue 428 refactor smoke test and cases #497

Merged

Conversation

thescientist13
Copy link
Member

@thescientist13 thescientist13 commented Mar 28, 2021

Related Issue

resolves test / TODOs for #428

Should be merged after #477

Summary of Changes

  1. Refactored smoke tests and applied to all tests
  2. Re-enabled coverage _nyc.config.js, will want to update this after Enhancement/issue 354 restore optimization config for no bundle #477 is merged
  3. Made all beforeEach -> before
  4. Identified a bug with plugin-graphql and <head></head> tag matching
  5. Added a missing test plugin-graphql for ChildrenQuery
  6. Cleaned up TOODs and added additional following items to Roadmap To 1.0 And Beyond #418 (most were already being tracked)
    • top level pages don't merge their own <title> tags (bug)
    • improve coverage coverage - get these branches up! and develop / serve tasks (+ server plugin / interface)
    • import from markdown (needs an issue)
    • default command should be build? (for npx)
    • pull assets copy from code?
    • extract front matter contents from remark-frontmatter instead of frontmatter lib
    • rollup - hardcoded paths to .greenwood/, public/, path references, lot of loops
    • path references (link commit in issue), also for <img> tag in header and banner component

@thescientist13 thescientist13 added the chore unit testing, maintenance, etc label Mar 28, 2021
This was referenced Apr 3, 2021
@thescientist13 thescientist13 force-pushed the chore/issue-428-refactor-smoke-test-and-cases branch from 3ac1de2 to b322ac0 Compare April 3, 2021 16:40
@thescientist13 thescientist13 merged commit 47f7d4d into release/0.10.0 Apr 3, 2021
@thescientist13 thescientist13 deleted the chore/issue-428-refactor-smoke-test-and-cases branch April 3, 2021 16:50
thescientist13 added a commit that referenced this pull request Apr 3, 2021
* refactor smoke test and re-enable

* cleanup dupe testing

* refactor beforeEach to before

* turn coverage checks back on

* delete invalid test cases

* xdescribe and xit

* fix smoke test tag matching and add smoke tests to ggraphql

* add more smoke testing

* add missing plugin-graphql ChildrenQuery spec

* cleanup TODOs for path references normalization

* remove TODOs that will be tracked via github

* clean up rollup todos

* restore local request blocking to puppeteer

* remove commented code from config

* final thresholds for now

* clean up missed TODOs and console logs
thescientist13 added a commit that referenced this pull request Apr 3, 2021
* refactor smoke test and re-enable

* cleanup dupe testing

* refactor beforeEach to before

* turn coverage checks back on

* delete invalid test cases

* xdescribe and xit

* fix smoke test tag matching and add smoke tests to ggraphql

* add more smoke testing

* add missing plugin-graphql ChildrenQuery spec

* cleanup TODOs for path references normalization

* remove TODOs that will be tracked via github

* clean up rollup todos

* restore local request blocking to puppeteer

* remove commented code from config

* final thresholds for now

* clean up missed TODOs and console logs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore unit testing, maintenance, etc
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant