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

fix: Fix build error #12

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

fix: Fix build error #12

wants to merge 1 commit into from

Conversation

kuroneko0441
Copy link

  • files in src/app/utils are for tests, so it should be excluded from app build

Here is the build log without this commit.

> testing-patterns@0.0.0 build C:\angular-testing-recipes
> ng build

Browserslist: caniuse-lite is outdated. Please run next command `npm update`

Date: 2019-08-06T13:47:40.592Z
Hash: 3e9fc52d966f2ee82ad1
Time: 4670ms
chunk {main} main.js, main.js.map (main) 647 bytes [initial] [rendered]
chunk {polyfills} polyfills.js, polyfills.js.map (polyfills) 93.1 kB [initial] [rendered]
chunk {runtime} runtime.js, runtime.js.map (runtime) 6.08 kB [entry] [rendered]
chunk {styles} styles.js, styles.js.map (styles) 16.2 kB [initial] [rendered]

ERROR in src/app/utils/custom-matchers.ts(22,37): error TS2503: Cannot find namespace 'jasmine'.
src/app/utils/custom-matchers.ts(80,30): error TS2503: Cannot find namespace 'jasmine'.
src/app/utils/helpers.ts(71,10): error TS2304: Cannot find name 'expect'.
src/app/utils/helpers.ts(92,35): error TS2304: Cannot find name 'expect'.

npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR! testing-patterns@0.0.0 build: `ng build`
npm ERR! Exit status 1
npm ERR! 
npm ERR! Failed at the testing-patterns@0.0.0 build script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR! A complete log of this run can be found in:
npm ERR!     C:\Users\user\AppData\Roaming\npm-cache\_logs\2019-08-06T13_47_40_645Z-debug.log
``

- files in `src/app/utils` are for tests, so it should be excluded from app build
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.

None yet

1 participant