Skip to content

Unittetsts - #9

Merged
bothadeon merged 20 commits into
masterfrom
unittetsts
Dec 19, 2018
Merged

bothadeon merged 20 commits into
masterfrom
unittetsts

Conversation

@bothadeon

Copy link
Copy Markdown
Member

No description provided.

@bothadeon

Copy link
Copy Markdown
Member Author

refer to pull request #10 replacing this .

@bothadeon bothadeon closed this Dec 19, 2018
@bothadeon bothadeon self-assigned this Dec 19, 2018
@bothadeon bothadeon reopened this Dec 19, 2018

@vgenev vgenev left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

remove the .idea/ folder from the PR and add it to .gitignore please

@bothadeon bothadeon left a comment

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.

.ide added to gitignore

@vgenev vgenev left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

happy

@bothadeon
bothadeon merged commit dbf22dc into master Dec 19, 2018
@vgenev
vgenev deleted the unittetsts branch January 25, 2019 08:04
vgenev added a commit that referenced this pull request May 10, 2019
* adding circleCI setup (#1) (#2)

* Develop (#8)

* adding circleCI setup (#1)

* message template changes (#3)

* message template changes

* added config to be loaded from environmental variables

* added healthcheck

* Added Dockerfile

* Add Dockerfile and dockerignore

* settings error fixed

* healthcheckport renamed to port

* kafka fix and log

* Added happy line testing to action observable

* Fix Chai deep euqal error

* Added code to do code coverage

* Added negative test case to the actions

* Start unit tests for templates/index.js

* changed /healtcheck to /health

* Added negative test for templates index

* fixed bug in the neagtive testing

* Added last unit test for templates/index.js

* changed version to 4.3.0

* Added two describe sections in the sendmailer unit test to increase test coverage

* Add unit test fixes for send mail

* changed code coverage to produce html report

* Add last changes before pull request

* Added .idea directory to gitignore

* Added a back slash after the directory name

* Ran git rm --cached .idea/workspace.xml to remove the .idea files

* changed to

* changed to .idea/*

* changed to .idea/*

* Unittetsts (#9)

* Added happy line testing to action observable

* Fix Chai deep euqal error

* Added code to do code coverage

* Added negative test case to the actions

* Start unit tests for templates/index.js

* Added negative test for templates index

* fixed bug in the neagtive testing

* Added last unit test for templates/index.js

* Added two describe sections in the sendmailer unit test to increase test coverage

* Add unit test fixes for send mail

* changed code coverage to produce html report

* Add last changes before pull request

* Added .idea directory to gitignore

* Added a back slash after the directory name

* Ran git rm --cached .idea/workspace.xml to remove the .idea files

* changed to

* changed to .idea/*

* changed to .idea/*

* refactor the mailer and unit tests. changed the logging of the kafka consumer

* WIP

* WIP

* wip

* wip

* wip

* setup only not working

* no marbles tape is used for the rest

* setup test added. not finished

* setup test with marbles

* setup tests. might need to be reworked to be stubbed only

* setup test finishing. coverage bad

* WIP

* switchMap changed to flatMap

* version bumped up

* got coverage over 80%

* added troubleshooting info into README.md

* consumer coverage 100 percent and unit and coverage added to circle ci

* fixed readme and added istanbul coverage check

* circleci setup

* fix some more comments

* fixed test:xunit script

* ..
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.

2 participants