Skip to content

Fix tests#67

Open
thejsj wants to merge 7 commits intomasterfrom
add-org-id-and-is-personal-account
Open

Fix tests#67
thejsj wants to merge 7 commits intomasterfrom
add-org-id-and-is-personal-account

Conversation

@thejsj
Copy link
Member

@thejsj thejsj commented Jun 27, 2017

What this PR does

  • Add isPersonallAccount and orgId to organization.allowed worker

Tests

  • Test with subscription create

@thejsj
Copy link
Member Author

thejsj commented Jun 29, 2017

Locally:

cream_1   |
cream_1   |
cream_1   |   327 passing (1m)
cream_1   |   8 pending
cream_1   |
cream_1   | =============================================================================
cream_1   | Writing coverage object [/app/coverage/coverage.json]
cream_1   | Writing coverage reports at [/app/coverage]
cream_1   | =============================================================================
cream_1   |
cream_1   | =============================== Coverage summary ===============================
cream_1   | Statements   : 100% ( 925/925 )
cream_1   | Branches     : 100% ( 145/145 )
cream_1   | Functions    : 100% ( 139/139 )
cream_1   | Lines        : 100% ( 916/916 )
cream_1   | ================================================================================
cream_1   | npm info it worked if it ends with ok
cream_1   | npm info using npm@2.8.3
cream_1   | npm info using node@v4.2.1
cream_1   | npm info precoverage-check cream@1.14.0
cream_1   | npm info coverage-check cream@1.14.0
cream_1   |
cream_1   | > cream@1.14.0 coverage-check /app
cream_1   | > istanbul check-coverage --statement 99 --functions 99 --branches 99 --lines 99 && echo 'Coverage check successful!'
cream_1   |
cream_1   | Coverage check successful!
cream_1   | npm info postcoverage-check cream@1.14.0
cream_1   | npm info ok
cream_1   | npm info postcoverage cream@1.14.0
cream_1   | npm info ok
cream_1   | npm info posttest cream@1.14.0
cream_1   | npm info ok
cream_cream_1 exited with code 0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant