Skip to content
This repository was archived by the owner on Jul 29, 2026. It is now read-only.

test(gen): make tests more strict and DRY - #531

Merged
DaftMonk merged 1 commit into
angular-fullstack:canaryfrom
kingcody:refactor/gen-tests
Sep 10, 2014
Merged

test(gen): make tests more strict and DRY#531
DaftMonk merged 1 commit into
angular-fullstack:canaryfrom
kingcody:refactor/gen-tests

Conversation

@kingcody

@kingcody kingcody commented Sep 7, 2014

Copy link
Copy Markdown
Member

Changes:

  • genFiles(ops) creates an array of expected files based on options given
  • test for generated files on every set of tests
  • test for unexpected files
  • use testExec to be more DRY

Exposes a bug with client/components/socket:
socket.service.js and socket.mock.js are not filtered.
See #530

@kingcody

kingcody commented Sep 7, 2014

Copy link
Copy Markdown
Member Author

This PR is expected to fail currently. It exposes an issue that was closed in master by #497 and an issue that will be closed by #530.

My hope is to be able to flush out such bugs in the future with the changes this PR implements.

Changes:
- `genFiles(ops)` creates an array of expected files based on options given
- test for generated files on every set of tests
- test for unexpected files
- use testExec to be more DRY

Exposes a bug with `client/components/socket`:
`socket.service.js` and `socket.mock.js` are not filtered.
See angular-fullstack#530

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

added default to switch

DaftMonk added a commit that referenced this pull request Sep 10, 2014
test(gen): make tests more strict and DRY
@DaftMonk
DaftMonk merged commit c16b3c9 into angular-fullstack:canary Sep 10, 2014
@kingcody
kingcody deleted the refactor/gen-tests branch September 10, 2014 09:09
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants