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.

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

(#170) Organizations test #178

Merged
merged 39 commits into from
May 26, 2024
Merged

(#170) Organizations test #178

merged 39 commits into from
May 26, 2024

Conversation

zniwiarzxxx
Copy link

tests for organizations service

OJ and others added 30 commits May 25, 2024 18:53
(#170) Application: Commands handlers tests
77a86db
(#170) Application: Omiting Dto in code covrege
1fe90d7
(#170) Application: Events external handlers tests
de6a402
(#170) Core: Etities AggregatedId an Organizations tests
8c5a33d
(#170) IdentityContext assebly fix
de1ca65
(#170) IdentityContext assebly another fix
7207093
Add like handler test
5a04eb0
Create comment Handler Tests
9032e88
delete Comment and Like Handler tests
d48c608
ubdate comment Handler test
f058570
external Handlers Tests
ff44767
(#148) core entity comment tests
47ed7aa
(#148) core enttity AggregatedId Tests
94965c7
(#148) infrastructure service MessageBroker Tests
378e8c7
@SaintAngeLs
(#148) udpate the build test script
9901f55
@SaintAngeLs
(#148) udpate the incorrect project directory
37aa37d
@SaintAngeLs
(#148) udpat the script
26f0399
(#148) Infrastucture: removing mongo database and decorators from tes…
ff7fdf9 
…t range
(#148) Core: excluding wrapers from core tests
fde97c3
(#148) Application: excluding dto from application tests range
e4bc2d5
(#148) Infrastucture: excluding logger related files from Infrastruct…
2e2c6c1 
…ure test range
@SaintAngeLs
(#148) update the build-tests script, update tesct coverage statistic…
6337fde 
…s upload to codecov
@SaintAngeLs
(#148) updat ebuild microserices
66ac9d1
@SaintAngeLs
(#148) udpate the history depth checkout
9d38d42
@SaintAngeLs
(#148) specifying base commit
49b3dad
@SaintAngeLs
(#148) adding the fail ci if error
6511d51
@SaintAngeLs
(#148) adding the fail ci if error
9770124
@SaintAngeLs
(#148) ujdpate test script
7bfebab
@SaintAngeLs
(#148) ujdpate test script
2590916
@SaintAngeLs
(#170) testing script update
4ba35b9
@SaintAngeLs SaintAngeLs force-pushed the organizations_test branch from 2a337a5 to 88064d9 Compare May 25, 2024 21:38
@eggwhat
Copy link
Contributor

eggwhat commented May 25, 2024

@zniwiarzxxx mentioned changes are featured in #180

@codecov Codecov
Copy link

codecov bot commented May 26, 2024 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 58.22%. Comparing base (9fead60) to head (fe2c1f7).

Additional details and impacted files 
@@            Coverage Diff             @@
##              dev     #178      +/-   ##
==========================================
+ Coverage   55.82%   58.22%   +2.39%     
==========================================
  Files         165      201      +36     
  Lines        2137     2590     +453     
  Branches      191      234      +43     
==========================================
+ Hits         1193     1508     +315     
- Misses        889     1019     +130     
- Partials       55       63       +8

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

eggwhat
eggwhat approved these changes May 26, 2024
View reviewed changes
Copy link
Contributor

@eggwhat eggwhat left a comment

Choose a reason for hiding this comment

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

Tests work great. I had to add test projects to solution but it should be fixed now.

@SaintAngeLs
Copy link
Owner

I suspect, there is a need to update the scripts.

@SaintAngeLs
Copy link
Owner

I suspect, there is a need to update the scripts.

No, there was no issue.

@SaintAngeLs SaintAngeLs merged commit bf37e4d into dev May 26, 2024
18 checks passed
@eggwhat eggwhat deleted the organizations_test branch May 27, 2024 20:30
@eggwhat eggwhat linked an issue May 30, 2024 that may be closed by this pull request
Unit tests - organizations service #170
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@eggwhat eggwhat eggwhat approved these changes

@SaintAngeLs SaintAngeLs Awaiting requested review from SaintAngeLs SaintAngeLs is a code owner

@olegkiprik olegkiprik Awaiting requested review from olegkiprik

@an2508374 an2508374 Awaiting requested review from an2508374

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Unit tests - organizations service
3 participants
@zniwiarzxxx @eggwhat @SaintAngeLs