Skip to content
This repository has been archived by the owner on Mar 4, 2022. It is now read-only.

Test: Add unit test coverage for all critical code paths. #9

Closed
1 of 2 tasks
ryan-roemer opened this issue Oct 28, 2015 · 0 comments
Closed
1 of 2 tasks

Test: Add unit test coverage for all critical code paths. #9

ryan-roemer opened this issue Oct 28, 2015 · 0 comments
Labels
enhancement

Comments

@ryan-roemer
Copy link
Member

  • Get coverage numbers way up
  • Tackle any remaining unimplemented tests and TODOs.
@ryan-roemer ryan-roemer changed the title Quality: Add unit tests for everything in lib/ Test: Add unit tests for everything in lib/ Dec 9, 2015
@ryan-roemer ryan-roemer mentioned this issue Feb 4, 2016
Closed
@ryan-roemer ryan-roemer changed the title Test: Add unit tests for everything in lib/ Test: Add unit test coverage for all critical code paths. Feb 4, 2016
ryan-roemer added a commit that referenced this issue Feb 4, 2016
@ryan-roemer
Add significantly more test coverage and various bug/doc fixes.
a6b9924 
Implementation work for #9

* Add comprephensive "almost" functional tests in `builder-core.spec.js` using `mock-fs` and a
  ton of actual usage scenarios.
* Capture JSON parsing errors for `builder envs` better.
* Fix bug wherein `builder --version` displayed help instead of version.
* Refactor `--setup` execution model to resolve and use a `Task` object straight up
  rather than a `builder run <setup-task-name>`.
* Refactor `config.js` and `log.js` to have stubable methods in tests.
* Enhance `builder-core` to take an options argument.
* Fix minor README.md typo.
@ryan-roemer ryan-roemer mentioned this issue Feb 4, 2016
Merged
ryan-roemer added a commit that referenced this issue Feb 4, 2016
@ryan-roemer
Add significantly more test coverage and various bug/doc fixes.
2f3a188 
Implementation work for #9

* Add comprephensive "almost" functional tests in `builder-core.spec.js` using `mock-fs` and a
  ton of actual usage scenarios.
* Capture JSON parsing errors for `builder envs` better.
* Fix bug wherein `builder --version` displayed help instead of version.
* Refactor Task to take `runner` option to avoid circular dependencies.
* Refactor `--setup` execution model to resolve and use a `Task` object straight up
  rather than a `builder run <setup-task-name>`.
* Refactor `config.js` and `log.js` to have stubable methods in tests.
* Enhance `builder-core` to take an options argument.
* Fix minor README.md typo.
ryan-roemer added a commit that referenced this issue Feb 4, 2016
@ryan-roemer
Add significantly more test coverage and various bug/doc fixes.
d86f11d 
Implementation work for #9

* Add comprehensive "almost" functional tests in `builder-core.spec.js` using `mock-fs` and a
  ton of actual usage scenarios.
* Capture JSON parsing errors for `builder envs` better.
* Fix bug wherein `builder --version` displayed help instead of version.
* Refactor Task to take `runner` option to avoid circular dependencies.
* Refactor `--setup` execution model to resolve and use a `Task` object straight up
  rather than a `builder run <setup-task-name>`.
* Refactor `config.js` and `log.js` to have stubable methods in tests.
* Enhance `builder-core` to take an options argument.
* Fix minor README.md typo.
ryan-roemer added a commit that referenced this issue Feb 4, 2016
@ryan-roemer
Add significantly more test coverage and various bug/doc fixes.
598bbb7 
Implementation work for #9

* Add comprehensive "almost" functional tests in `builder-core.spec.js` using `mock-fs` and a
  ton of actual usage scenarios.
* Capture JSON parsing errors for `builder envs` better.
* Fix bug wherein `builder --version` displayed help instead of version.
* Refactor Task to take `runner` option to avoid circular dependencies.
* Refactor `--setup` execution model to resolve and use a `Task` object straight up
  rather than a `builder run <setup-task-name>`.
* Refactor `config.js` and `log.js` to have stubable methods in tests.
* Enhance `builder-core` to take an options argument.
* Fix minor README.md typo.
ryan-roemer added a commit that referenced this issue Feb 5, 2016
@ryan-roemer
Add significantly more test coverage and various bug/doc fixes.
a652651 
Implementation work for #9

* Add comprehensive "almost" functional tests in `builder-core.spec.js` using `mock-fs` and a
  ton of actual usage scenarios.
* Capture JSON parsing errors for `builder envs` better.
* Fix bug wherein `builder --version` displayed help instead of version.
* Refactor Task to take `runner` option to avoid circular dependencies.
* Refactor `--setup` execution model to resolve and use a `Task` object straight up
  rather than a `builder run <setup-task-name>`.
* Refactor `config.js` and `log.js` to have stubable methods in tests.
* Enhance `builder-core` to take an options argument.
* Fix minor README.md typo.
@coopy coopy closed this as completed Mar 8, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ryan-roemer @coopy