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

Added tests of "Notification Service" #27

Open
wants to merge 22 commits into
base: R21BC-245
Choose a base branch
from
Open

Added tests of "Notification Service" #27

wants to merge 22 commits into from

Conversation

warmhmk
Copy link

@warmhmk warmhmk commented Feb 9, 2022

No description provided.

kjpranata and others added 19 commits September 17, 2021 17:26
@kjpranata
still under fixing for mutex lock problem
26547dd
@kjpranata
Still on Progress
07d5846
@kjpranata
minor fix on unique lock
9457b9f
@Cempl
Added basic structures for serdes
beede66
@Cempl
Fixed serialization error for "UnconfirmedRemoved" topic
433e1ce
@kjpranata
Update on Test Case
93c5a35
@kjpranata
Need to check on changes for timer
322db5b
@kjpranata
Need Checking on the Timer
65ba5f5
@kjpranata
Timer Fixed for 4 Notifiers
5ff905c
@kjpranata
Added EXPECT_TRUE test to variable isReceived
be46b17
@kjpranata
Fixed camelCase. Need checking for references.
3baa8d8
@Cempl
Reworked addConfirmedAddedNotifier, waitConfirmedAdded
ffa30a7
@kjpranata
Reworked 4 Test Cases
e530581
@kjpranata
Merge pull request #20 from proximax-storage/R21BC-57-TEST
6d90556 
R21BC-57-TEST
@kjpranata
PartialAddedNotifier still in Progress
bf6d3ae
@warmhmk
fixing error on addStatusNotifier
37db67f
@warmhmk
Revert "fixing error on addStatusNotifier"
06b66b3 
This reverts commit 37db67f.
@warmhmk
fix previous commit
31ebe1e
@warmhmk
addStatusNotifiers
e5896ce
@warmhmk warmhmk requested review from Cempl and otsybizov February 9, 2022 08:15
Cempl
Cempl requested changes Feb 9, 2022
View reviewed changes
Copy link

@Cempl Cempl left a comment

Choose a reason for hiding this comment

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

Remove cmake/git files and change master branch to develop branch please.

@warmhmk warmhmk changed the base branch from master to develop February 9, 2022 08:21
@otsybizov otsybizov changed the base branch from develop to R21BC-245 February 9, 2022 08:47
@otsybizov otsybizov changed the title R21 bc 57 test Added tests of "Notification Service" Feb 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Cempl Cempl Cempl requested changes

@otsybizov otsybizov Awaiting requested review from otsybizov

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

Successfully merging this pull request may close these issues.
3 participants
@warmhmk @Cempl @kjpranata