-
Notifications
You must be signed in to change notification settings - Fork 68
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
Adding Unit Test for Description Component #480
Adding Unit Test for Description Component #480
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good on the whole, just a few possible improvements and one clarification
tests/app/shared/issue/description/description.component.spec.ts
Outdated
Show resolved
Hide resolved
tests/app/shared/issue/description/description.component.spec.ts
Outdated
Show resolved
Hide resolved
tests/app/shared/issue/description/description.component.spec.ts
Outdated
Show resolved
Hide resolved
tests/app/shared/issue/description/description.component.spec.ts
Outdated
Show resolved
Hide resolved
tests/app/shared/issue/description/description.component.spec.ts
Outdated
Show resolved
Hide resolved
tests/app/shared/issue/description/description.component.spec.ts
Outdated
Show resolved
Hide resolved
tests/app/shared/issue/description/description.component.spec.ts
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can be merged once this typo is fixed
okay! Psps i clicked on the wrong button |
@seanlowjk could you re-trigger the Travis build (perhaps with an empty commit) ? |
Not too sure why the Travis build is failing, since it passed previously and the most recent change is trivial. |
okay sure, I think there is some problem with the Mac build in Travis. |
I have tried building the Mac version. It works. Either Travis is having some issue or it may be using the new Big Sur MacOS, causing the build to fail. |
* Add IsCategorical Tests * Add automated build for Linux and MacOS (CATcher-org#479) * Add Conflict Model Tests (CATcher-org#487) * Update usage instructions for Windows (CATcher-org#488) * Prevent unwanted caching by adding headers to API calls (CATcher-org#482) * Relax template correctness rule (CATcher-org#484) * Create logging (CATcher-org#481) * Update package.json * Direct import logger into main electron process (CATcher-org#489) * Update usage-notes.md * Update usage-notes.md * Update usage-notes.md * Fix Regex Checker and Tests * Fix for Copy Pasting of MacOS (CATcher-org#496) Edit Control Key Directives to include Mac * Fix Alt Tabbing Bug (CATcher-org#497) * Version Bump * Refactor Label Model * Adding Unit Test for Description Component (CATcher-org#480) * Adding Tests for Instance Methods for Issue Model (CATcher-org#495) Add Tests to the following methods: clone(), numOfUnresolvedDisputes(), createGithubTesterResponse(), createGithubTutorResponse(), createGithubTeamResponse(), createGithubIssueDescription() * Fix Pasting of images (CATcher-org#505) Used browser based paste event * CATcher version bump * Migrate CI to Github Actions from Appveyor and Travis (CATcher-org#509) - Migrate CI to Github Actions - Remove dependency on csv-parse * Update README.md * Fix Tests Names and Tests Grouping * Make Labelling Adjectives Clearer * Fix Typos Co-authored-by: Low Jun Kai, Sean <[email protected]> Co-authored-by: Low Jun Kai, Sean <[email protected]> Co-authored-by: Koh Jun Wei <[email protected]>
Summary
This fixes #468
Description
To be updated.