This repository has been archived by the owner on Nov 16, 2023. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 2.8k
removed chai and replaced it totally with jest #287
Open
nishat-sayyed
wants to merge
256
commits into
microsoft:master
Choose a base branch
from
nishat-sayyed:replace-chai-totally-with-jest
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
removed chai and replaced it totally with jest #287
nishat-sayyed
wants to merge
256
commits into
microsoft:master
from
nishat-sayyed:replace-chai-totally-with-jest
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Update references from bowdenk7 repo to Microsoft repo
added package-lock for npm5 and fixed bug with flash messages
Added `build` `preLaunchTask` for debug configuration
Removed duplication in npm scripts `serve` and `watch`
…MaxOptions'. fix err TypeScript-Node-Starter/src/controllers/api.ts[6, 9]: missing whitespace, 8 files fix bugs of issue 15
fit bugs of issue 15 and some other bugs caused by tslint
port changes from PR microsoft#11 and update package.lock from latest npm version
fix: copying issues
Bump tasks.json version from 0.10 to 2.00. Add default build task.
…added recommended extensions using vs code's extensions.json
…me issues with CI
Use update indexes access in Mongoose configuration. Fixes microsoft#225
- upgrade to latest packages - cast `req.user` as `UserDocument` - update logger to use latest Winston syntax - disable `no-inferrable-types`, Mongoose use unified topology Thx to @CarlosSolrac
ESLint, no any, no inferrable types, unused vars - warning thx to @GrayStrider
…e Mac OS Document use of the home directory on macOS 10.15 and above Thanks to @memasdeligeorgakis
- remove @types/dotenv - dotenv 8.1.0 > 8.2.0 - @types/connect-mongo 0.0.43 > 3.0.0 - concurrently 4.1.2 > 5.0.0 Thanks!
Update node-sass to enable it to compile on the Mac when using Node v13.2.0 Thanks to @CarlosKlapp
This updates Jest and its dependencies, removing the warning about the left-pad module deprecation. Thanks!
…ocumentation in README.md
…precated vscode-eslint v2 settings patch
Added SendGrid Username, Password, Port to .env.example
Added GitHub Actions CI Workflow
- `request` - `request-promise` `request` first appeared as part of initial commit but it seems to be left-over from non commited implementation: https://git.io/JvgNV `request-promise` was added in different PR but it seems without related code: https://git.io/JvgN6 /cc @naknode Thanks!
➖ remove unused dependencies. Closes microsoft#272
Bumps [mongoose](https://github.com/Automattic/mongoose) from 5.7.1 to 5.7.5. - [Release notes](https://github.com/Automattic/mongoose/releases) - [Changelog](https://github.com/Automattic/mongoose/blob/master/History.md) - [Commits](Automattic/mongoose@5.7.1...5.7.5) Signed-off-by: dependabot[bot] <[email protected]>
Update README.md to frame what this repo is
Can you, please, revert changes with white-space applied? It looks the semicolons were removed as well, not only unifiying the spaces. See ESLint config:
Not sure why it fails on CI, for me it works OK locallly. npm run lint |
Thanks @peterblazejewicz I'll try that once I am free. |
brittanydrandolph
force-pushed
the
master
branch
from
June 24, 2022 15:41
fbc1328
to
1e15e8b
Compare
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The chai testing framework is not required as Jest comes bundled with
expect
and other utility methods.Following two files were using chai
Also removed chai from package.json and added
"include": ["src/**/*", "test/**/*"]
in tsconfig.json to support Typescript in the test directory.All test cases are passing after the change as you can see in the attachment