Skip to content

Enhancement/unit tests #290

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 40 commits into from
May 28, 2019
Merged

Enhancement/unit tests #290

merged 40 commits into from
May 28, 2019

Conversation

salmamali
Copy link
Contributor

@salmamali salmamali commented May 27, 2019 •
edited
Loading

Add specs on the module for:

  • index.js
  • modules/BugReporting.js
  • modules/Chats.js
  • modules/Replies.js
  • modules/Surveys.js
  • modules/FeatureRequests.js
  • modules/CrashReporting.js
  • modules/NetworkLogger.js
  • utils/XhrNetworkInterceptor.js
  • models/Report.js

Add specs on the sample app for:

  • utils/InstabugUtils.js

Added all tests to run on CI.

salmamali and others added 30 commits May 7, 2019 09:24
@salmamali
✨ Add initial config for circleci
4133788
@salmamali
change node version
a946d52
@salmamali
📝 ci
eef5312
@salmamali
📝 ci
4899402
@salmamali
✨ add workflow and one passing test
6f3e572
@salmamali
📝 fix working directory
c46629b
@salmamali
📝 ci
0927be1
@salmamali
✨ setup the module for tests
03c4ea8
@salmamali
✨ add Chats module unit tests
3f21747
@salmamali
Merge branch 'enhancement/circleci' into enhancement/unit_tests
e49f87b 
# Conflicts:
#	InstabugSample/__tests__/App.js
@salmamali
✨ add mocks for BugReporting, XhrNetworkInterceptor, InstabugUtils, C…
45bc992 
…hats and Replies
@salmamali
✨ add unit tests for bugReporting and replies
d17d6ca
@salmamali
🎨 clean up
d960a37
@salmamali
📝 change ci to run in the module directory instead of the sample app …
c08a907 
…directory
@salmamali
🐛 fix CI path
12ab51d
@salmamali
✨ Replies unit tests
6c2b1ea
@salmamali
✨ surveys unit tests
7b80896
@alyezz
Merge branch 'enhancement/unit_tests' of https://github.com/Instabug/…
aeb5207 
…instabug-reactnative-private into enhancement/unit_tests

# Conflicts:
#	utils/InstabugConstants.js
@salmamali
✨ add Instabug module unit tests
0bab360
@salmamali
🐛 fix incorrect API name
1847432
@salmamali
📝 add missing unit tests
e9cd827
@salmamali
feature requests and report unit tests
f6d9ff0
@salmamali
✨ add xhrNetworkInterceptor unit tests
378b44f
@salmamali
✨ add NetworkLogger unit tests
63c3b1a
@salmamali
✨ add CrashReporting module unit tests
484f3e6
@salmamali
🐛 fix wrong import
7c1c8b9
@salmamali
🐛 fix failing test
5ac077d
@salmamali
✨ job to run sample app tests
d646a18
@salmamali
🐛 fix sample app working directory
b2bd8fc
@salmamali
✨ add global error handler tests in sample app
0845e8e
@salmamali salmamali self-assigned this May 27, 2019
@salmamali
Merge commit 'a7ae9724de2e7d0c5b252bbec77e2cc7ba7205ae' into enhancem…
9e63332 
…ent/unit_tests

# Conflicts:
#	.circleci/config.yml
#	package.json
@salmamali salmamali requested review from alyezz and HeshamMegid May 27, 2019 10:29
@salmamali salmamali merged commit de83081 into master May 28, 2019
@delete-merged-branch delete-merged-branch bot deleted the enhancement/unit_tests branch May 28, 2019 09:14
@salmamali salmamali restored the enhancement/unit_tests branch September 24, 2019 09:58
@alyezz alyezz deleted the enhancement/unit_tests branch September 6, 2020 14:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@HeshamMegid HeshamMegid Awaiting requested review from HeshamMegid

1 more reviewer

@alyezz alyezz alyezz approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees

@salmamali salmamali

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@salmamali @alyezz